diff options
author | Pascal Quantin <pascal.quantin@gmail.com> | 2017-08-28 21:36:37 +0200 |
---|---|---|
committer | Dario Lombardo <lomato@gmail.com> | 2017-08-29 07:44:28 +0000 |
commit | fa8a6a3eee3a6ce938cec4c1d8cc0540b27c451a (patch) | |
tree | 7045b7e0bccd938fbd9d13a6f02afe2dd821d7bc /extcap.c | |
parent | 7a71ec887ce78a48933d02e55d3b6d2b3ac49cfb (diff) |
extcap: check that argument parameter is not null before adding it to command line
Without this test, we add an argument with an empty parameter
Bug: 14006
Change-Id: I7e5313f652691808735f54b533cf5a7f7adeeae1
Reviewed-on: https://code.wireshark.org/review/23271
Reviewed-by: Pascal Quantin <pascal.quantin@gmail.com>
Petri-Dish: Pascal Quantin <pascal.quantin@gmail.com>
Tested-by: Petri Dish Buildbot <buildbot-no-reply@wireshark.org>
Reviewed-by: Dario Lombardo <lomato@gmail.com>
Diffstat (limited to 'extcap.c')
-rw-r--r-- | extcap.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -1316,7 +1316,7 @@ GPtrArray *extcap_prepare_arguments(interface_options *interface_opts) } else { - if (stored) { + if (stored && strlen(stored) > 0) { add_arg(arg_iter->call); add_arg(stored); } |