]> git.proxmox.com Git - mirror_lxc.git/blobdiff - src/tests/get_item.c
confile: add lxc.init.groups to keep additional groups
[mirror_lxc.git] / src / tests / get_item.c
index 270ced9087631beabf8d7f4b89c6c02b4451c3c5..c503ddb994e7e250fb6ab9f449611ade0a5890af 100644 (file)
 #include <stdlib.h>
 #include <errno.h>
 #include <string.h>
+
 #include "lxc/state.h"
+#include "lxctest.h"
 
 #define MYNAME "lxctest1"
 
 int main(int argc, char *argv[])
 {
-       struct lxc_container *c;
        int ret;
+       struct lxc_container *c;
+       int fret = EXIT_FAILURE;
        char v1[2], v2[256], v3[2048];
 
        if ((c = lxc_container_new("testxyz", NULL)) == NULL) {
                fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
-               ret = 1;
+               exit(EXIT_FAILURE);
+       }
+
+       /* EXPECT SUCCESS: lxc.log.syslog with valid value. */
+       if (!c->set_config_item(c, "lxc.log.syslog", "local0")) {
+               lxc_error("%s\n", "Failed to set lxc.log.syslog.\n");
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.log.syslog", v2, 255);
+       if (ret < 0) {
+               lxc_error("Failed to retrieve lxc.log.syslog: %d.\n", ret);
                goto out;
        }
 
+       if (strcmp(v2, "local0") != 0) {
+               lxc_error("Expected: local0 == %s.\n", v2);
+               goto out;
+       }
+       lxc_debug("Retrieving value for lxc.log.syslog correctly returned: %s.\n", v2);
+
+       /* EXPECT FAILURE: lxc.log.syslog with invalid value. */
+       if (c->set_config_item(c, "lxc.log.syslog", "NONSENSE")) {
+               lxc_error("%s\n", "Succeeded int setting lxc.log.syslog to invalid value \"NONSENSE\".\n");
+               goto out;
+       }
+       lxc_debug("%s\n", "Successfully failed to set lxc.log.syslog to invalid value.\n");
+
        if (!c->set_config_item(c, "lxc.hook.pre-start", "hi there")) {
                fprintf(stderr, "%d: failed to set hook.pre-start\n", __LINE__);
-               ret = 1;
                goto out;
        }
+
        ret = c->get_config_item(c, "lxc.hook.pre-start", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.hook.pre-start) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        fprintf(stderr, "lxc.hook.pre-start returned %d %s\n", ret, v2);
 
-       ret = c->get_config_item(c, "lxc.network", v2, 255);
+       ret = c->get_config_item(c, "lxc.net", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
-       fprintf(stderr, "%d: get_config_item(lxc.network) returned %d %s\n", __LINE__, ret, v2);
-       if (!c->set_config_item(c, "lxc.tty", "4")) {
+       fprintf(stderr, "%d: get_config_item(lxc.net) returned %d %s\n", __LINE__, ret, v2);
+
+       if (!c->set_config_item(c, "lxc.tty.max", "4")) {
                fprintf(stderr, "%d: failed to set tty\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.tty", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.tty.max", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.tty) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        fprintf(stderr, "lxc.tty returned %d %s\n", ret, v2);
 
        if (!c->set_config_item(c, "lxc.arch", "x86")) {
                fprintf(stderr, "%d: failed to set arch\n", __LINE__);
-               ret = 1;
                goto out;
        }
+
        ret = c->get_config_item(c, "lxc.arch", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.arch) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("lxc.arch returned %d %s\n", ret, v2);
 
-       if (!c->set_config_item(c, "lxc.init_uid", "100")) {
+       if (!c->set_config_item(c, "lxc.init.uid", "100")) {
                fprintf(stderr, "%d: failed to set init_uid\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.init_uid", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.init.uid", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.init_uid) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("lxc.init_uid returned %d %s\n", ret, v2);
 
-       if (!c->set_config_item(c, "lxc.init_gid", "100")) {
+       if (!c->set_config_item(c, "lxc.init.gid", "100")) {
                fprintf(stderr, "%d: failed to set init_gid\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.init_gid", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.init.gid", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.init_gid) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("lxc.init_gid returned %d %s\n", ret, v2);
 
+       if (!c->set_config_item(c, "lxc.init.groups", "")) {
+               fprintf(stderr, "%d: failed to set init_groups\n", __LINE__);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.init.groups", "10,20,foo,40")) {
+               printf("failed to set init_groups to '10,20,foo,40' as expected\n");
+       } else {
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.init.groups", "10,20,30,40")) {
+               fprintf(stderr, "%d: failed to set init_groups\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.init.groups", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.init_gid) returned %d\n",
+                       __LINE__, ret);
+               goto out;
+       }
+       ret = strcmp("10,20,30,40", v2);
+       printf("lxc.init_groups returned %d %s\n", ret, v2);
+       if (ret != 0) {
+               goto out;
+       }
+
 #define HNAME "hostname1"
        // demonstrate proper usage:
        char *alloced;
-       if (!c->set_config_item(c, "lxc.utsname", HNAME)) {
+       int len;
+
+       if (!c->set_config_item(c, "lxc.uts.name", HNAME)) {
                fprintf(stderr, "%d: failed to set utsname\n", __LINE__);
-               ret = 1;
                goto out;
        }
 
-       int len;
-       len = c->get_config_item(c, "lxc.utsname", NULL, 0);  // query the size of the string
+       len = c->get_config_item(c, "lxc.uts.name", NULL, 0);  // query the size of the string
        if (len < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.utsname) returned %d\n", __LINE__, len);
-               ret = 1;
                goto out;
        }
        printf("lxc.utsname returned %d\n", len);
@@ -136,20 +185,19 @@ int main(int argc, char *argv[])
        alloced = malloc(len+1);
        if (!alloced) {
                fprintf(stderr, "%d: failed to allocate %d bytes for utsname\n", __LINE__, len);
-               ret = 1;
                goto out;
        }
+
        // now pass in the malloc'd array, and pass in length of string + 1: again
        // because we need room for the trailing \0
-       ret = c->get_config_item(c, "lxc.utsname", alloced, len+1);
+       ret = c->get_config_item(c, "lxc.uts.name", alloced, len+1);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.utsname) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
+
        if (strcmp(alloced, HNAME) != 0 || ret != len) {
                fprintf(stderr, "lxc.utsname returned wrong value: %d %s not %d %s\n", ret, alloced, len, HNAME);
-               ret = 1;
                goto out;
        }
        printf("lxc.utsname returned %d %s\n", len, alloced);
@@ -157,26 +205,239 @@ int main(int argc, char *argv[])
 
        if (!c->set_config_item(c, "lxc.mount.entry", "hi there")) {
                fprintf(stderr, "%d: failed to set mount.entry\n", __LINE__);
-               ret = 1;
                goto out;
        }
+
        ret = c->get_config_item(c, "lxc.mount.entry", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.mount.entry) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("lxc.mount.entry returned %d %s\n", ret, v2);
 
-       if (!c->set_config_item(c, "lxc.aa_profile", "unconfined")) {
+       ret = c->get_config_item(c, "lxc.prlimit", v3, 2047);
+       if (ret != 0) {
+               fprintf(stderr, "%d: get_config_item(limit) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.prlimit.nofile", "1234:unlimited")) {
+               fprintf(stderr, "%d: failed to set limit.nofile\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.prlimit.nofile", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.prlimit.nofile) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "1234:unlimited")) {
+               fprintf(stderr, "%d: lxc.prlimit.nofile returned wrong value: %d %s not 14 1234:unlimited\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.prlimit.nofile returned %d %s\n", ret, v2);
+
+       if (!c->set_config_item(c, "lxc.prlimit.stack", "unlimited")) {
+               fprintf(stderr, "%d: failed to set limit.stack\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.prlimit.stack", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.prlimit.stack) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "unlimited")) {
+               fprintf(stderr, "%d: lxc.prlimit.stack returned wrong value: %d %s not 9 unlimited\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.prlimit.stack returned %d %s\n", ret, v2);
+
+#define LIMIT_STACK "lxc.prlimit.stack = unlimited\n"
+#define ALL_LIMITS "lxc.prlimit.nofile = 1234:unlimited\n" LIMIT_STACK
+       ret = c->get_config_item(c, "lxc.prlimit", v3, 2047);
+       if (ret != sizeof(ALL_LIMITS)-1) {
+               fprintf(stderr, "%d: get_config_item(limit) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, ALL_LIMITS)) {
+               fprintf(stderr, "%d: lxc.prlimit returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(ALL_LIMITS)-1, ALL_LIMITS);
+               goto out;
+       }
+       printf("lxc.prlimit returned %d %s\n", ret, v3);
+
+       if (!c->clear_config_item(c, "lxc.prlimit.nofile")) {
+               fprintf(stderr, "%d: failed clearing limit.nofile\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.prlimit", v3, 2047);
+       if (ret != sizeof(LIMIT_STACK)-1) {
+               fprintf(stderr, "%d: get_config_item(limit) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, LIMIT_STACK)) {
+               fprintf(stderr, "%d: lxc.prlimit returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(LIMIT_STACK)-1, LIMIT_STACK);
+               goto out;
+       }
+       printf("lxc.prlimit returned %d %s\n", ret, v3);
+
+#define SYSCTL_SOMAXCONN "lxc.sysctl.net.core.somaxconn = 256\n"
+#define ALL_SYSCTLS "lxc.sysctl.net.ipv4.ip_forward = 1\n" SYSCTL_SOMAXCONN
+
+       ret = c->get_config_item(c, "lxc.sysctl", v3, 2047);
+       if (ret != 0) {
+               fprintf(stderr, "%d: get_config_item(sysctl) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.sysctl.net.ipv4.ip_forward", "1")) {
+               fprintf(stderr, "%d: failed to set lxc.sysctl.net.ipv4.ip_forward\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.sysctl.net.ipv4.ip_forward", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.sysctl.net.ipv4.ip_forward) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "1")) {
+               fprintf(stderr, "%d: lxc.sysctl.net.ipv4.ip_forward returned wrong value: %d %s not 1\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.sysctl.net.ipv4.ip_forward returned %d %s\n", ret, v2);
+
+       if (!c->set_config_item(c, "lxc.sysctl.net.core.somaxconn", "256")) {
+               fprintf(stderr, "%d: failed to set lxc.sysctl.net.core.somaxconn\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.sysctl.net.core.somaxconn", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.sysctl.net.core.somaxconn) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "256")) {
+               fprintf(stderr, "%d: lxc.sysctl.net.core.somaxconn returned wrong value: %d %s not 256\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.sysctl.net.core.somaxconn returned %d %s\n", ret, v2);
+
+       ret = c->get_config_item(c, "lxc.sysctl", v3, 2047);
+       if (ret != sizeof(ALL_SYSCTLS)-1) {
+               fprintf(stderr, "%d: get_config_item(sysctl) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, ALL_SYSCTLS)) {
+               fprintf(stderr, "%d: lxc.sysctl returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(ALL_SYSCTLS) - 1, ALL_SYSCTLS);
+               goto out;
+       }
+       printf("lxc.sysctl returned %d %s\n", ret, v3);
+
+       if (!c->clear_config_item(c, "lxc.sysctl.net.ipv4.ip_forward")) {
+               fprintf(stderr, "%d: failed clearing lxc.sysctl.net.ipv4.ip_forward\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.sysctl", v3, 2047);
+       if (ret != sizeof(SYSCTL_SOMAXCONN) - 1) {
+               fprintf(stderr, "%d: get_config_item(sysctl) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, SYSCTL_SOMAXCONN)) {
+               fprintf(stderr, "%d: lxc.sysctl returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(SYSCTL_SOMAXCONN) - 1, SYSCTL_SOMAXCONN);
+               goto out;
+       }
+       printf("lxc.sysctl returned %d %s\n", ret, v3);
+
+#define PROC_OOM_SCORE_ADJ "lxc.proc.oom_score_adj = 10\n"
+#define ALL_PROCS "lxc.proc.setgroups = allow\n" PROC_OOM_SCORE_ADJ
+
+       ret = c->get_config_item(c, "lxc.proc", v3, 2047);
+       if (ret != 0) {
+               fprintf(stderr, "%d: get_config_item(proc) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.proc.setgroups", "allow")) {
+               fprintf(stderr, "%d: failed to set lxc.proc.setgroups\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.proc.setgroups", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.proc.setgroups) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "allow")) {
+               fprintf(stderr, "%d: lxc.proc.setgroups returned wrong value: %d %s not 10\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.proc.setgroups returned %d %s\n", ret, v2);
+
+       if (!c->set_config_item(c, "lxc.proc.oom_score_adj", "10")) {
+               fprintf(stderr, "%d: failed to set lxc.proc.oom_score_adj\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.proc.oom_score_adj", v2, 255);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(lxc.proc.oom_score_adj) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v2, "10")) {
+               fprintf(stderr, "%d: lxc.proc.oom_score_adj returned wrong value: %d %s not 10\n", __LINE__, ret, v2);
+               goto out;
+       }
+       printf("lxc.proc.oom_score_adj returned %d %s\n", ret, v2);
+
+       ret = c->get_config_item(c, "lxc.proc", v3, 2047);
+       if (ret != sizeof(ALL_PROCS)-1) {
+               fprintf(stderr, "%d: get_config_item(proc) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, ALL_PROCS)) {
+               fprintf(stderr, "%d: lxc.proc returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(ALL_PROCS) - 1, ALL_PROCS);
+               goto out;
+       }
+       printf("lxc.proc returned %d %s\n", ret, v3);
+
+       if (!c->clear_config_item(c, "lxc.proc.setgroups")) {
+               fprintf(stderr, "%d: failed clearing lxc.proc.setgroups\n", __LINE__);
+               goto out;
+       }
+
+       ret = c->get_config_item(c, "lxc.proc", v3, 2047);
+       if (ret < 0) {
+               fprintf(stderr, "%d: get_config_item(proc) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+
+       if (strcmp(v3, PROC_OOM_SCORE_ADJ)) {
+               fprintf(stderr, "%d: lxc.proc returned wrong value: %d %s not %d %s\n", __LINE__, ret, v3, (int)sizeof(PROC_OOM_SCORE_ADJ) - 1, PROC_OOM_SCORE_ADJ);
+               goto out;
+       }
+       printf("lxc.proc returned %d %s\n", ret, v3);
+
+       if (!c->set_config_item(c, "lxc.apparmor.profile", "unconfined")) {
                fprintf(stderr, "%d: failed to set aa_profile\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.aa_profile", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.apparmor.profile", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.aa_profile) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("lxc.aa_profile returned %d %s\n", ret, v2);
@@ -186,7 +447,6 @@ int main(int argc, char *argv[])
        // new test with real container
        if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
                fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
-               ret = 1;
                goto out;
        }
        c->destroy(c);
@@ -194,130 +454,139 @@ int main(int argc, char *argv[])
 
        if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
                fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
-               ret = 1;
                goto out;
        }
+
        if (!c->createl(c, "busybox", NULL, NULL, 0, NULL)) {
                fprintf(stderr, "%d: failed to create a trusty container\n", __LINE__);
-               ret = 1;
                goto out;
        }
-
        lxc_container_put(c);
 
        /* XXX TODO load_config needs to clear out any old config first */
        if ((c = lxc_container_new(MYNAME, NULL)) == NULL) {
                fprintf(stderr, "%d: error opening lxc_container %s\n", __LINE__, MYNAME);
-               ret = 1;
                goto out;
        }
 
        ret = c->get_config_item(c, "lxc.cap.drop", NULL, 300);
        if (ret < 5 || ret > 255) {
                fprintf(stderr, "%d: get_config_item(lxc.cap.drop) with NULL returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
+
        ret = c->get_config_item(c, "lxc.cap.drop", v1, 1);
        if (ret < 5 || ret > 255) {
                fprintf(stderr, "%d: get_config_item(lxc.cap.drop) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
+
        ret = c->get_config_item(c, "lxc.cap.drop", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(lxc.cap.drop) returned %d %s\n", __LINE__, ret, v2);
-               ret = 1;
                goto out;
        }
        printf("%d: get_config_item(lxc.cap.drop) returned %d %s\n", __LINE__, ret, v2);
-       ret = c->get_config_item(c, "lxc.network", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.net", v2, 255);
        if (ret < 0) {
-               fprintf(stderr, "%d: get_config_item returned %d\n", __LINE__, ret);
-               ret = 1;
+               fprintf(stderr, "%d: get_config_item(lxc.net) returned %d\n", __LINE__, ret);
+               goto out;
+       }
+       printf("%d: get_config_item(lxc.net) returned %d %s\n", __LINE__, ret, v2);
+
+       if (!c->set_config_item(c, "lxc.net.0.type", "veth")) {
+               fprintf(stderr, "%d: failed to set lxc.net.0.type\n", __LINE__);
                goto out;
        }
-       printf("%d: get_config_item(lxc.network) returned %d %s\n", __LINE__, ret, v2);
 
-       if (!c->set_config_item(c, "lxc.network.ipv4", "10.2.3.4")) {
+       if (!c->set_config_item(c, "lxc.net.0.link", "lxcbr0")) {
+               fprintf(stderr, "%d: failed to set network.link\n", __LINE__);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.net.0.flags", "up")) {
+               fprintf(stderr, "%d: failed to set network.flags\n", __LINE__);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.net.0.hwaddr", "00:16:3e:xx:xx:xx")) {
+               fprintf(stderr, "%d: failed to set network.hwaddr\n", __LINE__);
+               goto out;
+       }
+
+       if (!c->set_config_item(c, "lxc.net.0.ipv4.address", "10.2.3.4")) {
                fprintf(stderr, "%d: failed to set ipv4\n", __LINE__);
-               ret = 1;
                goto out;
        }
 
-       ret = c->get_config_item(c, "lxc.network.0.ipv4", v2, 255);
+       ret = c->get_config_item(c, "lxc.net.0.ipv4.address", v2, 255);
        if (ret <= 0) {
-               fprintf(stderr, "%d: lxc.network.0.ipv4 returned %d\n", __LINE__, ret);
-               ret = 1;
+               fprintf(stderr, "%d: lxc.net.0.ipv4 returned %d\n", __LINE__, ret);
                goto out;
        }
-       if (!c->clear_config_item(c, "lxc.network.0.ipv4")) {
+
+       if (!c->clear_config_item(c, "lxc.net.0.ipv4.address")) {
                fprintf(stderr, "%d: failed clearing all ipv4 entries\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.network.0.ipv4", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.net.0.ipv4.address", v2, 255);
        if (ret != 0) {
                fprintf(stderr, "%d: after clearing ipv4 entries get_item(lxc.network.0.ipv4 returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
 
-       if (!c->set_config_item(c, "lxc.network.ipv4.gateway", "10.2.3.254")) {
+       if (!c->set_config_item(c, "lxc.net.0.ipv4.gateway", "10.2.3.254")) {
                fprintf(stderr, "%d: failed to set ipv4.gateway\n", __LINE__);
-               ret = 1;
                goto out;
        }
 
-       ret = c->get_config_item(c, "lxc.network.0.ipv4.gateway", v2, 255);
+       ret = c->get_config_item(c, "lxc.net.0.ipv4.gateway", v2, 255);
        if (ret <= 0) {
-               fprintf(stderr, "%d: lxc.network.0.ipv4.gateway returned %d\n", __LINE__, ret);
-               ret = 1;
+               fprintf(stderr, "%d: lxc.net.0.ipv4.gateway returned %d\n", __LINE__, ret);
                goto out;
        }
-       if (!c->set_config_item(c, "lxc.network.0.ipv4.gateway", "")) {
+
+       if (!c->set_config_item(c, "lxc.net.0.ipv4.gateway", "")) {
                fprintf(stderr, "%d: failed clearing ipv4.gateway\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.network.0.ipv4.gateway", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.net.0.ipv4.gateway", v2, 255);
        if (ret != 0) {
                fprintf(stderr, "%d: after clearing ipv4.gateway get_item(lxc.network.0.ipv4.gateway returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
 
-       ret = c->get_config_item(c, "lxc.network.0.link", v2, 255);
+       ret = c->get_config_item(c, "lxc.net.0.link", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("%d: get_config_item (link) returned %d %s\n", __LINE__, ret, v2);
-       ret = c->get_config_item(c, "lxc.network.0.name", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.net.0.name", v2, 255);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("%d: get_config_item (name) returned %d %s\n", __LINE__, ret, v2);
 
-       if (!c->clear_config_item(c, "lxc.network")) {
+       if (!c->clear_config_item(c, "lxc.net")) {
                fprintf(stderr, "%d: clear_config_item failed\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       ret = c->get_config_item(c, "lxc.network", v2, 255);
+
+       ret = c->get_config_item(c, "lxc.net", v2, 255);
        if (ret != 0) {
                fprintf(stderr, "%d: network was not actually cleared (get_network returned %d)\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
 
        ret = c->get_config_item(c, "lxc.cgroup", v3, 2047);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(cgroup.devices) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("%d: get_config_item (cgroup.devices) returned %d %s\n", __LINE__, ret, v3);
@@ -325,35 +594,59 @@ int main(int argc, char *argv[])
        ret = c->get_config_item(c, "lxc.cgroup.devices.allow", v3, 2047);
        if (ret < 0) {
                fprintf(stderr, "%d: get_config_item(cgroup.devices.devices.allow) returned %d\n", __LINE__, ret);
-               ret = 1;
                goto out;
        }
        printf("%d: get_config_item (cgroup.devices.devices.allow) returned %d %s\n", __LINE__, ret, v3);
 
        if (!c->clear_config_item(c, "lxc.cgroup")) {
                fprintf(stderr, "%d: failed clearing lxc.cgroup\n", __LINE__);
-               ret = 1;
                goto out;
        }
+
        if (!c->clear_config_item(c, "lxc.cap.drop")) {
                fprintf(stderr, "%d: failed clearing lxc.cap.drop\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       if (!c->clear_config_item(c, "lxc.mount.entries")) {
-               fprintf(stderr, "%d: failed clearing lxc.mount.entries\n", __LINE__);
-               ret = 1;
+
+       if (!c->clear_config_item(c, "lxc.mount.entry")) {
+               fprintf(stderr, "%d: failed clearing lxc.mount.entry\n", __LINE__);
                goto out;
        }
+
        if (!c->clear_config_item(c, "lxc.hook")) {
                fprintf(stderr, "%d: failed clearing lxc.hook\n", __LINE__);
-               ret = 1;
                goto out;
        }
-       c->destroy(c);
+
+       if (!lxc_config_item_is_supported("lxc.arch")) {
+               fprintf(stderr, "%d: failed to report \"lxc.arch\" as supported configuration item\n", __LINE__);
+               goto out;
+       }
+
+       if (lxc_config_item_is_supported("lxc.nonsense")) {
+               fprintf(stderr, "%d: failed to detect \"lxc.nonsense\" as unsupported configuration item\n", __LINE__);
+               goto out;
+       }
+
+       if (lxc_config_item_is_supported("lxc.arch.nonsense")) {
+               fprintf(stderr, "%d: failed to detect \"lxc.arch.nonsense\" as unsupported configuration item\n", __LINE__);
+               goto out;
+       }
+
+       if (c->set_config_item(c, "lxc.notaconfigkey", "invalid")) {
+               fprintf(stderr, "%d: Managed to set \"lxc.notaconfigkey\"\n", __LINE__);
+               goto out;
+       }
+
+
        printf("All get_item tests passed\n");
-       ret = 0;
+       fret = EXIT_SUCCESS;
+
 out:
-       lxc_container_put(c);
-       exit(ret);
-};
+       if (c) {
+               c->destroy(c);
+               lxc_container_put(c);
+       }
+
+       exit(fret);
+}