diff options
author | Steven Danneman <steven.danneman@isilon.com> | 2009-02-02 21:37:51 -0800 |
---|---|---|
committer | Steven Danneman <steven.danneman@isilon.com> | 2009-02-09 23:56:17 -0800 |
commit | 6272f4c2f453c509b8a3893d4c2ac5fc356b348d (patch) | |
tree | 82dce05824a6106a371febc4ec9ca6b2864cbcde /source3/smbd | |
parent | 25d345eb39c69b2b42a966846ae893b068de40a4 (diff) | |
download | samba-6272f4c2f453c509b8a3893d4c2ac5fc356b348d.tar.gz samba-6272f4c2f453c509b8a3893d4c2ac5fc356b348d.tar.bz2 samba-6272f4c2f453c509b8a3893d4c2ac5fc356b348d.zip |
s3: Added SMB_VFS_INIT_SEARCH_OP to initialize data at the beginning of SMB search requests.
By default this VFS call is a NOOP, but the onefs vfs module takes advantage
of it to initialize direntry search caches at the beginning of each
TRANS2_FIND_FIRST, TRANS2_FIND_NEXT, SMBffirst, SMBsearch, and SMBunique
Diffstat (limited to 'source3/smbd')
-rw-r--r-- | source3/smbd/dir.c | 8 | ||||
-rw-r--r-- | source3/smbd/reply.c | 3 | ||||
-rw-r--r-- | source3/smbd/trans2.c | 12 |
3 files changed, 20 insertions, 3 deletions
diff --git a/source3/smbd/dir.c b/source3/smbd/dir.c index c4f6b4fe55..6618bf07b0 100644 --- a/source3/smbd/dir.c +++ b/source3/smbd/dir.c @@ -681,6 +681,14 @@ void dptr_DirCacheAdd(struct dptr_struct *dptr, const char *name, long offset) } /**************************************************************************** + Initialize variables & state data at the beginning of all search SMB requests. +****************************************************************************/ +void dptr_init_search_op(struct dptr_struct *dptr) +{ + return SMB_VFS_INIT_SEARCH_OP(dptr->conn, dptr->dir_hnd->dir); +} + +/**************************************************************************** Fill the 5 byte server reserved dptr field. ****************************************************************************/ diff --git a/source3/smbd/reply.c b/source3/smbd/reply.c index 628f481c1c..256160c402 100644 --- a/source3/smbd/reply.c +++ b/source3/smbd/reply.c @@ -1380,6 +1380,9 @@ void reply_search(struct smb_request *req) DEBUG(4,("dptr_num is %d\n",dptr_num)); + /* Initialize per SMBsearch/SMBffirst/SMBfunique operation data */ + dptr_init_search_op(conn->dirptr); + if ((dirtype&0x1F) == aVOLID) { char buf[DIR_STRUCT_SIZE]; memcpy(buf,status,21); diff --git a/source3/smbd/trans2.c b/source3/smbd/trans2.c index 6a6e59a581..cea066e97f 100644 --- a/source3/smbd/trans2.c +++ b/source3/smbd/trans2.c @@ -1914,7 +1914,7 @@ close_if_end = %d requires_resume_key = %d level = 0x%x, max_data_bytes = %d\n", /* W2K3 seems to treat zero as 1. */ maxentries = 1; } - + switch (info_level) { case SMB_FIND_INFO_STANDARD: case SMB_FIND_EA_SIZE: @@ -2043,7 +2043,7 @@ total_data=%u (should be %u)\n", (unsigned int)total_data, (unsigned int)IVAL(pd } params = *pparams; - /* Save the wildcard match and attribs we are using on this directory - + /* Save the wildcard match and attribs we are using on this directory - needed as lanman2 assumes these are being saved between calls */ ntstatus = dptr_create(conn, @@ -2064,6 +2064,9 @@ total_data=%u (should be %u)\n", (unsigned int)total_data, (unsigned int)IVAL(pd dptr_num = dptr_dnum(conn->dirptr); DEBUG(4,("dptr_num is %d, wcard = %s, attr = %d\n", dptr_num, mask, dirtype)); + /* Initialize per TRANS2_FIND_FIRST operation data */ + dptr_init_search_op(conn->dirptr); + /* We don't need to check for VOL here as this is returned by a different TRANS2 call. */ @@ -2078,7 +2081,7 @@ total_data=%u (should be %u)\n", (unsigned int)total_data, (unsigned int)IVAL(pd for (i=0;(i<maxentries) && !finished && !out_of_space;i++) { bool got_exact_match = False; - /* this is a heuristic to avoid seeking the dirptr except when + /* this is a heuristic to avoid seeking the dirptr except when absolutely necessary. It allows for a filename of about 40 chars */ if (space_remaining < DIRLEN_GUESS && numentries > 0) { out_of_space = True; @@ -2374,6 +2377,9 @@ total_data=%u (should be %u)\n", (unsigned int)total_data, (unsigned int)IVAL(pd (long)conn->dirptr, dptr_TellDir(conn->dirptr))); + /* Initialize per TRANS2_FIND_NEXT operation data */ + dptr_init_search_op(conn->dirptr); + /* We don't need to check for VOL here as this is returned by a different TRANS2 call. */ |