diff options
author | Jeremy Allison <jra@samba.org> | 2006-06-28 00:05:53 +0000 |
---|---|---|
committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:19:01 -0500 |
commit | f6d5bae4a105eee1b1d5b1aaa70a675705345d9e (patch) | |
tree | 7e4c1e5095e7d6a1a2208cdfe24a1b8f711a7b39 /source/nmbd | |
parent | 9cbfaf62a3c4bb7d2e594e412449506ab0af4063 (diff) | |
download | samba-f6d5bae4a105eee1b1d5b1aaa70a675705345d9e.tar.gz samba-f6d5bae4a105eee1b1d5b1aaa70a675705345d9e.tar.xz samba-f6d5bae4a105eee1b1d5b1aaa70a675705345d9e.zip |
r16579: Fix Klocwork #2016. Possible null deref.
Jeremy.
Diffstat (limited to 'source/nmbd')
-rw-r--r-- | source/nmbd/nmbd_serverlistdb.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/source/nmbd/nmbd_serverlistdb.c b/source/nmbd/nmbd_serverlistdb.c index cc762ae8d23..ea27f9d4e5a 100644 --- a/source/nmbd/nmbd_serverlistdb.c +++ b/source/nmbd/nmbd_serverlistdb.c @@ -214,7 +214,7 @@ static uint32 write_this_server_name( struct subnet_record *subrec, struct work_record *iwork; /* Go through all the subnets we have already seen. */ - for (ssub = FIRST_SUBNET; ssub != subrec; ssub = NEXT_SUBNET_INCLUDING_UNICAST(ssub)) { + for (ssub = FIRST_SUBNET; ssub && (ssub != subrec); ssub = NEXT_SUBNET_INCLUDING_UNICAST(ssub)) { for(iwork = ssub->workgrouplist; iwork; iwork = iwork->next) { if(find_server_in_workgroup( iwork, servrec->serv.name) != NULL) { /* |