[PATCH v4] File Server Remote VSS Protocol server

David Disseldorp ddiss at suse.de
Mon Mar 30 08:09:29 MDT 2015


On Fri, 27 Mar 2015 21:28:20 -0700, Jeremy Allison wrote:

> On Fri, Mar 27, 2015 at 12:44:09PM +0100, David Disseldorp wrote:
> > This patch-set adds a new File Server Remote VSS (Volume Shadow Copy
> > Service) Protocol server, test infrastructure, documentation, and
> > corresponding VFS layer changes to handle remote share snapshot
> > requests.  
> 
> Pushed, with a minor cleanup in:
> 
> fsrvp: prune shadow copies if associated path doesn't exist

Thanks for the push. Looks like there's a build failure in the unit
tests that I don't hit locally:

In file included from ../source4/torture/local/local.c:25:
default/source4/torture/rpc/proto.h:22: error: 'struct policy_handle' declared inside parameter list
default/source4/torture/rpc/proto.h:22: error: its scope is only this definition or declaration, which is probably not what you want
default/source4/torture/rpc/proto.h:22: error: 'struct dcerpc_binding_handle' declared inside parameter list
...

The autogenerated torture/rpc/proto.h header isn't needed here anyway,
as torture_local_fsrvp() is provided by torture/local/proto.h.

Please squash this patch:

diff --git a/source4/torture/local/local.c b/source4/torture/local/local.c
index db729af..5d3b4e1 100644
--- a/source4/torture/local/local.c
+++ b/source4/torture/local/local.c
@@ -22,7 +22,6 @@
 #include "torture/local/proto.h"
 #include "torture/ndr/proto.h"
 #include "torture/auth/proto.h"
-#include "torture/rpc/proto.h"
 #include "../lib/crypto/test_proto.h"
 #include "lib/registry/tests/proto.h"
 #include "lib/replace/replace-testsuite.h"

...into "torture: add local FSRVP server state tests"

Otherwise I can send the whole patch-set as a new round if you'd
prefer.

Cheers, David


More information about the samba-technical mailing list