Attention is currently required from: fixeria.
iedemam has posted comments on this change. ( https://gerrit.osmocom.org/c/osmo-bsc/+/31617 )
Change subject: utils: fix incorrect string checks in meas_db_insert() ......................................................................
Patch Set 1: Code-Review+1
(2 comments)
Patchset:
PS1: Good catch, apologies for the miss.
This looks ok to me, only a note about how this check was performed before.
File src/utils/meas_db.c:
https://gerrit.osmocom.org/c/osmo-bsc/+/31617/comment/50de4a0d_fcfccde0 PS1, Line 97: if (mfm->imsi[0] != '\0') The original checks before my refactor were using strlen(). Maybe that's cleaner?