]> git.ipfire.org Git - thirdparty/lxc.git/commitdiff
CVE 2018-6556: verify netns fd in lxc-user-nic
authorChristian Brauner <christian.brauner@ubuntu.com>
Wed, 25 Jul 2018 17:56:54 +0000 (19:56 +0200)
committerStéphane Graber <stgraber@ubuntu.com>
Mon, 6 Aug 2018 16:05:29 +0000 (12:05 -0400)
Signed-off-by: Christian Brauner <christian.brauner@ubuntu.com>
src/lxc/lxc_user_nic.c
src/lxc/utils.c
src/lxc/utils.h

index 2a5c3a43a09f3ac2dedc5d0508a249ed4e03a948..b7c72abd777b753585160067e885c44f17b7ea7e 100644 (file)
@@ -1129,12 +1129,41 @@ int main(int argc, char *argv[])
                        exit(EXIT_FAILURE);
                }
        } else if (request == LXC_USERNIC_DELETE) {
-               netns_fd = open(args.pid, O_RDONLY);
+               char opath[LXC_PROC_PID_FD_LEN];
+
+               /* Open the path with O_PATH which will not trigger an actual
+                * open(). Don't report an errno to the caller to not leak
+                * information whether the path exists or not.
+                * When stracing setuid is stripped so this is not a concern
+                * either.
+                */
+               netns_fd = open(args.pid, O_PATH | O_CLOEXEC);
                if (netns_fd < 0) {
-                       usernic_error("Could not open \"%s\": %s\n", args.pid,
-                                     strerror(errno));
+                       usernic_error("Failed to open \"%s\"\n", args.pid);
                        exit(EXIT_FAILURE);
                }
+
+               if (!fhas_fs_type(netns_fd, NSFS_MAGIC)) {
+                       usernic_error("Path \"%s\" does not refer to a network namespace path\n", args.pid);
+                       close(netns_fd);
+                       exit(EXIT_FAILURE);
+               }
+
+               ret = snprintf(opath, sizeof(opath), "/proc/self/fd/%d", netns_fd);
+               if (ret < 0 || (size_t)ret >= sizeof(opath)) {
+                       close(netns_fd);
+                       exit(EXIT_FAILURE);
+               }
+
+               /* Now get an fd that we can use in setns() calls. */
+               ret = open(opath, O_RDONLY | O_CLOEXEC);
+               if (ret < 0) {
+                       usernic_error("Failed to open \"%s\": %s\n", args.pid, strerror(errno));
+                       close(netns_fd);
+                       exit(EXIT_FAILURE);
+               }
+               close(netns_fd);
+               netns_fd = ret;
        }
 
        if (!create_db_dir(LXC_USERNIC_DB)) {
index 10e14b7f3548df259ce5f620a48647dbc784395c..eb0af822288a09ccc3ee746e3fc832af8eb9c8b8 100644 (file)
@@ -2319,6 +2319,18 @@ bool has_fs_type(const char *path, fs_type_magic magic_val)
        return has_type;
 }
 
+bool fhas_fs_type(int fd, fs_type_magic magic_val)
+{
+       int ret;
+       struct statfs sb;
+
+       ret = fstatfs(fd, &sb);
+       if (ret < 0)
+               return false;
+
+       return is_fs_type(&sb, magic_val);
+}
+
 bool lxc_nic_exists(char *nic)
 {
 #define __LXC_SYS_CLASS_NET_LEN 15 + IFNAMSIZ + 1
index e4d8519dbefc30c4383b79d4042f217e26e39f90..56fed0a42c3d1c5a6a745e783856615e03eda29c 100644 (file)
 #define CGROUP2_SUPER_MAGIC 0x63677270
 #endif
 
+#ifndef NSFS_MAGIC
+#define NSFS_MAGIC 0x6e736673
+#endif
+
 /* Useful macros */
 /* Maximum number for 64 bit integer is a string with 21 digits: 2^64 - 1 = 21 */
 #define LXC_NUMSTRLEN64 21
@@ -555,6 +559,7 @@ extern void *must_realloc(void *orig, size_t sz);
 /* __typeof__ should be safe to use with all compilers. */
 typedef __typeof__(((struct statfs *)NULL)->f_type) fs_type_magic;
 extern bool has_fs_type(const char *path, fs_type_magic magic_val);
+extern bool fhas_fs_type(int fd, fs_type_magic magic_val);
 extern bool is_fs_type(const struct statfs *fs, fs_type_magic magic_val);
 extern bool lxc_nic_exists(char *nic);
 extern int lxc_make_tmpfile(char *template, bool rm);