svn commit: samba r13916 - branches/SAMBA_3_0/source/smbd trunk/source/smbd

vlendec at samba.org vlendec at samba.org
Tue Mar 7 09:09:14 GMT 2006


Author: vlendec
Date: 2006-03-07 09:09:13 +0000 (Tue, 07 Mar 2006)
New Revision: 13916

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

Log:
Fix Coverity bug #29. Looks like my code. I wonder how much there is still
lurking...

Volker

Modified:
   branches/SAMBA_3_0/source/smbd/oplock.c
   trunk/source/smbd/oplock.c


Changeset:
Modified: branches/SAMBA_3_0/source/smbd/oplock.c
===================================================================
--- branches/SAMBA_3_0/source/smbd/oplock.c	2006-03-07 06:31:04 UTC (rev 13915)
+++ branches/SAMBA_3_0/source/smbd/oplock.c	2006-03-07 09:09:13 UTC (rev 13916)
@@ -764,6 +764,7 @@
 	if (lck == NULL) {
 		DEBUG(0,("release_level_2_oplocks_on_change: failed to lock "
 			 "share mode entry for file %s.\n", fsp->fsp_name ));
+		return;
 	}
 
 	DEBUG(10,("release_level_2_oplocks_on_change: num_share_modes = %d\n", 

Modified: trunk/source/smbd/oplock.c
===================================================================
--- trunk/source/smbd/oplock.c	2006-03-07 06:31:04 UTC (rev 13915)
+++ trunk/source/smbd/oplock.c	2006-03-07 09:09:13 UTC (rev 13916)
@@ -764,6 +764,7 @@
 	if (lck == NULL) {
 		DEBUG(0,("release_level_2_oplocks_on_change: failed to lock "
 			 "share mode entry for file %s.\n", fsp->fsp_name ));
+		return;
 	}
 
 	DEBUG(10,("release_level_2_oplocks_on_change: num_share_modes = %d\n", 



More information about the samba-cvs mailing list