diff options
author | Phil Sutter <phil@nwl.cc> | 2021-08-31 12:29:43 +0200 |
---|---|---|
committer | Phil Sutter <phil@nwl.cc> | 2021-08-31 18:19:04 +0200 |
commit | e438b9766fcc86d9847312ff05f1d1dac61acf1f (patch) | |
tree | 2a7a550cd343caadb67a2e2331d533c56dc0f4da /iptables/xshared.c | |
parent | ca11c7b7036b5821c17b8d08dc2a29f55b461a93 (diff) |
nft: Use xtables_{m,c}alloc() everywhere
Make use of libxtables allocators where sensible to have implicit error
checking. Leave library-internal calls in place to not create unexpected
program exit points for users, apart from xt_xlate_alloc() as that
function called xtables_error() in error case which exits by itself
already.
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'iptables/xshared.c')
-rw-r--r-- | iptables/xshared.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/iptables/xshared.c b/iptables/xshared.c index ed3e9c5a..2d3ef679 100644 --- a/iptables/xshared.c +++ b/iptables/xshared.c @@ -220,9 +220,7 @@ void xs_init_target(struct xtables_target *target) { if (target->udata_size != 0) { free(target->udata); - target->udata = calloc(1, target->udata_size); - if (target->udata == NULL) - xtables_error(RESOURCE_PROBLEM, "malloc"); + target->udata = xtables_calloc(1, target->udata_size); } if (target->init != NULL) target->init(target->t); @@ -238,9 +236,7 @@ void xs_init_match(struct xtables_match *match) * Same goes for target. */ free(match->udata); - match->udata = calloc(1, match->udata_size); - if (match->udata == NULL) - xtables_error(RESOURCE_PROBLEM, "malloc"); + match->udata = xtables_calloc(1, match->udata_size); } if (match->init != NULL) match->init(match->m); |