From 0533fed85adefc9aa74b6d5ae6b4293fb9c11d6a Mon Sep 17 00:00:00 2001 From: Stefan Metzmacher Date: Mon, 6 Mar 2006 15:38:07 +0000 Subject: r13868: remove useless talloc_free() wrapper metze (This used to be commit bd3162e6a7f154630c2d88be70ce19e8dd977133) --- source4/smb_server/smb/receive.c | 4 ++-- source4/smb_server/smb/reply.c | 8 ++++---- source4/smb_server/smb/request.c | 10 +--------- 3 files changed, 7 insertions(+), 15 deletions(-) diff --git a/source4/smb_server/smb/receive.c b/source4/smb_server/smb/receive.c index 1dc65fd5b5..bb37167e79 100644 --- a/source4/smb_server/smb/receive.c +++ b/source4/smb_server/smb/receive.c @@ -515,7 +515,7 @@ static void switch_message(int type, struct smbsrv_request *req) (req->smb_conn->signing.signing_state != SMB_SIGNING_ENGINE_OFF)) { DEBUG(1,("SKIP ERROR REPLY: %s %s because of unknown smb signing case\n", smb_fn_name(type), nt_errstr(status))); - req_destroy(req); + talloc_free(req); return; } req_reply_error(req, status); @@ -543,7 +543,7 @@ static void switch_message(int type, struct smbsrv_request *req) (req->smb_conn->signing.signing_state != SMB_SIGNING_ENGINE_OFF)) { DEBUG(1,("SKIP ERROR REPLY: %s %s because of unknown smb signing case\n", smb_fn_name(type), nt_errstr(status))); - req_destroy(req); + talloc_free(req); return; } req_reply_error(req, status); diff --git a/source4/smb_server/smb/reply.c b/source4/smb_server/smb/reply.c index a5665d3525..3c92c6f6a5 100644 --- a/source4/smb_server/smb/reply.c +++ b/source4/smb_server/smb/reply.c @@ -1759,7 +1759,7 @@ static void reply_lockingX_send(struct smbsrv_request *req) /* if it was an oplock break ack then we only send a reply if there was an error */ if (lck->lockx.in.ulock_cnt + lck->lockx.in.lock_cnt == 0) { - req_destroy(req); + talloc_free(req); return; } @@ -2336,7 +2336,7 @@ void smbsrv_reply_ntcancel(struct smbsrv_request *req) { /* NOTE: this request does not generate a reply */ ntvfs_cancel(req); - req_destroy(req); + talloc_free(req); } /**************************************************************************** @@ -2452,10 +2452,10 @@ void smbsrv_reply_special(struct smbsrv_request *req) case SMBkeepalive: /* session keepalive - swallow it */ - req_destroy(req); + talloc_free(req); return; } DEBUG(0,("Unexpected NBT session packet (%d)\n", msg_type)); - req_destroy(req); + talloc_free(req); } diff --git a/source4/smb_server/smb/request.c b/source4/smb_server/smb/request.c index eb56c9ba17..f8b58fefab 100644 --- a/source4/smb_server/smb/request.c +++ b/source4/smb_server/smb/request.c @@ -32,14 +32,6 @@ /* we over allocate the data buffer to prevent too many realloc calls */ #define REQ_OVER_ALLOCATION 0 -/* destroy a request structure */ -void req_destroy(struct smbsrv_request *req) -{ - /* ahh, its so nice to destroy a complex structure in such a - * simple way! */ - talloc_free(req); -} - /**************************************************************************** construct a basic request packet, mostly used to construct async packets such as change notify and oplock break requests @@ -304,7 +296,7 @@ void req_send_reply_nosign(struct smbsrv_request *req) if (!NT_STATUS_IS_OK(status)) { smbsrv_terminate_connection(req->smb_conn, nt_errstr(status)); } - req_destroy(req); + talloc_free(req); } /* -- cgit