[PATCH] smbd: Fix snapshot query on shares with DFS enabled

[Christof Schmitt]

On Wed, Aug 17, 2016 at 10:32:38AM -0700, Jeremy Allison wrote:
> On Wed, Aug 17, 2016 at 10:18:21AM -0700, Christof Schmitt wrote:
> > 
> > If cli_set_previous_version(cli, time_t t) already registers the
> > timestamp that should be used for opens, then the cli code could create
> > the twrp context from that for SMB2, or in the SMB1 case insert the @GMT
> > token in the path. That way the protocol differences between SMB1 and
> > SMB2 would be hidden from the application using the cli.
> 
> It's the 'insert the @GMT token in the path' for SMB1 part that is
> hard. It would touch a *LOT* of client code :-(.

Ok, that is the issue of SMB1 doing everything based on the path. If we
want to avoid large changes, then i am not sure how we could have a
clean API.

> 
> > > > For the time being, I think we should apply your shadow_copy_data()
> > > > patches and not my patch that puts the @GMT at the end. That would make
> > > > "allinfo" properly list previous versions against all (reasonably
> > > > recent) versions of samba in SMB2 mode, at the expense of providing a
> > > > not-quite-realistic test for vfs_shadow_copy2.
> > > 
> > > I need to fix the source3/modules/vfs_snapper.c code first :-).
> > > 
> > > I'll send a patch to review soon !
> > 
> > Should we push the fixes you have for shadowcopy2, or do you want to wait
> > until everything is complete, including snapper and tests?
> 
> Already pushed the shadowcopy2 fixes with your reviewed-by.

Found them now. Thanks for moving those along.

> 
> I'm working on the snapper fix right now which I'll propose
> for review today, and then the tests.

Christof