Re: PR/50546 CVS commit: src/usr.bin/pmap

[Joerg Sonnenberger <joerg%britannica.bec.de@localhost>]

On Mon, Dec 14, 2015 at 12:10:00AM +0000, Christos Zoulas wrote:
> The following reply was made to PR bin/50546; it has been noted by GNATS.
> 
> From: christos%zoulas.com@localhost (Christos Zoulas)
> To: gnats-bugs%NetBSD.org@localhost, gnats-admin%netbsd.org@localhost, netbsd-bugs%netbsd.org@localhost, 
> 	dcb314%hotmail.com@localhost
> Cc: 
> Subject: Re: PR/50546 CVS commit: src/usr.bin/pmap
> Date: Sun, 13 Dec 2015 19:08:24 -0500
> 
>  On Dec 13, 11:50pm, joerg%britannica.bec.de@localhost (Joerg Sonnenberger) wrote:
>  -- Subject: Re: PR/50546 CVS commit: src/usr.bin/pmap
>  
>  |  Please use calloc.
>  
>  I had it and reverted it, because it does not help since it does the multiply
>  immediately to load the data so zeroing is just a waste.

It still ensures the overflow protection. For any non-trivial size, the
zeroing is likely already done by the kernel anyway.

Joerg