[PATCH] Fix bug 10982 - fallocate() returned values on failure

Jeremy Allison jra at samba.org
Sun Dec 7 16:15:05 MST 2014


On Sun, Dec 07, 2014 at 08:33:46PM +0100, David Disseldorp wrote:
> On Fri, 5 Dec 2014 15:43:15 -0800
> Jeremy Allison <jra at samba.org> wrote:
> 
> > Here is the patchset for master confirmed as
> > working by Jones <jones.kstw at gmail.com> for
> > bug #10982 - fallocate() returned values on failure.
> > 
> > Please review and push !
> 
> A couple of minor comments:
> - sys_posix_fallocate() may still return an errno value

Yes, that's deliberate.

Because posix_fallocate() by definition returns either
zero or errno, I want our wrapper to do the same.

The conversion to -1,errno is done in the caller
of sys_posix_fallocate(), so 

> - smb_time_audit_fallocate() should save errno before calling
>   clock_gettime()

Got it - thanks !

> Otherwise looks good.

Is that a 'Reviewed-by:' ? I'm assuming yes :-).


More information about the samba-technical mailing list