Failure with compound requests and aio enabled

Christof Schmitt cs at samba.org
Fri Sep 22 17:44:40 UTC 2017


On Thu, Sep 21, 2017 at 07:36:44PM -0700, Jeremy Allison wrote:
> On Thu, Sep 21, 2017 at 06:02:52PM -0700, Jeremy Allison via samba-technical wrote:
> > On Thu, Sep 21, 2017 at 05:49:24PM -0700, Christof Schmitt wrote:
> > > On Thu, Sep 21, 2017 at 05:34:44PM -0700, Jeremy Allison wrote:
> > > > On Thu, Sep 21, 2017 at 05:31:16PM -0700, Christof Schmitt wrote:
> > > > > 
> > > > > Nevermind, i found a way. schedule_smb2_sendfile_read() already checks
> > > > > smb2req->in.vector_count to determine whether the request is part of a
> > > > > compound. I will reuse this check. Updated patch will follow.
> > > > 
> > > > Oh that's *great* - thanks ! Not adding that bool
> > > > preserves the VFS ABI. I thought there must be a
> > > > way to detect this...
> > > 
> > > See attached patches. Should i open a bugzilla for a backport?
> > 
> > Thanks - I'll review. There is already a bug report I think
> > I closed :-). You could always reopen that one :-).
> 
> Couldn't find the original so I opened:
> 
> https://bugzilla.samba.org/show_bug.cgi?id=13047
> 
> Reviewed-by: Jeremy Allison <jra at samba.org>
> 
> Pushed - thanks !

Thank you!

Christof



More information about the samba-technical mailing list