[PATCH] openbsc[master]: gprs: Fix compiler warning about uninitalized cause code

dexter gerrit-no-reply at lists.osmocom.org
Wed Jan 25 11:27:55 UTC 2017

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

gprs: Fix compiler warning about uninitalized cause code

in gprs_gmm.c:gsm48_rx_gmm_ra_upd_req the variable reject_cause
is not initalized, which is ok, since it gets initalized before
the jump into the "rejected" path. However, the compiler still
throws a warning. This commit fixes the problem by preinitalizing
the reject_cause to GMM_CAUSE_PROTO_ERR_UNSPEC

Change-Id: I84cffb631e4cad3d4748512b47e3876208f53727
M openbsc/src/gprs/gprs_gmm.c
1 file changed, 1 insertion(+), 1 deletion(-)

  git pull ssh://gerrit.osmocom.org:29418/openbsc refs/changes/61/1661/1

diff --git a/openbsc/src/gprs/gprs_gmm.c b/openbsc/src/gprs/gprs_gmm.c
index 88be512..dc73693 100644
--- a/openbsc/src/gprs/gprs_gmm.c
+++ b/openbsc/src/gprs/gprs_gmm.c
@@ -1415,7 +1415,7 @@
 	struct gprs_ra_id old_ra_id;
 	struct tlv_parsed tp;
 	uint8_t upd_type;
-	enum gsm48_gmm_cause reject_cause;
+	enum gsm48_gmm_cause reject_cause = GMM_CAUSE_PROTO_ERR_UNSPEC;
 	int rc;
 	/* TODO: In iu mode - handle follow-on request */

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I84cffb631e4cad3d4748512b47e3876208f53727
Gerrit-PatchSet: 1
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: dexter <pmaier at sysmocom.de>

More information about the gerrit-log mailing list