From: Tobias Ulmer <tu@emlix.com>
Date: Thu, 1 Nov 2012 14:37:59 +0000 (+0100)
Subject: e2lib.mv(): it's safer to pass arguments in a vector
X-Git-Tag: e2factory-2.3.13rc1~99
X-Git-Url: https://git.e2factory.org/?a=commitdiff_plain;h=953ae57eb44272f6e3737f82954ddb3ac0b33ad9;p=e2factory.git

e2lib.mv(): it's safer to pass arguments in a vector

Signed-off-by: Tobias Ulmer <tu@emlix.com>
---

diff --git a/generic/e2lib.lua b/generic/e2lib.lua
index fda1cee..b339349 100644
--- a/generic/e2lib.lua
+++ b/generic/e2lib.lua
@@ -1615,8 +1615,10 @@ end
 -- @return bool
 -- @return the last line ouf captured output
 function e2lib.mv(src, dst)
-    local args = string.format("'%s' '%s'", src, dst)
-    return e2lib.call_tool("mv", args)
+    assert(type(src) == "string" and type(dst) == "string")
+    assert(string.len(src) > 0 and string.len(dst) > 0)
+
+    return e2lib.call_tool_argv("mv", { src, dst })
 end
 
 --- call the cp command