summaryrefslogtreecommitdiff
path: root/source3/rpc_server
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2001-07-02 13:43:46 +0000
committerAndrew Bartlett <abartlet@samba.org>2001-07-02 13:43:46 +0000
commite682d6d54e3f23266e3a75d1461e8dff2d729006 (patch)
treec0f641ca85b81911a371b3a3401cc8ff521ed214 /source3/rpc_server
parentb07605976a00bf542ba9f0ed8357ced0f05acc3a (diff)
downloadsamba-e682d6d54e3f23266e3a75d1461e8dff2d729006.tar.gz
samba-e682d6d54e3f23266e3a75d1461e8dff2d729006.tar.bz2
samba-e682d6d54e3f23266e3a75d1461e8dff2d729006.zip
This fix from Eelco Vriezekolk <eelco@nexus.com.na> is for a SIG11 bug where we
would always file_free(fsp) twice, once in close_file and once afterwoulds. The bug was reported in SAMBA_2_2, but a code inspection shows it to be in HEAD as well. (Unfortunetly I don't have the facilites to actualy check this, but the change is quite simple, makes sence and compiles). Andrew Bartlett (This used to be commit 5d3f622375842961a79536d36ee6f6ad508b91f1)
Diffstat (limited to 'source3/rpc_server')
-rw-r--r--source3/rpc_server/srv_srvsvc_nt.c2
1 files changed, 0 insertions, 2 deletions
diff --git a/source3/rpc_server/srv_srvsvc_nt.c b/source3/rpc_server/srv_srvsvc_nt.c
index 39e482dc3c..44821d1432 100644
--- a/source3/rpc_server/srv_srvsvc_nt.c
+++ b/source3/rpc_server/srv_srvsvc_nt.c
@@ -1662,7 +1662,6 @@ uint32 _srv_net_file_query_secdesc(pipes_struct *p, SRV_Q_NET_FILE_QUERY_SECDESC
psd->dacl->revision = (uint16) NT4_ACL_REVISION;
close_file(fsp, True);
- file_free(fsp);
close_cnum(conn, user.vuid);
return r_u->status;
@@ -1671,7 +1670,6 @@ uint32 _srv_net_file_query_secdesc(pipes_struct *p, SRV_Q_NET_FILE_QUERY_SECDESC
if(fsp) {
close_file(fsp, True);
- file_free(fsp);
}
if (conn)