diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2022-09-28 14:33:12 +1000 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2022-09-29 12:21:41 +0200 |
commit | 84fec4e998b625f36f2d89dd9784ed4bb393631a (patch) | |
tree | 10a550d8ffe97e3d3f04e00ec67b30b93a28b3b8 /icmp.c | |
parent | a1a058533c0cfaee621bf5d49ccd7d1e038550d9 (diff) | |
download | passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar.gz passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar.bz2 passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar.lz passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar.xz passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.tar.zst passt-84fec4e998b625f36f2d89dd9784ed4bb393631a.zip |
Clean up parsing of port ranges
conf_ports() parses ranges of ports for the -t, -u, -T and -U options.
The code is quite difficult to the follow, to the point that clang-tidy
and cppcheck disagree on whether one of the pointers can be NULL at some
points.
Rework the code with the use of two new helper functions:
* parse_port_range() operates a bit like strtoul(), but can parse a whole
port range specification (e.g. '80' or '1000-1015')
* next_chunk() does the necessary wrapping around strchr() to advance to
just after the next given delimiter, while cleanly handling if there
are no more delimiters
The new version is easier to follow, and also removes some cppcheck
warnings.
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'icmp.c')
0 files changed, 0 insertions, 0 deletions