wireguard: Fix use after free

Use freeaddrinfo after rp has been used. rp points to an element in results. (cherry picked from commit 4b8b2e3)
sailfishos · Jul 10, 2024 · d37d965 · d37d965
1 parent 8b6b44e
commit d37d965
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/connman/vpn/plugins/wireguard.c b/connman/vpn/plugins/wireguard.c
@@ -145,12 +145,14 @@ static int parse_endpoint(const char *host, const char *port, wg_peer *peer)
 
 		close(sk);
 	}
-	freeaddrinfo(result);
 
-	if (!rp)
+	if (!rp) {
+		freeaddrinfo(result);
 		return -EINVAL;
+	}
 
 	memcpy(&peer->endpoint.addr, rp->ai_addr, rp->ai_addrlen);
+	freeaddrinfo(result);
 
 	return 0;
 }