diff mbox

osmo-pcap[master]: sock_src_init(): Don't freeaddrinfo() undefined src_result

Message ID gerrit.1500635490647.I3b6778d9110583ecb1daec59ef2c86465d5818b9@gerrit.osmocom.org
State New
Headers show

Commit Message

gerrit-no-reply@lists.osmocom.org July 21, 2017, 11:11 a.m. UTC
Review at  https://gerrit.osmocom.org/3324

sock_src_init(): Don't freeaddrinfo() undefined src_result

src_result is only valid "if (src)", so we cannot unconditionally
free it:

(gdb) bt
    host=0x52 <error: Cannot access memory at address 0x52>, src=0x0)
    at /usr/src/debug/osmo-pcap/0.0.6+gitrAUTOINC+4776b2972e-r1d/git/src/osmo_client_network.c:165

Change-Id: I3b6778d9110583ecb1daec59ef2c86465d5818b9
M src/osmo_client_network.c
1 file changed, 2 insertions(+), 1 deletion(-)

  git pull ssh://gerrit.osmocom.org:29418/osmo-pcap refs/changes/24/3324/1


gerrit-no-reply@lists.osmocom.org July 21, 2017, 11:55 a.m. UTC | #1
Patch Set 1: Code-Review+1


File src/osmo_client_network.c:

Line 47:  * Move to libosmocore... if the api makes source
s/source/sense/ lol...

Line 165: 	if (src)
Why not src_result directly?
gerrit-no-reply@lists.osmocom.org July 21, 2017, 1:17 p.m. UTC | #2
Patch Set 1:

(1 comment)

File src/osmo_client_network.c:

Line 165: 	if (src)
> Why not src_result directly?
because it's not initialized with NULL, so we cannot check if it contains random chunk or valid data.  Hence I used the same condition that you use before getaddrinfo().
gerrit-no-reply@lists.osmocom.org July 21, 2017, 1:24 p.m. UTC | #3
Patch Set 1: Code-Review+2

> (1 comment)

This explains the crash. Do you have a backtrace as well? It would be interesting to see why it is NULL. But very minor issue so let's not lose too much time here.
diff mbox


diff --git a/src/osmo_client_network.c b/src/osmo_client_network.c
index 937caa0..27c649a 100644
--- a/src/osmo_client_network.c
+++ b/src/osmo_client_network.c
@@ -162,7 +162,8 @@ 
-	freeaddrinfo(src_result);
+	if (src)
+		freeaddrinfo(src_result);
 	if (rp == NULL) {
 		fprintf(stderr, "unable to connect/bind socket: %s:%u: %s\n",