From be828fcec88a43306ddfb53473fda52df1d9965b Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Fri, 28 Apr 2000 00:39:23 +0000 Subject: Made changes suggested by Andrew review. fd_close now calls fd_close_posix() directly. set_posix_lock/release_posix_lock() now handle the reference counting. More changes due when this gets moved to the file locking/posix.c Jeremy. (This used to be commit 239abd48f049c6a8d2bbc0636eacf347ab77588c) --- source3/include/proto.h | 2 +- source3/locking/locking.c | 89 +++++++++++++++++++++++++++-------------------- source3/smbd/open.c | 16 +-------- 3 files changed, 54 insertions(+), 53 deletions(-) diff --git a/source3/include/proto.h b/source3/include/proto.h index 8c6fcd56b7..f9479532fa 100644 --- a/source3/include/proto.h +++ b/source3/include/proto.h @@ -831,7 +831,7 @@ int brl_forall(BRLOCK_FN(fn)); /*The following definitions come from locking/locking.c */ -BOOL fd_close_posix_locks(struct connection_struct *conn, files_struct *fsp); +int fd_close_posix(struct connection_struct *conn, files_struct *fsp); BOOL is_locked(files_struct *fsp,connection_struct *conn, SMB_BIG_UINT count,SMB_BIG_UINT offset, enum brl_type lock_type); diff --git a/source3/locking/locking.c b/source3/locking/locking.c index 088693b6d4..d69ab18006 100644 --- a/source3/locking/locking.c +++ b/source3/locking/locking.c @@ -102,10 +102,11 @@ static BOOL add_fd_to_close_entry(struct pending_closes *pc, int fd) Deal with pending closes needed by POSIX locking support. ****************************************************************************/ -BOOL fd_close_posix_locks(struct connection_struct *conn, files_struct *fsp) +int fd_close_posix(struct connection_struct *conn, files_struct *fsp) { struct pending_closes *pc; int saved_errno = 0; + int ret; size_t i; if (!lp_posix_locking(SNUM(conn))) @@ -116,9 +117,11 @@ BOOL fd_close_posix_locks(struct connection_struct *conn, files_struct *fsp) if (!pc) { /* * No other open with a POSIX lock on this dev/inode within this smbd. - * Just exit. + * Just close the fd. */ - return True; + ret = conn->vfs_ops.close(fsp->fd); + fsp->fd = -1; + return ret; } if (pc->num_posix_locks) { @@ -132,10 +135,10 @@ BOOL fd_close_posix_locks(struct connection_struct *conn, files_struct *fsp) return False; fsp->fd = -1; - return True; + return 0; } - DEBUG(10,("fd_close_posix_locks: doing close on %u fd's.\n", (unsigned int)pc->fd_array_size )); + DEBUG(10,("fd_close_posix: doing close on %u fd's.\n", (unsigned int)pc->fd_array_size )); /* * This is the last close. If there are pending fd's close them @@ -157,11 +160,16 @@ BOOL fd_close_posix_locks(struct connection_struct *conn, files_struct *fsp) free((char *)pc); + ret = conn->vfs_ops.close(fsp->fd); + if (saved_errno != 0) { errno = saved_errno; + ret = -1; } - return True; + fsp->fd = -1; + + return ret; } /**************************************************************************** @@ -497,8 +505,10 @@ static BOOL set_posix_lock(files_struct *fsp, SMB_BIG_UINT u_offset, SMB_BIG_UIN * pretend it was successful. */ - if(!posix_lock_in_range(&offset, &count, u_offset, u_count)) + if(!posix_lock_in_range(&offset, &count, u_offset, u_count)) { + increment_posix_lock_list(fsp); return True; + } /* * Note that setting multiple overlapping read locks on different @@ -510,42 +520,55 @@ static BOOL set_posix_lock(files_struct *fsp, SMB_BIG_UINT u_offset, SMB_BIG_UIN ret = fcntl_lock(fsp->fd,SMB_F_SETLK,offset,count,map_posix_lock_type(fsp,lock_type)); + if (ret) + increment_posix_lock_list(fsp); + return ret; } /**************************************************************************** - POSIX function to release a lock. Returns True if the + POSIX function to release a lock given a list. Returns True if the lock could be released, False if not. ****************************************************************************/ -static BOOL release_posix_lock(files_struct *fsp, SMB_BIG_UINT u_offset, SMB_BIG_UINT u_count) +static BOOL release_posix_lock(files_struct *fsp, struct unlock_list *ulist) { - SMB_OFF_T offset; - SMB_OFF_T count; BOOL ret = True; - DEBUG(5,("release_posix_lock: File %s, offset = %.0f, count = %.0f\n", - fsp->fsp_name, (double)u_offset, (double)u_count )); + for(; ulist; ulist = ulist->next) { + SMB_OFF_T offset = ulist->start; + SMB_OFF_T count = ulist->size; + + DEBUG(5,("release_posix_lock: File %s, offset = %.0f, count = %.0f\n", + fsp->fsp_name, (double)offset, (double)count )); + + if(count == 0) { - if(u_count == 0) { + /* + * This lock must overlap with an existing read-only lock + * held by another fd. Don't do any POSIX call. + */ + + continue; + } /* - * This lock must overlap with an existing read-only lock - * help by another fd. Don't do any POSIX call. + * If the requested lock won't fit in the POSIX range, we will + * pretend it was successful. */ - return True; + if(!posix_lock_in_range(&offset, &count, offset, count)) + continue; + + ret = fcntl_lock(fsp->fd,SMB_F_SETLK,offset,count,F_UNLCK); } /* - * If the requested lock won't fit in the POSIX range, we will - * pretend it was successful. - */ - - if(!posix_lock_in_range(&offset, &count, u_offset, u_count)) - return True; + * We treat this as one unlock request for POSIX accounting purposes even + * if it may have been split into multiple smaller POSIX unlock ranges. + */ - ret = fcntl_lock(fsp->fd,SMB_F_SETLK,offset,count,F_UNLCK); + decrement_posix_lock_list(fsp); return ret; } @@ -610,7 +633,7 @@ BOOL do_lock(files_struct *fsp,connection_struct *conn, offset, count, lock_type); - if(ok && lp_posix_locking(SNUM(conn))) { + if (ok && lp_posix_locking(SNUM(conn))) { /* * Try and get a POSIX lock on this range. @@ -618,9 +641,9 @@ BOOL do_lock(files_struct *fsp,connection_struct *conn, * overlapping on a different fd. JRA. */ - if((ok = set_posix_lock(fsp, offset, count, lock_type)) == True) - increment_posix_lock_list(fsp); - else { + ok = set_posix_lock(fsp, offset, count, lock_type); + + if (!ok) { /* * We failed to map - we must now remove the brl * lock entry. @@ -725,18 +748,10 @@ BOOL do_unlock(files_struct *fsp,connection_struct *conn, * Release the POSIX locks on the list of ranges returned. */ - for(; ulist; ulist = ulist->next) - (void)release_posix_lock(fsp, ulist->start, ulist->size); + (void)release_posix_lock(fsp, ulist); talloc_destroy(ul_ctx); - /* - * We treat this as one unlock request for POSIX accounting purposes even - * if it may have been split into multiple smaller POSIX unlock ranges. - */ - - decrement_posix_lock_list(fsp); - return True; /* Did unlock */ } diff --git a/source3/smbd/open.c b/source3/smbd/open.c index 737b9b5ef3..9aadb39bc9 100644 --- a/source3/smbd/open.c +++ b/source3/smbd/open.c @@ -55,21 +55,7 @@ static int fd_open(struct connection_struct *conn, char *fname, int fd_close(struct connection_struct *conn, files_struct *fsp) { - int ret = 0; - - /* - * Deal with transferring any pending fd's if there - * are POSIX locks outstanding. - */ - - if(!fd_close_posix_locks(conn,fsp)) - return -1; - - if(fsp->fd != -1) - ret = conn->vfs_ops.close(fsp->fd); - - fsp->fd = -1; - return ret; + return fd_close_posix(conn, fsp); } -- cgit