svn commit: samba r14014 - in branches/SAMBA_3_0/source/rpc_parse: .
jra at samba.org
jra at samba.org
Wed Mar 8 06:16:55 GMT 2006
Author: jra
Date: 2006-03-08 06:16:54 +0000 (Wed, 08 Mar 2006)
New Revision: 14014
WebSVN: http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=rev&root=samba&rev=14014
Log:
Coverity paranoia. Shut it up by making the guarentee
in the code explicit - but this was a false positive (CID #16).
Jeremy.
Modified:
branches/SAMBA_3_0/source/rpc_parse/parse_buffer.c
Changeset:
Modified: branches/SAMBA_3_0/source/rpc_parse/parse_buffer.c
===================================================================
--- branches/SAMBA_3_0/source/rpc_parse/parse_buffer.c 2006-03-08 05:47:51 UTC (rev 14013)
+++ branches/SAMBA_3_0/source/rpc_parse/parse_buffer.c 2006-03-08 06:16:54 UTC (rev 14014)
@@ -47,7 +47,7 @@
{
prs_debug(ps, depth, desc, "prs_rpcbuffer");
depth++;
-
+
/* reading */
if (UNMARSHALLING(ps)) {
buffer->size=0;
@@ -135,7 +135,12 @@
}
return True;
}
-
+
+ /* Coverity paranoia. Buffer must be valid. */
+ if (!*buffer) {
+ return False;
+ }
+
return prs_rpcbuffer( desc, ps, depth, *buffer);
}
More information about the samba-cvs
mailing list