[sheepdog] [PATCH v2 2/4] dog: use nr_copies of inode object	instead of command line parameter
    Hitoshi Mitake 
    mitake.hitoshi at gmail.com
       
    Wed May  7 16:38:28 CEST 2014
    
    
  
From: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
Current "dog vdi clone" has a possibility of creating clone volume
which has a different replica number from its original volume when it
receives -c option. This patch forbid such a situation by letting
vdi_clone() use inode->nr_copies instead of vdi_cmd_data.nr_copies
(the variable which holds a value passed with -c option).
Cc: MORITA Kazutaka <morita.kazutaka at lab.ntt.co.jp>
Cc: Valerio Pachera <sirio81 at gmail.com>
Cc: Alessandro Bolgia <alessandro at extensys.it>
Signed-off-by: Hitoshi Mitake <mitake.hitoshi at lab.ntt.co.jp>
---
v2: enhance commit log
 dog/vdi.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dog/vdi.c b/dog/vdi.c
index 1851a2a..7cf1bf1 100644
--- a/dog/vdi.c
+++ b/dog/vdi.c
@@ -562,7 +562,7 @@ static int vdi_clone(int argc, char **argv)
 		goto out;
 
 	ret = do_vdi_create(dst_vdi, inode->vdi_size, base_vid, &new_vid, false,
-			    vdi_cmd_data.nr_copies, inode->copy_policy,
+			    inode->nr_copies, inode->copy_policy,
 			    inode->store_policy);
 	if (ret != EXIT_SUCCESS || !vdi_cmd_data.prealloc)
 		goto out;
-- 
1.9.1
    
    
More information about the sheepdog
mailing list