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

gb_proxy_peer: sgsn can never be NULL in gbproxy_sgsn_by_nri() 

Redundant check makes Coverity think that sgsn can be NULL...

Change-Id: Iffdeb18e0a5575a2a09129eed9b8a41c3340dcbf
Fixes: CID#215870
This commit is contained in:
Vadim Yanitskiy 2021-01-05 14:36:52 +01:00
parent 3844da98f8
commit 1c33e4af05
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/gbproxy/gb_proxy_peer.c
View File

@ -482,7 +482,7 @@ struct gbproxy_sgsn *gbproxy_sgsn_by_nri(struct gbproxy_config *cfg, uint16_t nr
llist_for_each_entry(sgsn, &cfg->sgsns, list) {
if (osmo_nri_v_matches_ranges(nri, sgsn->pool.nri_ranges)) {
/* Also check if the NRI we're looking for is a NULL NRI */
if (sgsn && null_nri) {
if (null_nri) {
if (osmo_nri_v_matches_ranges(nri, cfg->pool.null_nri_ranges))
*null_nri = true;
else