| commit | 32cbf20e65578b3d5e71d571d81d561f33a887e2 | |
| author | Jeremy Allison <jra@samba.org> | |
| Sat, 19 May 2012 02:29:36 +0000 (18 19:29 -0700) | ||
| committer | Jeremy Allison <jra@samba.org> | |
| Sat, 19 May 2012 04:26:33 +0000 (19 06:26 +0200) | ||
| tree | 0753a8e2b257a085a7ce91137c36b39c43be1f8d | treesnapshot (tar.gz zip) |
| parent | 87a62df8e69bfd45c944f44fb7d03da244dc9a44 | commitdiff |
Move the set_write_time() call to after get_existing_share_mode_lock() returns with a share mode.
get_existing_share_mode_lock() isn't really the right
call here, as we're being called after
close_remove_share_mode() inside close_normal_file()
so it's quite normal to not have an existing share
mode here. However, get_share_mode_lock() doesn't
work because that will create a new share mode if
one doesn't exist - so stick with this call (just
ignore any error we get if the share mode doesn't
exist.
The previous commit raised the error message debug
level inside get_share_mode_lock_internal() so
we don't always get a level 1 error message if
get_existing_share_mode_lock() fails.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat May 19 06:26:33 CEST 2012 on sn-devel-104
get_existing_share_mode_lock() isn't really the right
call here, as we're being called after
close_remove_share_mode() inside close_normal_file()
so it's quite normal to not have an existing share
mode here. However, get_share_mode_lock() doesn't
work because that will create a new share mode if
one doesn't exist - so stick with this call (just
ignore any error we get if the share mode doesn't
exist.
The previous commit raised the error message debug
level inside get_share_mode_lock_internal() so
we don't always get a level 1 error message if
get_existing_share_mode_lock() fails.
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Sat May 19 06:26:33 CEST 2012 on sn-devel-104
| source3/smbd/close.c | diffblobblamehistory |