summaryrefslogtreecommitdiff
path: root/source3/namedbname.c
diff options
context:
space:
mode:
authorAndrew Tridgell <tridge@samba.org>1996-10-05 10:41:13 +0000
committerAndrew Tridgell <tridge@samba.org>1996-10-05 10:41:13 +0000
commite5893bdfbef0ac16772199d7ec6fac7d3e4f8431 (patch)
treec531a4507eead09e646760bfc7a7412bcca86522 /source3/namedbname.c
parentc33d98d5731cf7901c11786b9cbfe25ac59e0b83 (diff)
downloadsamba-e5893bdfbef0ac16772199d7ec6fac7d3e4f8431.tar.gz
samba-e5893bdfbef0ac16772199d7ec6fac7d3e4f8431.tar.bz2
samba-e5893bdfbef0ac16772199d7ec6fac7d3e4f8431.zip
I have fixed quite a few important bugs in this commit.
Luke, can you take special note of the bug fixes to nmbd so you can propogate them to your new code. - rewrote the code that used to use fromhost(). We now call gethostbyaddr() only if necessary and a maximum of once per connection. Calling gethostbyaddr() causes problems on some systems so avoiding it if possible is a good thing :-) - added the "fake oplocks" option. See the docs in smb.conf(5) and Speed.txt - fixed a serious bug in nmbd where it would try a DNS lookup on FIND_SELF queries. This caused a lot of unnecessary (and incorrect) DNS lookups to happen. FIND_SELF queries should only go to the internal name tables. - don't set FIND_SELF for name queries if we are a wins proxy, as we are supposed to be answering queries for other hosts. - fixed a bug in nmbd which had "if (search | FIND_LOCAL)" instead of "if (search & FIND_LOCAL)". Luke, this was in nameservreply.c - the above 3 bugs together meant that DNS queries were being cached, but the cache wasn't being used, so every query was going to DNS, no wonder nmbd has been chewing so much CPU time! Another side effect was that queries on names in lmhosts weren't being answered for bcast queries with "wins proxy" set. - ignore the maxxmit for seconday session setups (see CIFS spec) - close user opened files in a uLogoffX for user level security (see CIFS spec) - added uid into the files struct to support the above change (This used to be commit ea472b7217b7693627a13a7b1e428a0a6a3d8755)
Diffstat (limited to 'source3/namedbname.c')
-rw-r--r--source3/namedbname.c10
1 files changed, 8 insertions, 2 deletions
diff --git a/source3/namedbname.c b/source3/namedbname.c
index 1f16553b0f..833a870c6c 100644
--- a/source3/namedbname.c
+++ b/source3/namedbname.c
@@ -150,7 +150,8 @@ struct name_record *find_name(struct name_record *n,
if (name_equal(&ret->name,name))
{
/* self search: self names only */
- if ((search&FIND_SELF) == FIND_SELF && ret->source != SELF)
+ if ((search&FIND_SELF) == FIND_SELF &&
+ ret->source != SELF)
continue;
return ret;
@@ -528,13 +529,18 @@ struct name_record *search_for_name(struct subnet_record **d,
if (*d == NULL) return NULL;
+ if (!n && (search & FIND_SELF)) {
+ DEBUG(3,("FIND_SELF set - failing lookup\n"));
+ return NULL;
+ }
+
DEBUG(4,("subnet %s ", inet_ntoa((*d)->bcast_ip)));
/* now try DNS lookup. */
if (!n)
{
struct in_addr dns_ip;
- uint32 a;
+ uint32 a;
/* only do DNS lookups if the query is for type 0x20 or type 0x0 */
if (!dns_type && name_type != 0x1b)