svn commit: samba r14236 - branches/SAMBA_3_0/source/libsmb trunk/source/libsmb

vlendec at samba.org vlendec at samba.org
Sun Mar 12 11:00:08 GMT 2006


Author: vlendec
Date: 2006-03-12 11:00:06 +0000 (Sun, 12 Mar 2006)
New Revision: 14236

WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=14236

Log:
Fix Coverity bug # 90
Modified:
   branches/SAMBA_3_0/source/libsmb/libsmbclient.c
   trunk/source/libsmb/libsmbclient.c


Changeset:
Modified: branches/SAMBA_3_0/source/libsmb/libsmbclient.c
===================================================================
--- branches/SAMBA_3_0/source/libsmb/libsmbclient.c	2006-03-12 10:57:42 UTC (rev 14235)
+++ branches/SAMBA_3_0/source/libsmb/libsmbclient.c	2006-03-12 11:00:06 UTC (rev 14236)
@@ -1070,6 +1070,7 @@
 		if (!cli_resolve_path( "", &srv->cli, path, &targetcli, targetpath))
 		{
 			d_printf("Could not resolve %s\n", path);
+			SAFE_FREE(file);
 			return NULL;
 		}
 		/*d_printf(">>>open: resolved %s as %s\n", path, targetpath);*/

Modified: trunk/source/libsmb/libsmbclient.c
===================================================================
--- trunk/source/libsmb/libsmbclient.c	2006-03-12 10:57:42 UTC (rev 14235)
+++ trunk/source/libsmb/libsmbclient.c	2006-03-12 11:00:06 UTC (rev 14236)
@@ -1070,6 +1070,7 @@
 		if (!cli_resolve_path( "", &srv->cli, path, &targetcli, targetpath))
 		{
 			d_printf("Could not resolve %s\n", path);
+			SAFE_FREE(file);
 			return NULL;
 		}
 		/*d_printf(">>>open: resolved %s as %s\n", path, targetpath);*/



More information about the samba-cvs mailing list