diff options
| author | Derrell Lipman <derrell@dworkin.(none)> | 2009-04-27 09:27:52 -0400 |
|---|---|---|
| committer | Derrell Lipman <derrell@dworkin.(none)> | 2009-04-27 09:34:25 -0400 |
| commit | 641e12561ad863f8bff8c3a98485a010c138ec30 (patch) | |
| tree | a7e37fb05fdc4d8b0c3a08c865e11773c0435427 | |
| parent | 65527216f1289f366d83beb936a668b32cf44d38 (diff) | |
| download | samba-641e12561ad863f8bff8c3a98485a010c138ec30.tar.gz samba-641e12561ad863f8bff8c3a98485a010c138ec30.tar.bz2 samba-641e12561ad863f8bff8c3a98485a010c138ec30.zip | |
[FIX Bug 6235] domain enumeration breaks if master browser has space in name
Jeremy: please review to ensure this doesn't appear to break anything
The function name_status_find() is documented as used for finding a server's
name given its IP address. It was, however, looking for the first matching
name which could be a group name at times. This fix ensures that group names
are skipped when scanning for a matching name.
Derrell
| -rw-r--r-- | source3/libsmb/namequery.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/source3/libsmb/namequery.c b/source3/libsmb/namequery.c index 19d89ce363..50fb9f1620 100644 --- a/source3/libsmb/namequery.c +++ b/source3/libsmb/namequery.c @@ -408,7 +408,8 @@ bool name_status_find(const char *q_name, goto done; for (i=0;i<count;i++) { - if (status[i].type == type) + /* Find first one of the requested type that's not a GROUP. */ + if (status[i].type == type && ! (status[i].flags & 0x80)) break; } if (i == count) |