cellular-infrastructure
/
osmo-pcu
mirror of https://gerrit.osmocom.org/osmo-pcu
9
0
Fork 1
Code Issues Releases Wiki Activity

pcu_l1_if.cpp: fix NULL-pointer dereference in imsi2paging_group() 

Passing NULL to strlen() would lead to a segmentation fault.

Change-Id: I838e3a21a3b25c2bc8260f67d156c6cc284f4456
Fixes: CID#207484
This commit is contained in:
Vadim Yanitskiy 2020-01-16 00:25:48 +07:00
parent 771de1f439
commit bd0b0b3242
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/pcu_l1_if.cpp
View File

@ -66,9 +66,10 @@ extern void *tall_pcu_ctx;
uint16_t imsi2paging_group(const char* imsi)
{
uint16_t pgroup = 0;
size_t len = strlen(imsi);
size_t len;
if (!imsi || len < PAGING_GROUP_LEN)
len = (imsi != NULL) ? strlen(imsi) : 0;
if (len < PAGING_GROUP_LEN)
return 0xFFFF;
imsi += len - PAGING_GROUP_LEN;