diff options
author | Phil Sutter <phil@nwl.cc> | 2018-10-12 17:23:24 +0200 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2018-10-15 13:37:51 +0200 |
commit | c7a5401943df8b6b96f6b5eedd9a1e0013e01d86 (patch) | |
tree | d0a04171cfd7b241d07db380db674c0fdb1c7d10 /COPYING | |
parent | a56fe55dbd3232e70514610a1c2df1d6b15b931f (diff) |
parser_json: Fix for ineffective family value checks
Since handle->family is unsigned, checking for value < 0 never yields
true. Overcome this by changing parse_family() to return an error code
and write the parsed family value into a pointer passed as parameter.
The above change required a bit more cleanup to avoid passing pointers
to signed variables to the function. Also leverage json_parse_family() a
bit more to reduce code side.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'COPYING')
0 files changed, 0 insertions, 0 deletions