summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorChristof Schmitt <cs@samba.org>2014-06-26 12:43:03 -0700
committerVolker Lendecke <vl@samba.org>2014-06-27 06:49:10 +0200
commitb053d4b3f9fd26f1ea4d81d3a4461a6f74bf7686 (patch)
tree0f35ffd271fc57199fdd33f79b8e3a1331c1b526
parent16a6181289b719baeae7fae7fe0ba746fd5d41a6 (diff)
downloadsamba-b053d4b3f9fd26f1ea4d81d3a4461a6f74bf7686.tar.gz
samba-b053d4b3f9fd26f1ea4d81d3a4461a6f74bf7686.tar.xz
samba-b053d4b3f9fd26f1ea4d81d3a4461a6f74bf7686.zip
s3-shadow-copy2: Remove TODO and fix comments
The patch "s3-shadow-copy2: Fix dir/@GMT-2012.10.15-13.48.43 form of paths" takes care of a case marked as TODO, remove it and adjust the comments accordingly. Signed-off-by: Christof Schmitt <cs@samba.org> Reviewed-by: Volker Lendecke <vl@samba.org>
-rw-r--r--source3/modules/vfs_shadow_copy2.c15
1 files changed, 8 insertions, 7 deletions
diff --git a/source3/modules/vfs_shadow_copy2.c b/source3/modules/vfs_shadow_copy2.c
index 82149f2cb7..029b155208 100644
--- a/source3/modules/vfs_shadow_copy2.c
+++ b/source3/modules/vfs_shadow_copy2.c
@@ -264,7 +264,12 @@ static bool shadow_copy2_strip_snapshot(TALLOC_CTX *mem_ctx,
goto no_snapshot;
}
if (q[0] == '\0') {
- /* the name consists of only the GMT token */
+ /*
+ * The name consists of only the GMT token or the GMT
+ * token is at the end of the path. XP seems to send
+ * @GMT- at the end under certain circumstances even
+ * with a path prefix.
+ */
if (pstripped != NULL) {
stripped = talloc_strndup(mem_ctx, name, p - name);
if (stripped == NULL) {
@@ -277,12 +282,8 @@ static bool shadow_copy2_strip_snapshot(TALLOC_CTX *mem_ctx,
}
if (q[0] != '/') {
/*
- * The GMT token is either at the end of the path
- * or it is not a complete path component, i.e. the
- * path component continues after the gmt-token.
- *
- * TODO: Is this correct? Or would the GMT tag as the
- * last component be a valid input?
+ * It is not a complete path component, i.e. the path
+ * component continues after the gmt-token.
*/
DEBUG(10, ("q[0] = %d\n", (int)q[0]));
goto no_snapshot;