diff options
| author | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-01-19 08:57:07 +0100 |
|---|---|---|
| committer | Jacob Erlbeck <jerlbeck@sysmocom.de> | 2015-01-19 15:10:57 +0100 |
| commit | 3793b21a42a3e33c0356a59d3153988bc40c7820 (patch) | |
| tree | 710143e6d88f58c2711781f9c83e588e80d919c3 /hlrsync/hlrsync.py | |
| parent | a1f7a53a95483560f91fe601ed837581e25ebda1 (diff) | |
sgsn/test: Make assert_substr safer (Coverity)
Currently, if assert_subscr were called with subscr == NULL, the
later call to subscr_put might fail, as Coverity has complained. In
addition, the call to subscr_put would free the subscr object if it
were in the cache with a refcount of 0 at the time assert_substr was
called.
This patch adds a check for the subscr being non-NULL and reorders
the checks, so that the subscr_put comes last.
Fixes: Coverity CID 1264590
Sponsored-by: On-Waves ehf
Diffstat (limited to 'hlrsync/hlrsync.py')
0 files changed, 0 insertions, 0 deletions