summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeels Hofmeyr <nhofmeyr@sysmocom.de>2017-06-21 22:54:46 +0200
committerNeels Hofmeyr <nhofmeyr@sysmocom.de>2017-07-03 17:29:05 +0000
commit1f82d0aecfa30884426fdcf583b32826527c639c (patch)
tree99e11e2aeb9b73a5e98c66f49d3da4ab97466519
parent4aaa8abcab71a7257e20c96aa03d4f675b29c57e (diff)
osmo_sock_get_name(): clearly indicate local and remote
In a string like 127.0.0.1:2905<->127.0.0.1:60661 it is hard to tell which is the local part. I'd have expected it on the left, but it is actually on the right. To avoid doubt and bypass bikesheds on which side should be what, clearly mark the two sides as remote and local. (r=127.0.0.1:2905<->l=127.0.0.1:60661) Change-Id: I43dcc6a1906429bd0955fd7fe2eb5b8495b592d8
-rw-r--r--src/socket.c4
-rw-r--r--tests/socket/socket_test.c6
2 files changed, 5 insertions, 5 deletions
diff --git a/src/socket.c b/src/socket.c
index b74edd43..28d9b1c4 100644
--- a/src/socket.c
+++ b/src/socket.c
@@ -607,11 +607,11 @@ char *osmo_sock_get_name(void *ctx, int fd)
if (rc < 0)
goto local_only;
- return talloc_asprintf(ctx, "(%s:%s<->%s:%s)", hostbuf_r, portbuf_r,
+ return talloc_asprintf(ctx, "(r=%s:%s<->l=%s:%s)", hostbuf_r, portbuf_r,
hostbuf_l, portbuf_l);
local_only:
- return talloc_asprintf(ctx, "(NULL<->%s:%s)", hostbuf_l, portbuf_l);
+ return talloc_asprintf(ctx, "(r=NULL<->l=%s:%s)", hostbuf_l, portbuf_l);
}
#endif /* HAVE_SYS_SOCKET_H */
diff --git a/tests/socket/socket_test.c b/tests/socket/socket_test.c
index b56d50c0..11ef8da4 100644
--- a/tests/socket/socket_test.c
+++ b/tests/socket/socket_test.c
@@ -49,7 +49,7 @@ static int test_sockinit(void)
* as apparently that won't work on FreeBSD if there's only one
* address (e.g. 127.0.0.1) assigned to the entire system, like
* the Osmocom FreeBSD build slaves */
- OSMO_ASSERT(!strncmp(name, "(NULL<->", 7));
+ OSMO_ASSERT(!strncmp(name, "(r=NULL<->", 9));
talloc_free(name);
/* expect it to be blocking */
rc = fcntl(fd, F_GETFL);
@@ -87,7 +87,7 @@ static int test_sockinit2(void)
* as apparently that won't work on FreeBSD if there's only one
* address (e.g. 127.0.0.1) assigned to the entire system, like
* the Osmocom FreeBSD build slaves */
- OSMO_ASSERT(!strncmp(name, "(NULL<->", 7));
+ OSMO_ASSERT(!strncmp(name, "(r=NULL<->", 9));
talloc_free(name);
/* expect it to be blocking */
rc = fcntl(fd, F_GETFL);
@@ -116,7 +116,7 @@ static int test_sockinit2(void)
/* For some reason, on the jenkins.osmocom.org build slave with
* FreeBSD 10 inside a jail, it fails. Works fine on laforge's
* FreeBSD 10 or 11 VM at home */
- OSMO_ASSERT(!strncmp(name, "(127.0.0.1:53<->127.0.0.1", 25));
+ OSMO_ASSERT(!strncmp(name, "(r=127.0.0.1:53<->l=127.0.0.1", 29));
#endif
talloc_free(name);