]> git.michaelhowe.org Git - packages/o/openafs.git/commitdiff
avoid call to rxi_FreePacket with NULL pointer
authorHans-Werner Paulsen <hans@MPA-Garching.MPG.DE>
Wed, 22 Jul 2009 13:14:33 +0000 (15:14 +0200)
committerJeffrey Altman <jaltman@openafs.org>
Wed, 22 Jul 2009 13:53:16 +0000 (06:53 -0700)
the function rx_ReadProc32 (src/rx/rx_rdwr.c) checks the pointer
call->currentPacket before the function rxi_FreePacket is called. This
test was missing in the function rx_ReadProc.

FIXES 125110

Reviewed-on: http://gerrit.openafs.org/http://gerrit.openafs.org/178
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
src/rx/rx_rdwr.c

index 5aca9168ea95ff2ba9b9f0c8d99c6f1a0c2cffdc..61f6b2aa24a43e49276f4b5e3b46797cacbd92b6 100644 (file)
@@ -343,7 +343,7 @@ rx_ReadProc(struct rx_call *call, char *buf, int nbytes)
        call->curlen = tcurlen - nbytes;
        call->nLeft = tnLeft - nbytes;
 
-        if (!call->nLeft) {
+        if (!call->nLeft && call->currentPacket != NULL) {
             /* out of packet.  Get another one. */
             NETPRI;
             MUTEX_ENTER(&call->lock);