summaryrefslogtreecommitdiffstats
path: root/include/osmocom/core/logging.h
diff options
context:
space:
mode:
authorHarald Welte <laforge@gnumonks.org>2016-11-11 15:10:33 +0100
committerHarald Welte <laforge@gnumonks.org>2016-11-11 14:49:31 +0000
commit6176b6e0924f2bd7d212472bcba9dddf8ac32f51 (patch)
treee91f5aaf6a4bbdddc018da16ad3e495bbe6082c0 /include/osmocom/core/logging.h
parentc4193d30e82307b87a7a7bce3ea5ed48f2f2f716 (diff)
bssgp_rx_paging(): Fix parsing of P-TMSI IE in Paging message
This was actually discovered by the following compiler warning in gcc-6.2.0: CC gprs_bssgp_bss.lo gprs_bssgp_bss.c: In function ‘bssgp_rx_paging’: gprs_bssgp_bss.c:544:2: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation] if (TLVP_PRESENT(&tp, BSSGP_IE_TMSI) && ^~ gprs_bssgp_bss.c:548:3: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’ *(pinfo->ptmsi) = ntohl(*(uint32_t *) ^ This is an actual bug. If we recevied a BSSGP Paging Request without P-TMSI, we might crash or report some random memory as P-TMSI to the caller in the output data structure. Change-Id: Ib4f307827cd7cccc91c1415a6fb5428d7cf8416d
Diffstat (limited to 'include/osmocom/core/logging.h')
0 files changed, 0 insertions, 0 deletions