lynxis lazus submitted this change.

Approvals:
  laforge: Looks good to me, but someone else must approve
  lynxis lazus: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified

rspro_server: don't access the msg on error cases

The msg is NULL on errors.

Related: SYS#7930
Change-Id: I4e95c2473acd72cd2e01df329691c8f31882b3d9
---
M src/server/rspro_server.c
1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/server/rspro_server.c b/src/server/rspro_server.c
index 70d59c1..a6fd421 100644
--- a/src/server/rspro_server.c
+++ b/src/server/rspro_server.c
@@ -712,7 +712,7 @@
 /* data was received from one of the client connections to the RSPRO socket */
 static int sock_read_cb(struct osmo_stream_srv *peer, int res, struct msgb *msg)
 {
-	enum ipaccess_proto ipa_proto = osmo_ipa_msgb_cb_proto(msg);
+	enum ipaccess_proto ipa_proto;
 	struct rspro_client_conn *conn = osmo_stream_srv_get_data(peer);
 	RsproPDU_t *pdu;
 	int rc;
@@ -722,6 +722,7 @@
 		goto err;
 	}
 
+	ipa_proto = osmo_ipa_msgb_cb_proto(msg);
 	switch (ipa_proto) {
 	case IPAC_PROTO_IPACCESS:
 		rc = _ipa_srv_conn_ccm(conn, msg);

To view, visit change 42601. To unsubscribe, or for help writing mail filters, visit settings.