diff options
| author | SChernykh <sergey.v.chernykh@gmail.com> | 2020-12-08 18:27:19 +0100 |
|---|---|---|
| committer | SChernykh <sergey.v.chernykh@gmail.com> | 2020-12-08 18:43:44 +0100 |
| commit | c0c75ac19d77a542abacd27796364c6518f7ae59 (patch) | |
| tree | 0c14970ec93dd12072549cece25f150c1a660d37 /contrib | |
| parent | Merge pull request #7072 (diff) | |
| download | monero-c0c75ac19d77a542abacd27796364c6518f7ae59.tar.xz | |
Fixed issues found by static analysis
- rolling_median: tried to free uninitialized pointer in a constructor
- net_node.inl: erase-remove idiom was used incorrectly. remove_if doesn't actually remove elements, see http://cpp.sh/6fcjv
- bulletproofs.cc: call to sizeof() instead of vector.size(), luckily it only impacts performance and not code logic there
Diffstat (limited to 'contrib')
| -rw-r--r-- | contrib/epee/include/rolling_median.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/contrib/epee/include/rolling_median.h b/contrib/epee/include/rolling_median.h index 088a71d3e..877814e57 100644 --- a/contrib/epee/include/rolling_median.h +++ b/contrib/epee/include/rolling_median.h @@ -141,7 +141,6 @@ public: rolling_median_t(rolling_median_t &&m) { - free(data); memcpy(this, &m, sizeof(rolling_median_t)); m.data = NULL; } |