storage: avoid segfault

When the "lxc.rootfs.path" property is not set and users request a container
copy we would segfault since strstr() would be called on a NULL pointer.

Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>

diff --git a/src/lxc/storage/storage.c b/src/lxc/storage/storage.c
index 820fd24078e143b84f98a11a1d75c14f0bc5ec9c..90053152ed55fcb3a2f0600f374df1b075529382 100644 (file)
--- a/src/lxc/storage/storage.c
+++ b/src/lxc/storage/storage.c
@@ -312,9 +312,14 @@ struct lxc_storage *storage_copy(struct lxc_container *c0, const char *cname,
        const char *oldpath = c0->config_path;
        struct rsync_data data;
 
-       /* if the container name doesn't show up in the rootfs path, then
-        * we don't know how to come up with a new name
+       /* If the container name doesn't show up in the rootfs path, then we
+        * don't know how to come up with a new name.
         */
+       if (!src) {
+               ERROR("No rootfs specified");
+               return NULL;
+       }
+
        if (strstr(src, oldname) == NULL) {
                ERROR(
                    "original rootfs path %s doesn't include container name %s",