| commit | 3c7f173dff889ba67cf200fe5a6f3b65c276b865 | |
| author | Jeremy Allison <jra@samba.org> | |
| Sat, 27 Sep 2008 01:33:07 +0000 (26 18:33 -0700) | ||
| committer | Karolin Seeger <kseeger@samba.org> | |
| Tue, 30 Sep 2008 07:31:47 +0000 (30 09:31 +0200) | ||
| tree | d4094d4e6ade301d6068430e181534b13a5ae0ac | treesnapshot (tar.gz zip) |
| parent | 79978a4098fda2bcbcdaffc82c035548917781c8 | commitdiff |
Second part of the fix for bug #5790 - samba returns STATUS_OBJECT_NAME_NOT_FOUND on set file disposition call.
This was my fault. I use a singleton cache (positive and negative) to speed up pathname based qfileinfo/setfileinfo lookups for alternate fsp's open on the same path. I only invalidated the negative cache on adding a new file fsp, as I incorrectly imagined the new fsp was put at the *end* of the open files list. DLIST_ADD puts it at the start, meaning any subsequent open wasn't seen once the cache was set. Doh !
Jeremy.
(cherry picked from commit 857433ec526caf73905dc9cb4fa25fb6355e5004)
This was my fault. I use a singleton cache (positive and negative) to speed up pathname based qfileinfo/setfileinfo lookups for alternate fsp's open on the same path. I only invalidated the negative cache on adding a new file fsp, as I incorrectly imagined the new fsp was put at the *end* of the open files list. DLIST_ADD puts it at the start, meaning any subsequent open wasn't seen once the cache was set. Doh !
Jeremy.
(cherry picked from commit 857433ec526caf73905dc9cb4fa25fb6355e5004)
| source/smbd/files.c | diffblobblamehistory |