Attachment 204534 Details for Bug 238043 – Proposed patch

[patch] Proposed patch

0001-xdr_rpcb_entry_list_ptr-fix-potential-null-pointer-d.patch (text/plain), 1017 bytes, created by Young on 2019-05-22 08:16:17 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Young

Created: 2019-05-22 08:16:17 UTC

Size: 1017 bytes

patch

obsolete

>From f3ccfff0511c786090a12053a67e552e825bd69a Mon Sep 17 00:00:00 2001
>From: Young Xiao <92siuyang@gmail.com>
>Date: Wed, 22 May 2019 16:16:20 +0800
>Subject: [PATCH] xdr_rpcb_entry_list_ptr: fix potential null pointer
> deference.
>
>Signed-off-by: Young Xiao <92siuyang@gmail.com>
>---
> sys/rpc/rpcb_prot.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/sys/rpc/rpcb_prot.c b/sys/rpc/rpcb_prot.c
>index a59dc50..6712f3c 100644
>--- a/sys/rpc/rpcb_prot.c
>+++ b/sys/rpc/rpcb_prot.c
>@@ -223,14 +223,14 @@ xdr_rpcb_entry_list_ptr(XDR *xdrs, rpcb_entry_list_ptr *rp)
> 		 * the case of freeing we must remember the next object
> 		 * before we free the current object ...
> 		 */
>-		if (freeing)
>+		if (freeing && *rp)
> 			next = (*rp)->rpcb_entry_next;
> 		if (! xdr_reference(xdrs, (caddr_t *)rp,
> 		    (u_int)sizeof (rpcb_entry_list),
> 				    (xdrproc_t)xdr_rpcb_entry)) {
> 			return (FALSE);
> 		}
>-		if (freeing && *rp) {
>+		if (freeing) {
> 			next_copy = next;
> 			rp = &next_copy;
> 			/*
>-- 
>2.7.4
>

Actions: View | Diff

Attachments on bug 238043: 204534