Re: [PATCH] nvmecontrol breaks world

From: Dimitry Andric <dim_at_FreeBSD.org>
Date: Tue, 2 Jul 2013 22:37:35 +0200
On Jul 2, 2013, at 22:30, Ed Schouten <ed_at_80386.nl> wrote:
> 2013/7/2 Steve Kargl <sgk_at_troutmask.apl.washington.edu>:
>> Could someone (this could even be me, but need approval) please
>> fix nvmecontrol?
> 
> off_t doesn't need to be intmax_t, right? Maybe add an explicit cast?

Yes, that is what Bruce has suggested for off_t many times in the past. :-)


> Also, the call of malloc(sb.st_size) is not really safe...

Sure, but that does not break buildworld (for gcc users, I assume? the tinderboxes are all green).

-Dimitry
Received on Tue Jul 02 2013 - 18:37:39 UTC

This archive was generated by hypermail 2.4.0 : Wed May 19 2021 - 11:40:39 UTC