diff options
| author | Tobias Klauser <tklauser@distanz.ch> | 2013-10-10 11:14:26 +0200 |
|---|---|---|
| committer | Tobias Klauser <tklauser@distanz.ch> | 2013-10-10 11:19:25 +0200 |
| commit | bed9b6bb58f83775a71361ef9f4f11f61afe452c (patch) | |
| tree | e973b3d3231378868c91391c556211d91b3b8a6a | |
| parent | 6e1ffd6cd41ba4ffb8fbc261c79ad124aa293253 (diff) | |
netsniff-ng: Don't modify optarg/argv
We shouldn't modify optarg (and thus argv) since it's e.g. used to
display the commandline string in `ps'. Since strtoul() reads until it
encounters the first non-numeric character and ignores the rest, we can
just revert from setting a NULL byte after the numeric part of the
string.
Reported-by: Jon Schipp <jonschipp@gmail.com>
Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
| -rw-r--r-- | netsniff-ng.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/netsniff-ng.c b/netsniff-ng.c index 50a9f75..c5966b1 100644 --- a/netsniff-ng.c +++ b/netsniff-ng.c @@ -1215,7 +1215,6 @@ int main(int argc, char **argv) ctx.reserve_size = 1 << 30; else panic("Syntax error in ring size param!\n"); - *ptr = 0; ctx.reserve_size *= strtoul(optarg, NULL, 0); break; @@ -1301,7 +1300,6 @@ int main(int argc, char **argv) panic("Syntax error in time/size param!\n"); } - *ptr = 0; ctx.dump_interval *= strtoul(optarg, NULL, 0); break; case 'V': |