[PATCH mdnsd 04/10] interface: fix possible null pointer dereference

Petr Štetiar ynezz at true.cz
Tue Oct 13 09:36:15 EDT 2020


Fixes following issue reported by clang-12 static analyzer:

 mdnsd/interface.c:250:6: warning: Access to field 'ipi_ifindex' results in a dereference of a null pointer (loaded from variable 'inp')
         if (inp->ipi_ifindex != iface->ifindex)
             ^~~~~~~~~~~~~~~~

 mdnsd/interface.c:323:6: warning: Access to field 'ipi6_ifindex' results in a dereference of a null pointer (loaded from variable 'inp')
         if (inp->ipi6_ifindex != iface->ifindex)
             ^~~~~~~~~~~~~~~~~

Signed-off-by: Petr Štetiar <ynezz at true.cz>
---
 interface.c | 6 ++++++
 1 file changed, 6 insertions(+)

diff --git a/interface.c b/interface.c
index 690e8579a44c..5fb9bad072ed 100644
--- a/interface.c
+++ b/interface.c
@@ -233,6 +233,9 @@ read_socket4(struct uloop_fd *u, unsigned int events)
 		}
 	}
 
+	if (!inp)
+		return;
+
 	if (debug > 1) {
 		char buf[256];
 
@@ -308,6 +311,9 @@ read_socket6(struct uloop_fd *u, unsigned int events)
 		}
 	}
 
+	if (!inp)
+		return;
+
 	if (debug > 1) {
 		char buf[256];
 



More information about the openwrt-devel mailing list