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_winsserver.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_winsserver.c')
-rw-r--r-- | source3/nmbd/nmbd_winsserver.c | 16 |
1 files changed, 7 insertions, 9 deletions
diff --git a/source3/nmbd/nmbd_winsserver.c b/source3/nmbd/nmbd_winsserver.c index 304e9f4dba..e8dd78f1ab 100644 --- a/source3/nmbd/nmbd_winsserver.c +++ b/source3/nmbd/nmbd_winsserver.c @@ -268,16 +268,14 @@ BOOL initialise_wins(void) /* add all entries that have 60 seconds or more to live */ if ((ttl - 60) > time_now || ttl == PERMANENT_TTL) { - struct name_record *namerec; - if(ttl != PERMANENT_TTL) ttl -= time_now; - DEBUG(4, ("initialise_wins: add name: %s#%02x ttl = %d first IP %s flags = %2x\n", + DEBUG( 4, ("initialise_wins: add name: %s#%02x ttl = %d first IP %s flags = %2x\n", name, type, ttl, inet_ntoa(ip_list[0]), nb_flags)); - namerec = add_name_to_subnet(wins_server_subnet, name, type, nb_flags, - ttl, REGISTER_NAME, num_ips, ip_list); + (void)add_name_to_subnet( wins_server_subnet, name, type, nb_flags, + ttl, REGISTER_NAME, num_ips, ip_list ); } else @@ -834,8 +832,8 @@ is one of our (WINS server) names. Denying registration.\n", namestr(question) ) * Name did not exist - add it. */ - add_name_to_subnet(subrec, question->name, question->name_type, - nb_flags, ttl, REGISTER_NAME, 1, &from_ip); + (void)add_name_to_subnet( subrec, question->name, question->name_type, + nb_flags, ttl, REGISTER_NAME, 1, &from_ip ); send_wins_name_registration_response(0, ttl, p); } @@ -1123,8 +1121,8 @@ is one of our (WINS server) names. Denying registration.\n", namestr(question) ) * Name did not exist - add it. */ - add_name_to_subnet(subrec, question->name, question->name_type, - nb_flags, ttl, REGISTER_NAME, 1, &from_ip); + (void)add_name_to_subnet( subrec, question->name, question->name_type, + nb_flags, ttl, REGISTER_NAME, 1, &from_ip ); send_wins_name_registration_response(0, ttl, p); } |