api_clone: call is_stopped_locked() to avoid deadlock.

Technically as Dwight has mentioned we should probably drop the locking from api_state() altogether, since those are protected through the lxc command system. Signed-off-by: Serge Hallyn <serge.hallyn@ubuntu.com>

api_clone: call is_stopped_locked() to avoid deadlock.
5790f7b7 · Serge Hallyn · 64f782ca · 5790f7b7
Commit 5790f7b7 authored May 28, 2013 by Serge Hallyn
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

lxccontainer.c src/lxc/lxccontainer.c +1 -1

No files found.
--- a/src/lxc/lxccontainer.c
+++ b/src/lxc/lxccontainer.c
@@ -1827,7 +1827,7 @@ struct lxc_container *lxcapi_clone(struct lxc_container *c, const char *newname,
 	if (container_mem_lock(c))
 		return NULL;

-	if (c->is_running(c)) {
+	if (!is_stopped_locked(c)) {
 		ERROR("error: Original container (%s) is running", c->name);
 		goto out;
 	}