s3: smbd: Cleanup - fix the comment for dptr_SearchDir().

It doesn't and never has skipped VETO or unreadable files.

It's only used in call_trans2findnext() in the SMB1 code
given a name already returned to the client to find a
resume position. Even if the client gave us a name that
it had never been given to it (client bug or malicious
client) we'd just start the search from the wrong position,
which doesn't cause problems (we still check for VETO
or unreadable before returning any names).

Worst thing that actually happens is the client messes
up their own search, which is what you'd expect from
giving an incorrect resume name.

Signed-off-by: Jeremy Allison <jra@samba.org>
Reviewed-by: Ralph Boehme <slow@samba.org>

diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c
index 9d5244e3c4e51a707adac0c54efd86839e36b052..946d86de3fde0574fcc78bab660171776cfa30c0 100644 (file)
--- a/source3/smbd/dir.c
+++ b/source3/smbd/dir.c
@@ -559,7 +559,7 @@ ret:
 }
 
 /****************************************************************************
- Search for a file by name, skipping veto'ed and not visible files.
+ Search for a file by name.
 ****************************************************************************/
 
 bool dptr_SearchDir(struct dptr_struct *dptr, const char *name, long *poffset, SMB_STRUCT_STAT *pst)