diff options
| author | Andrew Bartlett <abartlet@samba.org> | 2001-07-02 13:43:46 +0000 |
|---|---|---|
| committer | Andrew Bartlett <abartlet@samba.org> | 2001-07-02 13:43:46 +0000 |
| commit | 5d3f622375842961a79536d36ee6f6ad508b91f1 (patch) | |
| tree | 107b0d28ca58f2d41658b24ee585ba7dbe578f3a | |
| parent | d04ea88b5d4b010d77dec45104dcf6a2b5a071f2 (diff) | |
| download | samba-5d3f622375842961a79536d36ee6f6ad508b91f1.tar.gz samba-5d3f622375842961a79536d36ee6f6ad508b91f1.tar.xz samba-5d3f622375842961a79536d36ee6f6ad508b91f1.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
| -rw-r--r-- | source/rpc_server/srv_srvsvc_nt.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/source/rpc_server/srv_srvsvc_nt.c b/source/rpc_server/srv_srvsvc_nt.c index 39e482dc3cb..44821d1432f 100644 --- a/source/rpc_server/srv_srvsvc_nt.c +++ b/source/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) |