summaryrefslogtreecommitdiff
path: root/source3/smbd
diff options
context:
space:
mode:
authorSimo Sorce <idra@samba.org>2001-09-15 13:21:54 +0000
committerSimo Sorce <idra@samba.org>2001-09-15 13:21:54 +0000
commit72c35310b2b0d39c077c50a2175c76ced5d4c662 (patch)
treea3cba0379c82df419a10e98c06f6517bc7097369 /source3/smbd
parent7892c494e7321c64b20bf7e1d794a6b6508fe84a (diff)
downloadsamba-72c35310b2b0d39c077c50a2175c76ced5d4c662.tar.gz
samba-72c35310b2b0d39c077c50a2175c76ced5d4c662.tar.bz2
samba-72c35310b2b0d39c077c50a2175c76ced5d4c662.zip
use variables _before_ freeing them!
thanks to Andreas Moroder for spotting these two. (This used to be commit 01f30ca0ad472eaab8f80ec4278b1d391be3a1ae)
Diffstat (limited to 'source3/smbd')
-rw-r--r--source3/smbd/open.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/source3/smbd/open.c b/source3/smbd/open.c
index 145d12e3f3..7974ea0ae0 100644
--- a/source3/smbd/open.c
+++ b/source3/smbd/open.c
@@ -497,9 +497,9 @@ dev = %x, inode = %.0f\n", *p_oplock_request, share_entry->op_type, fname, (unsi
lock_share_entry(conn, dev, inode);
if(opb_ret == False) {
- free((char *)old_shares);
DEBUG(0,("open_mode_check: FAILED when breaking oplock (%x) on file %s, \
dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (double)inode));
+ free((char *)old_shares);
errno = EACCES;
unix_ERR_class = ERRDOS;
unix_ERR_code = ERRbadshare;
@@ -1234,11 +1234,10 @@ dev = %x, inode = %.0f\n", share_entry->op_type, fname, (unsigned int)dev, (doub
unlock_share_entry(conn, dev, inode);
if(request_oplock_break(share_entry, dev, inode) == False)
{
- free((char *)old_shares);
-
DEBUG(0,("check_file_sharing: FAILED when breaking oplock (%x) on file %s, \
dev = %x, inode = %.0f\n", old_shares[i].op_type, fname, (unsigned int)dev, (double)inode));
+ free((char *)old_shares);
return False;
}
lock_share_entry(conn, dev, inode);