From: Andrew Bartlett <abartlet@samba.org> 
Date: Fri, 17 Aug 2001 08:57:58 +0000 (+0000)
Subject: OK, so not freeing these was a mistake.  I'll try to be less exuberent next
X-Git-Tag: samba-2.2.5pre1~1555^2~144
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=8c3cf2db95a0fcf48b21274cac93f13abb42d4bf;p=thirdparty%2Fsamba.git

OK, so not freeing these was a mistake.  I'll try to be less exuberent next
time :-)
---

diff --git a/source/smbd/service.c b/source/smbd/service.c
index 524b647b5e2..edac97194e3 100644
--- a/source/smbd/service.c
+++ b/source/smbd/service.c
@@ -546,6 +546,7 @@ connection_struct *make_connection(char *service,char *user,char *password, int
 				*ecode = ERRaccess;
 				DEBUG(0,( "make_connection: connection to %s denied due to security descriptor.\n",
 					service ));
+				conn_free(conn);
 				return NULL;
 			} else {
 				conn->read_only = True;
@@ -556,6 +557,7 @@ connection_struct *make_connection(char *service,char *user,char *password, int
 
 	if (!vfs_init(conn)) {
 		DEBUG(0, ("vfs_init failed for service %s\n", lp_servicename(SNUM(conn))));
+		conn_free(conn);
 		return NULL;
 	}