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_become_lmb.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_become_lmb.c')
| -rw-r--r-- | source3/nmbd/nmbd_become_lmb.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/source3/nmbd/nmbd_become_lmb.c b/source3/nmbd/nmbd_become_lmb.c index b97da7d8b4..7329de6f69 100644 --- a/source3/nmbd/nmbd_become_lmb.c +++ b/source3/nmbd/nmbd_become_lmb.c @@ -43,8 +43,9 @@ void insert_permanent_name_into_unicast( struct subnet_record *subrec, if((namerec = find_name_on_subnet(unicast_subnet, nmbname, FIND_SELF_NAME)) == NULL) { /* The name needs to be created on the unicast subnet. */ - add_name_to_subnet( unicast_subnet, nmbname->name, nmbname->name_type, - nb_type, PERMANENT_TTL, PERMANENT_NAME, 1, &subrec->myip); + (void)add_name_to_subnet( unicast_subnet, nmbname->name, + nmbname->name_type, nb_type, + PERMANENT_TTL, PERMANENT_NAME, 1, &subrec->myip); } else { |