src: improve error handling in tcp_wrap.cc

jasnell · aduh95 · commit f845ad953e80 · 2025-03-09T10:58:32.000+01:00
PR-URL: #57211 Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> Reviewed-By: Matthew Aitken <maitken033380023@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
diff --git a/src/tcp_wrap.cc b/src/tcp_wrap.cc
@@ -390,8 +390,9 @@ MaybeLocal<Object> AddressToJS(Environment* env,
 
   int port;
 
-  if (info.IsEmpty())
+  if (info.IsEmpty()) {
     info = Object::New(env->isolate());
+  }
 
   switch (addr->sa_family) {
   case AF_INET6:
@@ -413,32 +414,45 @@ MaybeLocal<Object> AddressToJS(Environment* env,
       }
     }
     port = ntohs(a6->sin6_port);
-    info->Set(env->context(),
-              env->address_string(),
-              OneByteString(env->isolate(), ip)).Check();
-    info->Set(env->context(), env->family_string(), env->ipv6_string()).Check();
-    info->Set(env->context(),
-              env->port_string(),
-              Integer::New(env->isolate(), port)).Check();
+    if (info->Set(env->context(),
+                  env->address_string(),
+                  OneByteString(env->isolate(), ip))
+            .IsNothing() ||
+        info->Set(env->context(), env->family_string(), env->ipv6_string())
+            .IsNothing() ||
+        info->Set(env->context(),
+                  env->port_string(),
+                  Integer::New(env->isolate(), port))
+            .IsNothing()) {
+      return {};
+    }
     break;
 
   case AF_INET:
     a4 = reinterpret_cast<const sockaddr_in*>(addr);
     uv_inet_ntop(AF_INET, &a4->sin_addr, ip, sizeof ip);
     port = ntohs(a4->sin_port);
-    info->Set(env->context(),
-              env->address_string(),
-              OneByteString(env->isolate(), ip)).Check();
-    info->Set(env->context(), env->family_string(), env->ipv4_string()).Check();
-    info->Set(env->context(),
-              env->port_string(),
-              Integer::New(env->isolate(), port)).Check();
+    if (info->Set(env->context(),
+                  env->address_string(),
+                  OneByteString(env->isolate(), ip))
+            .IsNothing() ||
+        info->Set(env->context(), env->family_string(), env->ipv4_string())
+            .IsNothing() ||
+        info->Set(env->context(),
+                  env->port_string(),
+                  Integer::New(env->isolate(), port))
+            .IsNothing()) {
+      return {};
+    }
     break;
 
   default:
-    info->Set(env->context(),
-              env->address_string(),
-              String::Empty(env->isolate())).Check();
+    if (info->Set(env->context(),
+                  env->address_string(),
+                  String::Empty(env->isolate()))
+            .IsNothing()) {
+      return {};
+    }
   }
 
   return scope.Escape(info);
