aboutsummaryrefslogtreecommitdiff
path: root/src/test
diff options
context:
space:
mode:
authorHarris <brakmic@gmail.com>2020-05-09 11:09:52 +0200
committerHarris <brakmic@gmail.com>2020-05-09 11:09:52 +0200
commit420fa0770f37619bfa29898d59dac45b6a477abb (patch)
tree374e4e73842a177b43f13ef9cc5e4b531296e13a /src/test
parentb55866969e248cbd912a1c298e3ba375685575dc (diff)
fuzz: use std::optional for sep_pos variable
Co-authored-by: MarcoFalke <falke.marco@gmail.com>
Diffstat (limited to 'src/test')
-rw-r--r--src/test/fuzz/asmap_direct.cpp12
1 files changed, 6 insertions, 6 deletions
diff --git a/src/test/fuzz/asmap_direct.cpp b/src/test/fuzz/asmap_direct.cpp
index 790f80237d..6d8a65f5ab 100644
--- a/src/test/fuzz/asmap_direct.cpp
+++ b/src/test/fuzz/asmap_direct.cpp
@@ -6,6 +6,7 @@
#include <test/fuzz/fuzz.h>
#include <cstdint>
+#include <optional>
#include <vector>
#include <assert.h>
@@ -13,20 +14,19 @@
void test_one_input(const std::vector<uint8_t>& buffer)
{
// Encoding: [asmap using 1 bit / byte] 0xFF [addr using 1 bit / byte]
- bool have_sep = false;
- size_t sep_pos;
+ std::optional<size_t> sep_pos_opt;
for (size_t pos = 0; pos < buffer.size(); ++pos) {
uint8_t x = buffer[pos];
if ((x & 0xFE) == 0) continue;
if (x == 0xFF) {
- if (have_sep) return;
- have_sep = true;
- sep_pos = pos;
+ if (sep_pos_opt) return;
+ sep_pos_opt = pos;
} else {
return;
}
}
- if (!have_sep) return; // Needs exactly 1 separator
+ if (!sep_pos_opt) return; // Needs exactly 1 separator
+ const size_t sep_pos{sep_pos_opt.value()};
if (buffer.size() - sep_pos - 1 > 128) return; // At most 128 bits in IP address
// Checks on asmap