[MERGED] openbsc[master]: gprs_sgsn.c: initialize ptmsi with 0xdeadbeef

[Harald Welte]

Harald Welte has submitted this change and it was merged.

Change subject: gprs_sgsn.c: initialize ptmsi with 0xdeadbeef
......................................................................


gprs_sgsn.c: initialize ptmsi with 0xdeadbeef

Fix uninitialized memory access warning.
"Conditional jump or move depends on uninitialised value"
Found by valgrind.

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

Approvals:
  Harald Welte: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/openbsc/src/gprs/gprs_sgsn.c b/openbsc/src/gprs/gprs_sgsn.c
index 727524e..5d9af78 100644
--- a/openbsc/src/gprs/gprs_sgsn.c
+++ b/openbsc/src/gprs/gprs_sgsn.c
@@ -611,7 +611,7 @@
 uint32_t sgsn_alloc_ptmsi(void)
 {
 	struct sgsn_mm_ctx *mm;
-	uint32_t ptmsi;
+	uint32_t ptmsi = 0xdeadbeef;
 	int max_retries = 100;
 
 restart:

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibc2d585c5db899e6af20104211e32faf3822633a
Gerrit-PatchSet: 4
Gerrit-Project: openbsc
Gerrit-Branch: master
Gerrit-Owner: lynxis lazus <lynxis at fe80.eu>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Holger Freyther <holger at freyther.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: lynxis lazus <lynxis at fe80.eu>