[PATCH] osmo-sgsn[master]: gtphub: check for gsn_addr_from_sockaddr() errors

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Max gerrit-no-reply at lists.osmocom.org
Wed Dec 20 16:04:45 UTC 2017


Review at  https://gerrit.osmocom.org/5529

gtphub: check for gsn_addr_from_sockaddr() errors

Previously the return value was ignored. Check it and log error message.

Change-Id: If6babdad650dc6a84e6a784fc8d3c5abe146ecc1
---
M src/gprs/gtphub.c
1 file changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-sgsn refs/changes/29/5529/1

diff --git a/src/gprs/gtphub.c b/src/gprs/gtphub.c
index b0e75ca..2ff1b7c 100644
--- a/src/gprs/gtphub.c
+++ b/src/gprs/gtphub.c
@@ -1583,7 +1583,9 @@
 		 * gtphub_handle_buf(), we'd be creating a peer port just to
 		 * expire it right away. */
 		if (hub->sgsn_use_sender && (side_idx == GTPH_SIDE_SGSN)) {
-			gsn_addr_from_sockaddr(&use_addr, &use_port, &from_ctrl->sa);
+			int rc = gsn_addr_from_sockaddr(&use_addr, &use_port, &from_ctrl->sa);
+			if (rc < 0)
+				LOG(LOGL_ERROR, "%s(): failed to obtain GSN address\n", __func__);
 		} else {
 			use_port = gtphub_plane_idx_default_port[plane_idx];
 
@@ -2765,7 +2767,9 @@
 
 	struct gsn_addr gsna;
 	uint16_t port;
-	gsn_addr_from_sockaddr(&gsna, &port, addr);
+	int rc = gsn_addr_from_sockaddr(&gsna, &port, addr);
+	if (rc < 0)
+		LOG(LOGL_ERROR, "%s(): failed to obtain GSN address\n", __func__);
 
 	pa = gtphub_addr_find(bind, &gsna);
 	if (!pa)

-- 
To view, visit https://gerrit.osmocom.org/5529
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If6babdad650dc6a84e6a784fc8d3c5abe146ecc1
Gerrit-PatchSet: 1
Gerrit-Project: osmo-sgsn
Gerrit-Branch: master
Gerrit-Owner: Max <msuraev at sysmocom.de>



More information about the gerrit-log mailing list