diff options
| author | Jeremy Allison <jra@samba.org> | 1997-10-03 23:45:34 +0000 |
|---|---|---|
| committer | Jeremy Allison <jra@samba.org> | 1997-10-03 23:45:34 +0000 |
| commit | 21ecb46f56ac0f341803d0552c981e517d0322cb (patch) | |
| tree | 8197a62cea1e82f7766aba08d2209707d180d474 | |
| parent | 97ee4a5f69bd9cfbbc8710a1a04d80db0ee40104 (diff) | |
| download | samba-21ecb46f56ac0f341803d0552c981e517d0322cb.tar.gz | |
Making it harder for share_mode_entries to be left with
non-zero op_type fields. Paranoia code really.
Jeremy (jallison@whistle.com)
| -rw-r--r-- | source/locking/locking.c | 7 |
1 files changed, 2 insertions, 5 deletions
diff --git a/source/locking/locking.c b/source/locking/locking.c index 834f3e56589..abda5d39d47 100644 --- a/source/locking/locking.c +++ b/source/locking/locking.c @@ -668,8 +668,7 @@ record due to old locking version %d for file dev %d, inode %d hash bucket %d\n" while(entry_scanner_p) { if( (pid == entry_scanner_p->pid) && - (entry_scanner_p->op_port != 0) && - (entry_scanner_p->op_type != 0) && + (entry_scanner_p->share_mode == Files[fnum].share_mode) && (memcmp(&entry_scanner_p->time, &Files[fnum].open_time,sizeof(struct timeval)) == 0) ) { @@ -1463,9 +1462,7 @@ for share file %d\n", num_entries, fname)); if((IVAL(p,SME_SEC_OFFSET) != fs_p->open_time.tv_sec) || (IVAL(p,SME_USEC_OFFSET) != fs_p->open_time.tv_usec) || (IVAL(p,SME_SHAREMODE_OFFSET) != fs_p->share_mode) || - (IVAL(p,SME_PID_OFFSET) != pid) || - (SVAL(p,SME_PORT_OFFSET) == 0) || - (SVAL(p,SME_OPLOCK_TYPE_OFFSET) == 0)) + (IVAL(p,SME_PID_OFFSET) != pid)) continue; DEBUG(5,("remove_share_oplock: clearing oplock on entry number %d (of %d) \ |