diff options
| author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2014-09-22 09:28:27 +0200 |
|---|---|---|
| committer | Holger Hans Peter Freyther <holger@moiji-mobile.com> | 2014-10-09 17:47:58 +0200 |
| commit | ea71b4880cc6542ad13db75a0b9b9c85d640bad1 (patch) | |
| tree | 52fa80d1e6ac0d00ea074f15a9e8756ac3360675 /openbsc/tests/gbproxy | |
| parent | 51fde08b0759287b2d74ecf858925553ed915f2f (diff) | |
gbproxy/test: Fix IMSI length check (Coverity)
This fixes the IMSI length assertion, which currently uses a
greater-or-equal than zero comparison which always yields true. It is
replaced by a greater than zero check.
Fixes: Coverity CID 1239442
Sponsored-by: On-Waves ehf
Diffstat (limited to 'openbsc/tests/gbproxy')
| -rw-r--r-- | openbsc/tests/gbproxy/gbproxy_test.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/openbsc/tests/gbproxy/gbproxy_test.c b/openbsc/tests/gbproxy/gbproxy_test.c index df7b22ead..a99986bc4 100644 --- a/openbsc/tests/gbproxy/gbproxy_test.c +++ b/openbsc/tests/gbproxy/gbproxy_test.c @@ -3331,7 +3331,7 @@ static void test_gbproxy_keep_info() link_info = gbproxy_link_info_by_tlli(peer, foreign_tlli); OSMO_ASSERT(link_info); OSMO_ASSERT(link_info == link_info2); - OSMO_ASSERT(link_info->imsi_len >= 0); + OSMO_ASSERT(link_info->imsi_len > 0); OSMO_ASSERT(!link_info->is_deregistered); OSMO_ASSERT(link_info->imsi_acq_pending); |