svn commit: samba r15796 - in branches/SAMBA_4_0/source/smb_server:
.
tridge at samba.org
tridge at samba.org
Mon May 22 06:17:11 GMT 2006
Author: tridge
Date: 2006-05-22 06:17:10 +0000 (Mon, 22 May 2006)
New Revision: 15796
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=15796
Log:
this talloc_reference() was causing a memory leak on every NTVFS
request. The problem is that the talloc_free() in
smbsrv_send_reply_nosign() frees this reference, but doesn't free the
memory iself, thus leaving it attached to the connection context.
Metze, what was the idea behind this reference? If we need it, then we
need a corresponding talloc_free() to go with it.
Modified:
branches/SAMBA_4_0/source/smb_server/smb_server.h
Changeset:
Modified: branches/SAMBA_4_0/source/smb_server/smb_server.h
===================================================================
--- branches/SAMBA_4_0/source/smb_server/smb_server.h 2006-05-22 03:55:01 UTC (rev 15795)
+++ branches/SAMBA_4_0/source/smb_server/smb_server.h 2006-05-22 06:17:10 UTC (rev 15796)
@@ -421,10 +421,6 @@
smbsrv_send_error(req, NT_STATUS_NO_MEMORY); \
return; \
} \
- if (!talloc_reference(req->ntvfs, req)) { \
- smbsrv_send_error(req, NT_STATUS_NO_MEMORY); \
- return; \
- } \
req->ntvfs->frontend_data.private_data = req; \
} while (0)
More information about the samba-cvs
mailing list