diff options
| author | Sven Eckelmann <sven@narfation.org> | 2017-12-26 12:05:17 +0100 |
|---|---|---|
| committer | Sven Eckelmann <sven@narfation.org> | 2017-12-26 13:09:35 +0100 |
| commit | b2e7b1a301a656c318596aabf1f94957c60a8463 (patch) | |
| tree | 8bb57ab4cfb0e35c9e5ccdd002db927ea11512cf /batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch | |
| parent | 7b394394b534119d14ca1a049d882d32092b7382 (diff) | |
batctl: Merge bugfixes from v2017.4
* Handle allocation error in vlan_get_link_parse
* Simplify concatenation of pathnames
* tcpdump: Fix types for for TT v1
* Free nl_sock when if_nametoindex failed
* Free nl_sock on genl_ctrl_resolve error
* Handle nl_cb_alloc errors
* Handle nl_socket_alloc errors
* Handle nlmsg_alloc errors
* Handle allocation error for path_buff
* Handle failure during hash_iterator allocation
* Print dummy value when localtime failed
* suppress implicit-fallthrough compiler warning
* change PATH_BUFF_LEN to maximal possible value
Signed-off-by: Sven Eckelmann <sven@narfation.org>
Diffstat (limited to 'batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch')
| -rw-r--r-- | batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch | 50 |
1 files changed, 50 insertions, 0 deletions
diff --git a/batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch b/batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch new file mode 100644 index 0000000..0d78f7f --- /dev/null +++ b/batctl/patches/0007-batctl-Handle-nl_socket_alloc-errors.patch @@ -0,0 +1,50 @@ +From: Sven Eckelmann <sven.eckelmann@openmesh.com> +Date: Thu, 23 Nov 2017 15:04:39 +0100 +Subject: [PATCH] batctl: Handle nl_socket_alloc errors + +nl_socket_alloc may return NULL on errors. The processing has to be aborted +when this happens. + +Fixes: d8dd1ff1a0fe ("batctl: Use netlink to replace some of debugfs") +Signed-off-by: Sven Eckelmann <sven.eckelmann@openmesh.com> +Signed-off-by: Simon Wunderlich <sw@simonwunderlich.de> + +Origin: upstream, https://git.open-mesh.org/batctl.git/commit/bc6cd37e0d3ce08e3b89e3123ffa87dc55f24c09 +--- + netlink.c | 9 +++++++++ + 1 file changed, 9 insertions(+) + +diff --git a/netlink.c b/netlink.c +index 64afeedac46bf3eab14a1d89d7db4491fbef8d81..107ca52a4866e25b7b04428d770a885ca4e826d2 100644 +--- a/netlink.c ++++ b/netlink.c +@@ -295,6 +295,9 @@ static char *netlink_get_info(int ifindex, uint8_t nl_cmd, const char *header) + }; + + sock = nl_socket_alloc(); ++ if (!sock) ++ return NULL; ++ + genl_connect(sock); + + family = genl_ctrl_resolve(sock, BATADV_NL_NAME); +@@ -397,6 +400,9 @@ int netlink_print_routing_algos(void) + }; + + sock = nl_socket_alloc(); ++ if (!sock) ++ return -ENOMEM; ++ + genl_connect(sock); + + family = genl_ctrl_resolve(sock, BATADV_NL_NAME); +@@ -1104,6 +1110,9 @@ static int netlink_print_common(char *mesh_iface, char *orig_iface, + int family; + + sock = nl_socket_alloc(); ++ if (!sock) ++ return -ENOMEM; ++ + genl_connect(sock); + + family = genl_ctrl_resolve(sock, BATADV_NL_NAME); |
