[lxc-devel] [PATCH] lxc api: fix some config_path oddities
Serge Hallyn
serge.hallyn at canonical.com
Fri Feb 8 22:06:32 UTC 2013
1. When calling c->set_config_path(), update configfile. I.e. if we
are setting the config_path to /var/lib/lxc, then the configfile should
be changed to /var/lib/lxc/$container/config
2. Add an optional configpath argument to lxc_container_new. If NULL,
then the default will be used (as before). If set, then the passed-in
path will be used. This way you can do
c1 = lxc.Container("r1", "/var/lib/lxc");
c2 = lxc.Container("r2", "/home/user/lxcbase");
(Note I did *not* implement the python or lua binding to pass that
argument along)
Signed-off-by: Serge Hallyn <serge.hallyn at ubuntu.com>
---
src/lua-lxc/core.c | 2 +-
src/lxc/lxc.h | 2 +-
src/lxc/lxccontainer.c | 72 ++++++++++++++++++++++++++++++++-------
src/lxc/lxccontainer.h | 2 +-
src/python-lxc/lxc.c | 2 +-
src/tests/Makefile.am | 3 +-
src/tests/containertests.c | 4 +--
src/tests/createtest.c | 2 +-
src/tests/destroytest.c | 2 +-
src/tests/get_item.c | 8 ++---
src/tests/getkeys.c | 2 +-
src/tests/lxcpath.c | 85 ++++++++++++++++++++++++++++++++++++++++++++++
src/tests/saveconfig.c | 2 +-
src/tests/shutdowntest.c | 2 +-
src/tests/startone.c | 2 +-
15 files changed, 162 insertions(+), 30 deletions(-)
create mode 100644 src/tests/lxcpath.c
diff --git a/src/lua-lxc/core.c b/src/lua-lxc/core.c
index ae4d9b2..9225158 100644
--- a/src/lua-lxc/core.c
+++ b/src/lua-lxc/core.c
@@ -44,7 +44,7 @@
static int container_new(lua_State *L)
{
const char *name = luaL_checkstring(L, 1);
- struct lxc_container *c = lxc_container_new(name);
+ struct lxc_container *c = lxc_container_new(name, NULL);
if (c) {
lua_boxpointer(L, c);
diff --git a/src/lxc/lxc.h b/src/lxc/lxc.h
index 349bbbc..a651d04 100644
--- a/src/lxc/lxc.h
+++ b/src/lxc/lxc.h
@@ -182,7 +182,7 @@ extern const char const *lxc_version(void);
/*
* Create and return a new lxccontainer struct.
*/
-extern struct lxc_container *lxc_container_new(const char *name);
+extern struct lxc_container *lxc_container_new(const char *name, const char *configpath);
/*
* Returns 1 on success, 0 on failure.
diff --git a/src/lxc/lxccontainer.c b/src/lxc/lxccontainer.c
index 6f01645..733cbb6 100644
--- a/src/lxc/lxccontainer.c
+++ b/src/lxc/lxccontainer.c
@@ -871,10 +871,45 @@ static const char *lxcapi_get_config_path(struct lxc_container *c)
return (const char *)(c->config_path);
}
+/*
+ * not for export
+ * Just recalculate the c->configfile based on the
+ * c->config_path, which must be set.
+ * The lxc_container must be locked or not yet public.
+ */
+static bool set_config_filename(struct lxc_container *c)
+{
+ char *newpath;
+ int len, ret;
+
+ if (!c->config_path)
+ return false;
+
+ /* $lxc_path + "/" + c->name + "/" + "config" + '\0' */
+ len = strlen(c->config_path) + strlen(c->name) + strlen("config") + 3;
+ newpath = malloc(len);
+ if (!newpath)
+ return false;
+
+ ret = snprintf(newpath, len, "%s/%s/config", c->config_path, c->name);
+ if (ret < 0 || ret >= len) {
+ fprintf(stderr, "Error printing out config file name\n");
+ free(newpath);
+ return false;
+ }
+
+ if (c->configfile)
+ free(c->configfile);
+ c->configfile = newpath;
+
+ return true;
+}
+
static bool lxcapi_set_config_path(struct lxc_container *c, const char *path)
{
char *p;
bool b = false;
+ char *oldpath = NULL;
if (!c)
return b;
@@ -883,13 +918,28 @@ static bool lxcapi_set_config_path(struct lxc_container *c, const char *path)
return b;
p = strdup(path);
- if (!p)
+ if (!p) {
+ ERROR("Out of memory setting new lxc path");
goto err;
+ }
+
b = true;
if (c->config_path)
- free(c->config_path);
+ oldpath = c->config_path;
c->config_path = p;
+
+ /* Since we've changed the config path, we have to change the
+ * config file name too */
+ if (!set_config_filename(c)) {
+ ERROR("Out of memory setting new config filename");
+ b = false;
+ free(c->config_path);
+ c->config_path = oldpath;
+ oldpath = NULL;
+ }
err:
+ if (oldpath)
+ free(oldpath);
lxcunlock(c->privlock);
return b;
}
@@ -938,10 +988,9 @@ out:
}
-struct lxc_container *lxc_container_new(const char *name)
+struct lxc_container *lxc_container_new(const char *name, const char *configpath)
{
struct lxc_container *c;
- int ret, len;
c = malloc(sizeof(*c));
if (!c) {
@@ -950,7 +999,11 @@ struct lxc_container *lxc_container_new(const char *name)
}
memset(c, 0, sizeof(*c));
- c->config_path = default_lxc_path();
+ if (configpath)
+ c->config_path = strdup(configpath);
+ else
+ c->config_path = default_lxc_path();
+
if (!c->config_path) {
fprintf(stderr, "Out of memory");
goto err;
@@ -976,17 +1029,10 @@ struct lxc_container *lxc_container_new(const char *name)
goto err;
}
- len = strlen(c->config_path)+strlen(c->name)+strlen("/config")+2;
- c->configfile = malloc(len);
- if (!c->configfile) {
+ if (!set_config_filename(c)) {
fprintf(stderr, "Error allocating config file pathname\n");
goto err;
}
- ret = snprintf(c->configfile, len, "%s/%s/config", c->config_path, c->name);
- if (ret < 0 || ret >= len) {
- fprintf(stderr, "Error printing out config file name\n");
- goto err;
- }
if (file_exists(c->configfile))
lxcapi_load_config(c, NULL);
diff --git a/src/lxc/lxccontainer.h b/src/lxc/lxccontainer.h
index 32c501e..de802a8 100644
--- a/src/lxc/lxccontainer.h
+++ b/src/lxc/lxccontainer.h
@@ -82,7 +82,7 @@ struct lxc_container {
#endif
};
-struct lxc_container *lxc_container_new(const char *name);
+struct lxc_container *lxc_container_new(const char *name, const char *configpath);
int lxc_container_get(struct lxc_container *c);
int lxc_container_put(struct lxc_container *c);
int lxc_get_wait_states(const char **states);
diff --git a/src/python-lxc/lxc.c b/src/python-lxc/lxc.c
index 83e2659..16356f9 100644
--- a/src/python-lxc/lxc.c
+++ b/src/python-lxc/lxc.c
@@ -85,7 +85,7 @@ Container_init(Container *self, PyObject *args, PyObject *kwds)
&name))
return -1;
- self->container = lxc_container_new(name);
+ self->container = lxc_container_new(name, NULL);
if (!self->container) {
fprintf(stderr, "%d: error creating lxc_container %s\n", __LINE__, name);
return -1;
diff --git a/src/tests/Makefile.am b/src/tests/Makefile.am
index 5850727..a883d36 100644
--- a/src/tests/Makefile.am
+++ b/src/tests/Makefile.am
@@ -15,6 +15,7 @@ lxc_test_createtest_SOURCES = createtest.c
lxc_test_shutdowntest_SOURCES = shutdowntest.c
lxc_test_get_item_SOURCES = get_item.c
lxc_test_getkeys_SOURCES = getkeys.c
+lxc_test_lxcpath_SOURCES = lxcpath.c
AM_CFLAGS=-I$(top_srcdir)/src \
-DLXCROOTFSMOUNT=\"$(LXCROOTFSMOUNT)\" \
@@ -24,6 +25,6 @@ AM_CFLAGS=-I$(top_srcdir)/src \
bin_PROGRAMS = lxc-test-containertests lxc-test-locktests lxc-test-startone \
lxc-test-destroytest lxc-test-saveconfig lxc-test-createtest \
- lxc-test-shutdowntest lxc-test-get_item lxc-test-getkeys
+ lxc-test-shutdowntest lxc-test-get_item lxc-test-getkeys lxc-test-lxcpath
endif
diff --git a/src/tests/containertests.c b/src/tests/containertests.c
index d68f17c..8868faa 100644
--- a/src/tests/containertests.c
+++ b/src/tests/containertests.c
@@ -103,7 +103,7 @@ int main(int argc, char *argv[])
ret = 1;
/* test refcounting */
- c = lxc_container_new(MYNAME);
+ c = lxc_container_new(MYNAME, NULL);
if (!c) {
fprintf(stderr, "%d: error creating lxc_container %s\n", __LINE__, MYNAME);
goto out;
@@ -149,7 +149,7 @@ int main(int argc, char *argv[])
}
/* test a real container */
- c = lxc_container_new(MYNAME);
+ c = lxc_container_new(MYNAME, NULL);
if (!c) {
fprintf(stderr, "%d: error creating lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
diff --git a/src/tests/createtest.c b/src/tests/createtest.c
index 48ce922..c2abee2 100644
--- a/src/tests/createtest.c
+++ b/src/tests/createtest.c
@@ -33,7 +33,7 @@ int main(int argc, char *argv[])
struct lxc_container *c;
int ret = 1;
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/destroytest.c b/src/tests/destroytest.c
index 28f0577..0552b4c 100644
--- a/src/tests/destroytest.c
+++ b/src/tests/destroytest.c
@@ -65,7 +65,7 @@ int main(int argc, char *argv[])
struct lxc_container *c;
int ret = 1;
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/get_item.c b/src/tests/get_item.c
index 0d32eca..d3e6d29 100644
--- a/src/tests/get_item.c
+++ b/src/tests/get_item.c
@@ -35,7 +35,7 @@ int main(int argc, char *argv[])
int ret;
char v1[2], v2[256], v3[2048];
- if ((c = lxc_container_new("testxyz")) == NULL) {
+ if ((c = lxc_container_new("testxyz", NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
@@ -157,7 +157,7 @@ int main(int argc, char *argv[])
lxc_container_put(c);
// new test with real container
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
@@ -165,7 +165,7 @@ int main(int argc, char *argv[])
c->destroy(c);
lxc_container_put(c);
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
@@ -179,7 +179,7 @@ int main(int argc, char *argv[])
lxc_container_put(c);
/* XXX TODO load_config needs to clear out any old config first */
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/getkeys.c b/src/tests/getkeys.c
index 9bb5593..bab6883 100644
--- a/src/tests/getkeys.c
+++ b/src/tests/getkeys.c
@@ -35,7 +35,7 @@ int main(int argc, char *argv[])
int len, ret;
char v3[2048];
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/lxcpath.c b/src/tests/lxcpath.c
new file mode 100644
index 0000000..8d8ad9c
--- /dev/null
+++ b/src/tests/lxcpath.c
@@ -0,0 +1,85 @@
+/* liblxcapi
+ *
+ * Copyright © 2012 Serge Hallyn <serge.hallyn at ubuntu.com>.
+ * Copyright © 2012 Canonical Ltd.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License version 2, as
+ * published by the Free Software Foundation.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
+ */
+#include "../lxc/lxccontainer.h"
+
+#include <unistd.h>
+#include <signal.h>
+#include <stdio.h>
+#include <sys/types.h>
+#include <sys/wait.h>
+#include <stdlib.h>
+#include <errno.h>
+
+#define MYNAME "lxctest1"
+
+#define TSTERR(x) do { \
+ fprintf(stderr, "%d: %s", __LINE__, x); \
+} while (0)
+
+int main()
+{
+ struct lxc_container *c;
+ const char *p1, *p2;
+ int retval = -1;
+
+ c = lxc_container_new(MYNAME, NULL);
+ if (!c) {
+ TSTERR("create using default path");
+ goto err;
+ }
+ p1 = c->get_config_path(c);
+ p2 = c->config_file_name(c);
+ if (!p1 || !p2 || strncmp(p1, p2, strlen(p1))) {
+ TSTERR("Bad result for path names");
+ goto err;
+ }
+
+#define CPATH "/boo"
+#define FPATH "/boo/lxctest1/config"
+ if (!c->set_config_path(c, "/boo")) {
+ TSTERR("Error setting custom path");
+ goto err;
+ }
+ p1 = c->get_config_path(c);
+ p2 = c->config_file_name(c);
+ if (strcmp(p1, CPATH) || strcmp(p2, FPATH)) {
+ TSTERR("Bad result for path names after set_config_path()");
+ goto err;
+ }
+ lxc_container_put(c);
+
+ c = lxc_container_new(MYNAME, CPATH);
+ if (!c) {
+ TSTERR("create using custom path");
+ goto err;
+ }
+
+ p1 = c->get_config_path(c);
+ p2 = c->config_file_name(c);
+ if (strcmp(p1, CPATH) || strcmp(p2, FPATH)) {
+ TSTERR("Bad result for path names after create with custom path");
+ goto err;
+ }
+
+ retval = 0;
+
+err:
+ lxc_container_put(c);
+ return retval;
+}
diff --git a/src/tests/saveconfig.c b/src/tests/saveconfig.c
index bbaea19..fa84e32 100644
--- a/src/tests/saveconfig.c
+++ b/src/tests/saveconfig.c
@@ -65,7 +65,7 @@ int main(int argc, char *argv[])
struct lxc_container *c;
int ret = 1;
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/shutdowntest.c b/src/tests/shutdowntest.c
index 9ad8a65..a1a84e8 100644
--- a/src/tests/shutdowntest.c
+++ b/src/tests/shutdowntest.c
@@ -34,7 +34,7 @@ int main(int argc, char *argv[])
struct lxc_container *c;
int ret = 1;
- if ((c = lxc_container_new(MYNAME)) == NULL) {
+ if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
goto out;
diff --git a/src/tests/startone.c b/src/tests/startone.c
index f76ed1e..1eb3e99 100644
--- a/src/tests/startone.c
+++ b/src/tests/startone.c
@@ -103,7 +103,7 @@ int main(int argc, char *argv[])
ret = 1;
/* test a real container */
- c = lxc_container_new(MYNAME);
+ c = lxc_container_new(MYNAME, NULL);
if (!c) {
fprintf(stderr, "%d: error creating lxc_container %s\n", __LINE__, MYNAME);
ret = 1;
--
1.8.1.2
More information about the lxc-devel
mailing list