Fix segfault when destructing udp_client

nikita240 · nikita240 · commit aae734a36115 · 2022-04-05T10:26:00.000-07:00
diff --git a/net_port/net_port.cpp b/net_port/net_port.cpp
@@ -23,7 +23,7 @@ const char yes = 1;
 
 net_port::net_port()
 {
-
+    result_list = NULL;
 }
 
 //net_port (constructor)
@@ -36,7 +36,10 @@ net_port::net_port(const char * client_name, const char * port)
 
 net_port::~net_port()
 {
-    freeaddrinfo(result_list);
+    if(result_list)
+    {
+        freeaddrinfo(result_list);
+    }
 }
 
 bool net_port::udp_client(const char * client_name, const char * port)
@@ -76,6 +79,7 @@ bool net_port::udp_client(const char * client_name, const char * port)
     {
         memcpy(&addrDest, result_list->ai_addr, result_list->ai_addrlen);
         freeaddrinfo(result_list);
+        result_list = NULL;
         return(true);
     }
     else

Original file line number	Diff line number	Diff line change
`@@ -23,7 +23,7 @@ const char yes = 1;`
`23`	`23`
`24`	`24`	`net_port::net_port()`
`25`	`25`	`{`
`26`		`-`
	`26`	`+ result_list = NULL;`
`27`	`27`	`}`
`28`	`28`
`29`	`29`	`//net_port (constructor)`
`@@ -36,7 +36,10 @@ net_port::net_port(const char * client_name, const char * port)`
`36`	`36`
`37`	`37`	`net_port::~net_port()`
`38`	`38`	`{`
`39`		`- freeaddrinfo(result_list);`
	`39`	`+ if(result_list)`
	`40`	`+ {`
	`41`	`+ freeaddrinfo(result_list);`
	`42`	`+ }`
`40`	`43`	`}`
`41`	`44`
`42`	`45`	`bool net_port::udp_client(const char * client_name, const char * port)`
`@@ -76,6 +79,7 @@ bool net_port::udp_client(const char * client_name, const char * port)`
`76`	`79`	`{`
`77`	`80`	`memcpy(&addrDest, result_list->ai_addr, result_list->ai_addrlen);`
`78`	`81`	`freeaddrinfo(result_list);`
	`82`	`+ result_list = NULL;`
`79`	`83`	`return(true);`
`80`	`84`	`}`
`81`	`85`	`else`