[PATCH] Fix failure to update dirpath
Michael Adam
obnox at samba.org
Tue Sep 11 06:47:10 UTC 2018
On 2018-08-29 at 01:08 +0200, Michael Adam via samba-technical wrote:
> On 2018-08-28 at 14:08 -0700, Jeremy Allison wrote:
> > On Wed, Aug 22, 2018 at 01:51:00PM -0700, Jeremy Allison wrote:
> > > On Wed, Aug 22, 2018 at 04:35:16PM +0200, Michael Adam wrote:
> > > >
> > > > I do actually think that this is conceptionally
> > > > the much better patch. It fixes the slightly
> > > > arbitrary (and undocumented) behavior to return
> > > > "" for dirpath in the non-optimized case and had
> > > > all callers react to that arbitrary fact. But the
> > > > dirpath was not really useful for other processing.
> > > >
> > > > With this change you always return a valid
> > > > dirpath, and we don't need to change it
> > > > (to ".") for some callers...
> > > >
> > > > A few more cosmetic review commments below in the patch.
> > >
> > > OK, now the "simple" fix has gone in, here is the
> > > cleanup patchset for unix_convert() to ensure we
> > > always have a valid dirpath (incorporates your
> > > comments).
> > >
> > > Please review and let me know if it's OK.
> >
> > Ping ! As you wanted it this way, please take
> > the time to review :-).
>
> Yeah, I started.
> Hope to complete it tomorrow.
Well.... Almost... :-)
There was something more I would like to improve, but we
can do that later. This is a good start, thanks!
LGTM, Reviewed-by: me
Will push later.
(the machine I'm writing from does not have the
samba autobuild env set up.)
Cheers - Michael
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 163 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20180911/9e1426c2/signature.sig>
More information about the samba-technical
mailing list