[PATCH] Fix failure to update dirpath

[Michael Adam]

On 2018-09-11 at 08:47 +0200, Michael Adam via samba-technical wrote:
> 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.)

And pushed to autobuild.

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/a47bacf9/signature.sig>