[SCM] Samba Shared Repository - branch master updated
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
More information about the samba-technical