[PATCH] New VFS Module "unityed_media" for sharing AVID projects
Jeremy Allison
jra at samba.org
Tue Jan 6 10:19:14 MST 2015
On Tue, Jan 06, 2015 at 06:14:40PM +0100, Ralph Böhme wrote:
> Hi Jeremy,
>
> thanks for looking into this one!
>
> On Mon, Jan 05, 2015 at 02:35:30PM -0800, Jeremy Allison wrote:
> > > updated patch attached.
> >
> > Looks much better, but there's still some cruft
> > in the pathname processing probably left over
> > from the original code implementation.
> >
> > For example: is_in_media_dir() it looks for
> > "./" and "../" - these are stripped out in
> > source3/smbd/reply.c:check_path_syntax_internal()
> > (and have been for may years :-).
>
> Actually, I'm not so sure. I specifically asked a questino wrt to this
> back in October:
> <https://lists.samba.org/archive/samba-technical/2014-October/103197.html>
>
> :)
>
> I seem to remember that I most certainly saw paths starting with "./"
> and the whole thing fell apart when I removed those extra checks from
> the module.
That shouldn't happen of course :-). Can you remove
the checks once it's in and add abort() instead so
I can figure out where these things are coming from ?
> Push it! :)))
Ok, thanks - I'll finish looking today.
More information about the samba-technical
mailing list