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_namelistdb.c | |
parent | 9d001d5c959d8fb5b4f14529e73ed2453035283b (diff) | |
download | samba-7fda0a49cb665ba75c9071c1dc39fe93f103a9b5.tar.gz samba-7fda0a49cb665ba75c9071c1dc39fe93f103a9b5.tar.xz 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_namelistdb.c')
-rw-r--r-- | source3/nmbd/nmbd_namelistdb.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/source3/nmbd/nmbd_namelistdb.c b/source3/nmbd/nmbd_namelistdb.c index 4454cdfed06..d2c9ea2e71a 100644 --- a/source3/nmbd/nmbd_namelistdb.c +++ b/source3/nmbd/nmbd_namelistdb.c @@ -280,8 +280,8 @@ void standard_success_register(struct subnet_record *subrec, namerec = find_name_on_subnet( subrec, nmbname, FIND_SELF_NAME ); if( NULL == namerec ) - add_name_to_subnet( subrec, nmbname->name, nmbname->name_type, - nb_flags, ttl, SELF_NAME, 1, ®istered_ip ); + (void)add_name_to_subnet( subrec, nmbname->name, nmbname->name_type, + nb_flags, ttl, SELF_NAME, 1, ®istered_ip ); else update_name_ttl( namerec, ttl ); } |