diff options
| author | Jeremy Allison <jra@samba.org> | 2006-06-14 00:37:52 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 11:17:26 -0500 |
| commit | a5da0a72e1f1e026690df4b149a63754f29e6c28 (patch) | |
| tree | 9a9de0676b04550c887dca655bdc087427f84834 /source3/nmbd/nmbd_become_dmb.c | |
| parent | d4a80fdf38428c7a0e6cda6da838c8ccc5d8f2ab (diff) | |
| download | samba-a5da0a72e1f1e026690df4b149a63754f29e6c28.tar.gz samba-a5da0a72e1f1e026690df4b149a63754f29e6c28.tar.bz2 samba-a5da0a72e1f1e026690df4b149a63754f29e6c28.zip | |
r16213: Fix Klocwork #852. iface_n_ip can potentially
return NULL. Ensure we don't deref.
Jeremy.
(This used to be commit c2f0ea2ff2f6ad925cee9c85110c6ad828ffb7a9)
Diffstat (limited to 'source3/nmbd/nmbd_become_dmb.c')
| -rw-r--r-- | source3/nmbd/nmbd_become_dmb.c | 10 |
1 files changed, 9 insertions, 1 deletions
diff --git a/source3/nmbd/nmbd_become_dmb.c b/source3/nmbd/nmbd_become_dmb.c index fb1fb33a81..523ec6cabf 100644 --- a/source3/nmbd/nmbd_become_dmb.c +++ b/source3/nmbd/nmbd_become_dmb.c @@ -120,6 +120,7 @@ in workgroup %s on subnet %s\n", if( subrec == unicast_subnet ) { struct nmb_name nmbname; struct in_addr my_first_ip; + struct in_addr *nip; /* Put our name and first IP address into the workgroup struct as domain master browser. This @@ -130,7 +131,14 @@ in workgroup %s on subnet %s\n", work->dmb_name = nmbname; /* Pick the first interface ip address as the domain master browser ip. */ - my_first_ip = *iface_n_ip(0); + nip = iface_n_ip(0); + + if (!nip) { + DEBUG(0,("become_domain_master_stage2: Error. iface_n_ip returned NULL\n")); + return; + } + + my_first_ip = *nip; putip((char *)&work->dmb_addr, &my_first_ip); |