diff options
author | Phil Sutter <phil@nwl.cc> | 2018-03-01 15:00:32 +0100 |
---|---|---|
committer | Pablo Neira Ayuso <pablo@netfilter.org> | 2018-03-02 11:46:56 +0100 |
commit | 671851617c8d8c1dfe9822eee8dcc7b827fff850 (patch) | |
tree | 687bef695060da6b37dfcdb48e21f567dc8557ff /src/mnl.c | |
parent | 4342fcbd40534158107ebe6a10e5f7886b3e8ced (diff) |
netlink_delinearize: Fix resource leaks
Most of the cases are basically the same: Error path fails to free the
previously allocated statement or expression. A few cases received
special treatment though:
- In netlink_parse_payload_stmt(), the leak is easily avoided by code
reordering.
- In netlink_parse_exthdr(), there's no point in introducing a goto
label since there is but a single affected error check.
- In netlink_parse_hash() non-error path leaked as well if sreg
contained a concatenated expression.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
Diffstat (limited to 'src/mnl.c')
0 files changed, 0 insertions, 0 deletions