Re: [PATCH] nvmecontrol breaks world

2013-07-02 Thread Steve Kargl
On Tue, Jul 02, 2013 at 10:30:53PM +0200, Ed Schouten wrote: > 2013/7/2 Steve Kargl : > > 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? > Also, the call of malloc(sb.st_size) is not rea

Re: [PATCH] nvmecontrol breaks world

2013-07-02 Thread Steve Kargl
On Tue, Jul 02, 2013 at 10:37:35PM +0200, Dimitry Andric wrote: > On Jul 2, 2013, at 22:30, Ed Schouten wrote: > > 2013/7/2 Steve Kargl : > >> 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 expli

Re: [PATCH] nvmecontrol breaks world

2013-07-02 Thread Dimitry Andric
On Jul 2, 2013, at 22:30, Ed Schouten wrote: > 2013/7/2 Steve Kargl : >> 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

Re: [PATCH] nvmecontrol breaks world

2013-07-02 Thread Ed Schouten
2013/7/2 Steve Kargl : > 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? Also, the call of malloc(sb.st_size) is not really safe... -- Ed Schouten ___

[PATCH] nvmecontrol breaks world

2013-07-02 Thread Steve Kargl
Could someone (this could even be me, but need approval) please fix nvmecontrol? Index: nvmecontrol/firmware.c === --- nvmecontrol/firmware.c (revision 252514) +++ nvmecontrol/firmware.c (working copy) @@ -82,7 +82,7 @@ read