From: Ulrich Drepper Date: Fri, 2 Aug 2002 01:33:11 +0000 (+0000) Subject: (xdr_array): Check for overflow on multiplication. X-Git-Tag: upstream/2.30~21558 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=12892ded5d913b3c45ea8f22634afd3b273b473d;p=external%2Fglibc.git (xdr_array): Check for overflow on multiplication. --- diff --git a/sunrpc/xdr_array.c b/sunrpc/xdr_array.c index 461a6ad..9f67e7f 100644 --- a/sunrpc/xdr_array.c +++ b/sunrpc/xdr_array.c @@ -45,6 +45,7 @@ static char sccsid[] = "@(#)xdr_array.c 1.10 87/08/11 Copyr 1984 Sun Micro"; #include #include #include +#include #ifdef USE_IN_LIBIO # include @@ -81,7 +82,11 @@ xdr_array (xdrs, addrp, sizep, maxsize, elsize, elproc) return FALSE; } c = *sizep; - if ((c > maxsize) && (xdrs->x_op != XDR_FREE)) + /* + * XXX: Let the overflow possibly happen with XDR_FREE because mem_free() + * doesn't actually use its second argument anyway. + */ + if ((c > maxsize || c > UINT_MAX / elsize) && (xdrs->x_op != XDR_FREE)) { return FALSE; }