dlls/rpcrt4/rpc_server.c initiale variable

Gerald Pfeifer gerald at pfeifer.com
Wed Oct 24 07:33:39 CDT 2007


On Wed, 24 Oct 2007, Robert Shearman wrote:
> It's a false positive, probably coming from the compiler not picking
> up that it is set when the exception variable is false and that when 
> exception is true it is set earlier in the function.

Ah, thanks for the analysis.  I couldn't convince myself that was the
case. ;-)

> If you're going to change the code at all, then at least keep to the 
> coding convention used in the rest of the function and use NULL here.
 
Fair point!  Updated patch below.

Gerald

Index: dlls/rpcrt4/rpc_server.c
===================================================================
RCS file: /home/wine/wine/dlls/rpcrt4/rpc_server.c,v
retrieving revision 1.85
diff -u -3 -p -r1.85 rpc_server.c
--- dlls/rpcrt4/rpc_server.c	22 Aug 2007 11:24:40 -0000	1.85
+++ dlls/rpcrt4/rpc_server.c	24 Oct 2007 12:32:18 -0000
@@ -167,7 +167,7 @@ static void RPCRT4_process_packet(RpcCon
   RpcServerInterface* sif;
   RPC_DISPATCH_FUNCTION func;
   UUID *object_uuid;
-  RpcPktHdr *response;
+  RpcPktHdr *response = NULL;
   void *buf = msg->Buffer;
   RPC_STATUS status;
   BOOL exception;



More information about the wine-patches mailing list