[PATCH] nsiproxy: Fix memory leaks on error paths in get_ipv6_addr_scope_table (cppcheck)

Alex Henrie alexhenrie24 at gmail.com
Sun Dec 19 22:55:31 CST 2021


Signed-off-by: Alex Henrie <alexhenrie24 at gmail.com>
---
 dlls/nsiproxy.sys/tcp.c | 8 +++++---
 1 file changed, 5 insertions(+), 3 deletions(-)

diff --git a/dlls/nsiproxy.sys/tcp.c b/dlls/nsiproxy.sys/tcp.c
index 6dd0aaa9b45..0cd81ae86e5 100644
--- a/dlls/nsiproxy.sys/tcp.c
+++ b/dlls/nsiproxy.sys/tcp.c
@@ -235,7 +235,7 @@ static inline MIB_TCP_STATE tcp_state_to_mib_state( int state )
 
 struct ipv6_addr_scope *get_ipv6_addr_scope_table( unsigned int *size )
 {
-    struct ipv6_addr_scope *table = NULL;
+    struct ipv6_addr_scope *table = NULL, *new_table;
     unsigned int table_size = 0, num = 0;
 
 #ifdef __linux__
@@ -260,11 +260,12 @@ struct ipv6_addr_scope *get_ipv6_addr_scope_table( unsigned int *size )
             {
                 if (!table_size) table_size = 4;
                 else table_size *= 2;
-                if (!(table = realloc( table, table_size * sizeof(table[0]) )))
+                if (!(new_table = realloc( table, table_size * sizeof(table[0]) )))
                 {
                     fclose( fp );
                     goto failed;
                 }
+                table = new_table;
             }
 
             entry = table + num - 1;
@@ -292,11 +293,12 @@ struct ipv6_addr_scope *get_ipv6_addr_scope_table( unsigned int *size )
             {
                 if (!table_size) table_size = 4;
                 else table_size *= 2;
-                if (!(table = realloc( table, table_size * sizeof(table[0]) )))
+                if (!(new_table = realloc( table, table_size * sizeof(table[0]) )))
                 {
                     freeifaddrs( addrs );
                     goto failed;
                 }
+                table = new_table;
             }
 
             sin6 = (struct sockaddr_in6 *)cur->ifa_addr;
-- 
2.34.1




More information about the wine-devel mailing list