diff options
| author | Jeremy Allison <jra@samba.org> | 2007-04-30 01:34:28 +0000 |
|---|---|---|
| committer | Gerald (Jerry) Carter <jerry@samba.org> | 2007-10-10 12:19:48 -0500 |
| commit | 8ad13718af0ba1fcb10a6f1631b1ed3cb8d11175 (patch) | |
| tree | 9436cacf45161fb2b80b2d088d17f921aae62a03 /source/locking/brlock.c | |
| parent | c3df5d08dd6a983f9d53dc6628a50e571d322e8d (diff) | |
| download | samba-8ad13718af0ba1fcb10a6f1631b1ed3cb8d11175.tar.gz samba-8ad13718af0ba1fcb10a6f1631b1ed3cb8d11175.tar.xz samba-8ad13718af0ba1fcb10a6f1631b1ed3cb8d11175.zip | |
r22588: Make all uses of TALLOC_MEMDUP consistent.
Jeremy.
Diffstat (limited to 'source/locking/brlock.c')
| -rw-r--r-- | source/locking/brlock.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/source/locking/brlock.c b/source/locking/brlock.c index 899e211ffcc..6dd3ac6ea17 100644 --- a/source/locking/brlock.c +++ b/source/locking/brlock.c @@ -1283,10 +1283,15 @@ void brl_close_fnum(struct byte_range_lock *br_lck) unsigned int num_locks_copy; /* Copy the current lock array. */ - locks_copy = (struct lock_struct *)TALLOC_MEMDUP(br_lck, locks, br_lck->num_locks * sizeof(struct lock_struct)); - if (!locks_copy) { - smb_panic("brl_close_fnum: talloc fail.\n"); + if (br_lck->num_locks) { + locks_copy = (struct lock_struct *)TALLOC_MEMDUP(br_lck, locks, br_lck->num_locks * sizeof(struct lock_struct)); + if (!locks_copy) { + smb_panic("brl_close_fnum: talloc fail.\n"); + } + } else { + locks_copy = NULL; } + num_locks_copy = br_lck->num_locks; for (i=0; i < num_locks_copy; i++) { |