[SCM] Samba Shared Repository - branch master updated

Jeremy Allison jra at samba.org
Tue Nov 5 10:49:02 MST 2013


On Tue, Nov 05, 2013 at 04:04:37PM +0100, David Disseldorp wrote:
> Hi Jeremy,
> 
> On Mon,  4 Nov 2013 23:11:02 +0100 (CET)
> jra at samba.org (Jeremy Allison) wrote:
> 
> >     Fix bug #10229 - No access check verification on stream files.
> >     
> >     https://bugzilla.samba.org/show_bug.cgi?id=10229
> >     
> >     We need to check if the requested access mask
> >     could be used to open the underlying file (if
> >     it existed), as we're passing in zero for the
> >     access mask to the base filename.
> >     
> >     Signed-off-by: Jeremy Allison <jra at samba.org>
> >     Reviewed-by: Stefan Metzmacher <metze at samba.org>
> >     Reviewed-by: David Disseldorp <ddiss at suse.de>
> 
> Please avoid adding my reviewed-by tag unless I've explicitly acked the
> patch, via bugzilla, mail or otherwise.
> I'm okay with the change as is, but in this case it was in the tree
> before I saw your follow-up to my stat failure query.

Sorry, I saw the "I'm ok with this patch" in the
bug report and took this as a "Reviewed-by".

Hopefully you can see why I assumed you'd +1'ed
it.

Jeremy.


More information about the samba-technical mailing list