diff options
author | Corinna Vinschen <corinna@vinschen.de> | 2012-02-20 18:08:37 +0400 |
---|---|---|
committer | Corinna Vinschen <corinna@vinschen.de> | 2012-02-20 18:08:37 +0400 |
commit | 3c53eaeb5b32bfcb3e5a650f9ef9fa5c9e176b1e (patch) | |
tree | 1f6a9a0277e9d9177d7c2461ce3660099643e758 /winsup/cygwin/flock.cc | |
parent | 2196614e3df4433c43c44eb44b75c65af021089d (diff) |
* flock.cc (inode_t::del_my_locks): Drop useless counter lc. Close
lock object handle explicitely in case of called during fork. Add
comment to explain why.
* fork.cc (frok::child): Drop declaration of fixup_lockf_after_fork.
Diffstat (limited to 'winsup/cygwin/flock.cc')
-rw-r--r-- | winsup/cygwin/flock.cc | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/winsup/cygwin/flock.cc b/winsup/cygwin/flock.cc index 9b1de01a9..16233449e 100644 --- a/winsup/cygwin/flock.cc +++ b/winsup/cygwin/flock.cc @@ -1,6 +1,6 @@ /* flock.cc. NT specific implementation of advisory file locking. - Copyright 2003, 2008, 2009, 2010, 2011 Red Hat, Inc. + Copyright 2003, 2008, 2009, 2010, 2011, 2012 Red Hat, Inc. This file is part of Cygwin. @@ -347,14 +347,18 @@ inode_t::del_my_locks (long long id, HANDLE fhdl) { lockf_t *lock, *n_lock; lockf_t **prev = &i_lockf; - int lc = 0; for (lock = *prev; lock && (n_lock = lock->lf_next, 1); lock = n_lock) { if (lock->lf_flags & F_POSIX) { /* Delete all POSIX locks. */ *prev = n_lock; - ++lc; + /* When called during fork, the POSIX lock must get deleted but + *not* signalled. The lock is still active and locked in the + parent. So in case of fork, we call close_lock_obj explicitely, + since del_lock_obj is called from the destructor. */ + if (!id) + lock->close_lock_obj (); delete lock; } else if (id && lock->lf_id == id) |