diff options
| author | Phil Sutter <phil@nwl.cc> | 2023-01-10 22:36:58 +0100 |
|---|---|---|
| committer | Phil Sutter <phil@nwl.cc> | 2023-01-13 17:11:18 +0100 |
| commit | 927d5674e7bf656428f97c54c9171006e8c3c75e (patch) | |
| tree | 0d5435ea9cefd280f5362029aeb883fbc7b6568c /src/netlink.c | |
| parent | d4d47e5bdf943be494aeb5d5a29b8f5212acbddf (diff) | |
netlink: Fix for potential NULL-pointer deref
If memory allocation fails, calloc() returns NULL which was not checked
for. The code seems to expect zero array size though, so simply
replacing this call by one of the x*calloc() ones won't work. So guard
the call also by a check for 'len'.
Fixes: db0697ce7f602 ("src: support for flowtable listing")
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'src/netlink.c')
| -rw-r--r-- | src/netlink.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/netlink.c b/src/netlink.c index 51de9c9c..efae1251 100644 --- a/src/netlink.c +++ b/src/netlink.c @@ -1790,7 +1790,8 @@ netlink_delinearize_flowtable(struct netlink_ctx *ctx, while (dev_array[len]) len++; - flowtable->dev_array = calloc(1, len * sizeof(char *)); + if (len) + flowtable->dev_array = xmalloc(len * sizeof(char *)); for (i = 0; i < len; i++) flowtable->dev_array[i] = xstrdup(dev_array[i]); |