diff options
author | Christopher R. Hertel <crh@samba.org> | 1998-06-23 08:15:05 +0000 |
---|---|---|
committer | Christopher R. Hertel <crh@samba.org> | 1998-06-23 08:15:05 +0000 |
commit | 7fda0a49cb665ba75c9071c1dc39fe93f103a9b5 (patch) | |
tree | be963416e054853a7020f2882e1395679fa465fd /source3/nmbd/nmbd_winsproxy.c | |
parent | 9d001d5c959d8fb5b4f14529e73ed2453035283b (diff) | |
download | samba-7fda0a49cb665ba75c9071c1dc39fe93f103a9b5.tar.gz samba-7fda0a49cb665ba75c9071c1dc39fe93f103a9b5.tar.bz2 samba-7fda0a49cb665ba75c9071c1dc39fe93f103a9b5.zip |
The function add_name_to_subnet(), in file nmbd_namelistdb.c, returns a
pointer to the newly constructed name list entry. In most cases, this
return value is ignored. The two exceptions are in asyncdns.c and
nmbd_winsproxy.c.
Most of the calls which ignored the return value were not cast to void, so
I added the cast. This helped me sort out which calls really did use the
return value. I also discovered one case, in nmbd_winsserver.c, in which
the return value was being stored to a variable which, in turn, was not
used.
Chris -)-----
(This used to be commit 384122d165ed6d5d211a29e5a63a63bf5cd82c75)
Diffstat (limited to 'source3/nmbd/nmbd_winsproxy.c')
-rw-r--r-- | source3/nmbd/nmbd_winsproxy.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/source3/nmbd/nmbd_winsproxy.c b/source3/nmbd/nmbd_winsproxy.c index 97caef7f82..5635124bcd 100644 --- a/source3/nmbd/nmbd_winsproxy.c +++ b/source3/nmbd/nmbd_winsproxy.c @@ -78,8 +78,9 @@ returned for name %s.\n", namestr(nmbname) )); if(rrec == PERMANENT_TTL) ttl = lp_max_ttl(); - namerec = add_name_to_subnet( orig_broadcast_subnet, nmbname->name, nmbname->name_type, - nb_flags, ttl, WINS_PROXY_NAME, num_ips, iplist); + namerec = add_name_to_subnet( orig_broadcast_subnet, nmbname->name, + nmbname->name_type, nb_flags, ttl, + WINS_PROXY_NAME, num_ips, iplist ); if(iplist != &ip) free((char *)iplist); |