dns: fix cares memory leak

nodejs · Jul 20, 2022 · ce1c1a6 · ce1c1a6
1 parent 95d9140
commit ce1c1a6
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/src/cares_wrap.cc b/src/cares_wrap.cc
@@ -1473,11 +1473,16 @@ void AfterGetAddrInfo(uv_getaddrinfo_t* req, int status, struct addrinfo* res) {
       return Just(true);
     };
 
-    if (add(true, verbatim).IsNothing())
+    if (add(true, verbatim).IsNothing()) {
+      uv_freeaddrinfo(res);
       return;
+    }
+
     if (verbatim == false) {
-      if (add(false, true).IsNothing())
+      if (add(false, true).IsNothing()) {
+        uv_freeaddrinfo(res);
         return;
+      }
     }
 
     // No responses were found to return