[posix] fixed uninitialized pointer read (openthread#9552)

This commit added validation for whether the pointer (reqs) being freed is nullptr or not.
hastigondaliya · Oct 25, 2023 · 0370c5e · 0370c5e
1 parent 9106817
commit 0370c5e
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/src/posix/platform/infra_if.cpp b/src/posix/platform/infra_if.cpp
@@ -671,9 +671,9 @@ void InfraNetif::DiscoverNat64PrefixDone(union sigval sv)
 
 otError InfraNetif::DiscoverNat64Prefix(uint32_t aInfraIfIndex)
 {
-    otError          error = OT_ERROR_NONE;
-    struct addrinfo *hints = nullptr;
-    struct gaicb    *reqs[1];
+    otError          error   = OT_ERROR_NONE;
+    struct addrinfo *hints   = nullptr;
+    struct gaicb    *reqs[1] = {nullptr};
     struct sigevent  sig;
     int              status;
 
@@ -710,7 +710,10 @@ otError InfraNetif::DiscoverNat64Prefix(uint32_t aInfraIfIndex)
         {
             freeaddrinfo(hints);
         }
-        free(reqs[0]);
+        if (reqs[0])
+        {
+            free(reqs[0]);
+        }
     }
     return error;
 }