Make LXC_CLONE_KEEPNAME work

Passing the LXC_CLONE_KEEPNAME flag to do_lxcapi_clone() was not respected. We wrap clear_unexp_config_line() and set_config_item_line() in an appropriate if-condition. Signed-off-by: Christian Brauner <christianvanbrauner@gmail.com>

Make LXC_CLONE_KEEPNAME work
eab218fb · Christian Brauner · 840d2afe · eab218fb
Commit eab218fb authored Aug 08, 2015 by Christian Brauner
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 4 deletions

lxccontainer.c src/lxc/lxccontainer.c +7 -4

No files found.
--- a/src/lxc/lxccontainer.c
+++ b/src/lxc/lxccontainer.c
@@ -2906,12 +2906,15 @@ static struct lxc_container *do_lxcapi_clone(struct lxc_container *c, const char
 	if (ret < 0)
 		goto out;

-	clear_unexp_config_line(c2->lxc_conf, "lxc.utsname", false);

 	// update utsname
-	if (!set_config_item_locked(c2, "lxc.utsname", newname)) {
-		ERROR("Error setting new hostname");
-		goto out;
+	if (!(flags & LXC_CLONE_KEEPNAME)) {
+		clear_unexp_config_line(c2->lxc_conf, "lxc.utsname", false);
+
+		if (!set_config_item_locked(c2, "lxc.utsname", newname)) {
+			ERROR("Error setting new hostname");
+			goto out;
+		}
 	}

 	// copy hooks