[PATCH 2/3] btrfs-progs: send: fix handling of multiple snapshots (-p option)

[Tsutomu Itoh]

We cannot send multiple snapshots at once by -p option.

[before]
# btrfs send -f /tmp/data0 -p Snap0 Snap[12]
At subvol Snap1
At subvol Snap2
ERROR: parent determination failed for 0
# 

[after]
# btrfs send -f /tmp/data0 -p Snap0 Snap[12]
At subvol Snap1
At subvol Snap2
# 

Signed-off-by: Tsutomu Itoh <t-i...@jp.fujitsu.com>
---
 cmds-send.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/cmds-send.c b/cmds-send.c
index dfdfe01..2a8a697 100644
--- a/cmds-send.c
+++ b/cmds-send.c
@@ -650,7 +650,7 @@ int cmd_send(int argc, char **argv)
                        goto out;
                }
 
-               if (!full_send && !parent_root_id) {
+               if (!full_send && root_id) {
                        ret = find_good_parent(&send, root_id, &parent_root_id);
                        if (ret < 0) {
                                error("parent determination failed for %lld",
@@ -673,7 +673,7 @@ int cmd_send(int argc, char **argv)
                if (ret < 0)
                        goto out;
 
-               if (!full_send) {
+               if (!full_send && root_id) {
                        /* done with this subvol, so add it to the clone 
sources */
                        ret = add_clone_source(&send, root_id);
                        if (ret < 0) {
@@ -681,8 +681,6 @@ int cmd_send(int argc, char **argv)
                                goto out;
                        }
                }
-
-               parent_root_id = 0;
        }
 
        ret = 0;
-- 
2.9.3
--
To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html