agents: fix crash in StatsDAgent

With some specific udp send errors, it's not possible to retrieve the peername. Handle this case to avoid crashes. Fixes: #31 PR-URL: #32 Reviewed-by: Trevor Norris <[email protected]>
nodesource · Dec 1, 2023 · 5a47d7e · 5a47d7e
1 parent b2c1bb3
commit 5a47d7e
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/agents/statsd/src/statsd_agent.cc b/agents/statsd/src/statsd_agent.cc
@@ -264,9 +264,13 @@ void StatsDUdp::write_cb_(nsuv::ns_udp_send* req, int status, StatsDUdp* udp) {
     struct sockaddr_storage ss;
     struct sockaddr* addr = reinterpret_cast<struct sockaddr*>(&ss);
     int len = sizeof(ss);
-    ASSERT_EQ(0, uv_udp_getpeername(req->handle(), addr, &len));
-    Debug("Error '%s' sending data to: %s.\n",
-          uv_err_name(status), addr_to_string(addr).c_str());
+    int r = uv_udp_getpeername(req->handle(), addr, &len);
+    if (r == 0) {
+      Debug("Error '%s' sending data to: %s.\n",
+            uv_err_name(status), addr_to_string(addr).c_str());
+    } else {
+      Debug("Error '%s' sending data to\n", uv_err_name(status));
+    }
   }
 
   udp_req_data_tup* req_data = req->get_data<udp_req_data_tup>();