From 5c0870024d40e6880366430a5c78393c2c3a3ea3 Mon Sep 17 00:00:00 2001 From: smitsohu Date: Thu, 4 Mar 2021 14:33:58 +0100 Subject: [PATCH] private-lib: move from copying to mounting (#3980) --- src/firejail/fs_lib.c | 214 ++++++++++++++++++----------------------- src/firejail/fs_lib2.c | 41 ++++---- src/lib/ldd_utils.c | 4 +- 3 files changed, 113 insertions(+), 146 deletions(-) diff --git a/src/firejail/fs_lib.c b/src/firejail/fs_lib.c index 99d57fbbbaa..5c1229f2c18 100644 --- a/src/firejail/fs_lib.c +++ b/src/firejail/fs_lib.c @@ -23,7 +23,8 @@ #include #include #include -#include +#include +#include #include #define MAXBUF 4096 @@ -105,22 +106,6 @@ char *find_in_path(const char *program) { return NULL; } -static void report_duplication(const char *full_path) { - char *fname = strrchr(full_path, '/'); - if (fname && *(++fname) != '\0') { - // report the file on all bin paths - int i = 0; - while (default_lib_paths[i]) { - char *p; - if (asprintf(&p, "%s/%s", default_lib_paths[i], fname) == -1) - errExit("asprintf"); - fs_logger2("clone", p); - free(p); - i++; - } - } -} - static char *build_dest_dir(const char *full_path) { assert(full_path); if (strstr(full_path, "/x86_64-linux-gnu/")) @@ -128,57 +113,107 @@ static char *build_dest_dir(const char *full_path) { return RUN_LIB_DIR; } -// copy fname in private_run_dir -void fslib_duplicate(const char *full_path) { +// return name of mount target in allocated memory +static char *build_dest_name(const char *full_path) { assert(full_path); + char *fname = strrchr(full_path, '/'); + assert(fname); + fname++; + assert(*fname != '\0'); - struct stat s; - if (stat(full_path, &s) != 0 || s.st_uid != 0 || access(full_path, R_OK) - || !valid_full_path(full_path)) - return; + char *dest; + if (asprintf(&dest, "%s/%s", build_dest_dir(full_path), fname) == -1) + errExit("asprintf"); + return dest; +} - char *dest_dir = build_dest_dir(full_path); +static void fslib_mount_dir(const char *src) { + // create new directory and mount the original on top of it + char *dest = build_dest_name(src); + if (mkdir(dest, 0755) == -1) { + if (errno == EEXIST) { // directory has been mounted already, nothing to do + free(dest); + return; + } + errExit("mkdir"); + } - // don't copy it if the file is already there - char *ptr = strrchr(full_path, '/'); - if (!ptr) - return; - ptr++; - if (*ptr == '\0') - return; + if (arg_debug || arg_debug_private_lib) + printf(" mounting %s on %s\n", src, dest); + // if src is a symbolic link, mount will follow it + if (mount(src, dest, NULL, MS_BIND|MS_REC, NULL) < 0) + errExit("mount bind"); + free(dest); + dir_cnt++; +} - char *name; - if (asprintf(&name, "%s/%s", dest_dir, ptr) == -1) - errExit("asprintf"); - if (stat(name, &s) == 0) { - free(name); - return; +static void fslib_mount_file(const char *src) { + // create new file and mount the original on top of it + char *dest = build_dest_name(src); + int fd = open(dest, O_RDONLY|O_CREAT|O_EXCL|O_CLOEXEC, S_IRUSR | S_IWUSR); + if (fd == -1) { + if (errno == EEXIST) { // file has been mounted already, nothing to do + free(dest); + return; + } + errExit("open"); } - free(name); + close(fd); if (arg_debug || arg_debug_private_lib) - printf(" copying %s to private %s\n", full_path, dest_dir); - - sbox_run(SBOX_ROOT| SBOX_SECCOMP, 4, PATH_FCOPY, "--follow-link", full_path, dest_dir); - report_duplication(full_path); + printf(" mounting %s on %s\n", src, dest); + // if src is a symbolic link, mount will follow it + if (mount(src, dest, NULL, MS_BIND, NULL) < 0) + errExit("mount bind"); + free(dest); lib_cnt++; } +void fslib_mount(const char *full_path) { + assert(full_path); + struct stat s; + + if (!valid_full_path(full_path) || + access(full_path, F_OK) != 0 || + stat(full_path, &s) != 0 || + s.st_uid != 0) + return; + + if (S_ISREG(s.st_mode) && is_lib_64(full_path)) + fslib_mount_file(full_path); + else if (S_ISDIR(s.st_mode)) + fslib_mount_dir(full_path); +} + // requires full path for lib // it could be a library or an executable // lib is not copied, only libraries used by it -static void fslib_copy_libs(const char *full_path, unsigned mask) { +void fslib_mount_libs(const char *full_path, unsigned user) { + assert(full_path); + // if library/executable does not exist or the user does not have read access to it + // print a warning and exit the function. + if (user && access(full_path, R_OK)) { + if (arg_debug || arg_debug_private_lib) + printf("Cannot read %s, skipping...\n", full_path); + return; + } + + if (arg_debug || arg_debug_private_lib) + printf(" fslib_mount_libs %s (parse as %s)\n", full_path, user ? "user" : "root"); // create an empty RUN_LIB_FILE and allow the user to write to it unlink(RUN_LIB_FILE); // in case is there create_empty_file_as_root(RUN_LIB_FILE, 0644); - if (mask & SBOX_USER) { - if (chown(RUN_LIB_FILE, getuid(), getgid())) - errExit("chown"); - } + if (user && chown(RUN_LIB_FILE, getuid(), getgid())) + errExit("chown"); // run fldd to extract the list of files if (arg_debug || arg_debug_private_lib) printf(" running fldd %s\n", full_path); + unsigned mask; + if (user) + mask = SBOX_USER; + else + mask = SBOX_ROOT; sbox_run(mask | SBOX_SECCOMP | SBOX_CAPS_NONE, 3, PATH_FLDD, full_path, RUN_LIB_FILE); // open the list of libraries and install them on by one @@ -192,96 +227,31 @@ static void fslib_copy_libs(const char *full_path, unsigned mask) { char *ptr = strchr(buf, '\n'); if (ptr) *ptr = '\0'; - fslib_duplicate(buf); + + fslib_mount(buf); } fclose(fp); unlink(RUN_LIB_FILE); } -void fslib_copy_libs_parse_as_root(const char *full_path) { - assert(full_path); - if (arg_debug || arg_debug_private_lib) - printf(" fslib_copy_libs_parse_as_root %s\n", full_path); - - struct stat s; - if (stat(full_path, &s)) { - if (arg_debug || arg_debug_private_lib) - printf("cannot find %s for private-lib, skipping...\n", full_path); - return; - } - fslib_copy_libs(full_path, SBOX_ROOT); -} - -// if library/executable does not exist or the user does not have read access to it -// print a warning and exit the function. -void fslib_copy_libs_parse_as_user(const char *full_path) { - assert(full_path); - if (arg_debug || arg_debug_private_lib) - printf(" fslib_copy_libs_parse_as_user %s\n", full_path); - - if (access(full_path, R_OK)) { - if (arg_debug || arg_debug_private_lib) - printf("cannot find %s for private-lib, skipping...\n", full_path); - return; - } - fslib_copy_libs(full_path, SBOX_USER); -} - -void fslib_copy_dir(const char *full_path) { - assert(full_path); - if (arg_debug || arg_debug_private_lib) - printf(" fslib_copy_dir %s\n", full_path); - - // do nothing if the directory does not exist or is not owned by root - struct stat s; - if (stat(full_path, &s) != 0 || s.st_uid != 0 || !S_ISDIR(s.st_mode) || access(full_path, R_OK) - || !valid_full_path(full_path)) - return; - - char *dir_name = strrchr(full_path, '/'); - assert(dir_name); - dir_name++; - assert(*dir_name != '\0'); - - // do nothing if the directory is already there - char *dest; - if (asprintf(&dest, "%s/%s", build_dest_dir(full_path), dir_name) == -1) - errExit("asprintf"); - if (stat(dest, &s) == 0) { - free(dest); - return; - } - - // create new directory and mount the original on top of it - mkdir_attr(dest, 0755, 0, 0); - - if (mount(full_path, dest, NULL, MS_BIND|MS_REC, NULL) < 0 || - mount(NULL, dest, NULL, MS_BIND|MS_REMOUNT|MS_RDONLY|MS_NOSUID|MS_NODEV|MS_REC, NULL) < 0) - errExit("mount bind"); - fs_logger2("clone", full_path); - fs_logger2("mount", full_path); - dir_cnt++; - free(dest); -} - // fname should be a vallid full path at this point static void load_library(const char *fname) { assert(fname); assert(*fname == '/'); - // existing file owned by root, read access + // existing file owned by root struct stat s; - if (stat(fname, &s) == 0 && s.st_uid == 0 && !access(fname, R_OK)) { + if (!access(fname, F_OK) && stat(fname, &s) == 0 && s.st_uid == 0) { // load directories, regular 64 bit libraries, and 64 bit executables - if (is_dir(fname) || is_lib_64(fname)) { - if (is_dir(fname)) - fslib_copy_dir(fname); + if (S_ISDIR(s.st_mode) || is_lib_64(fname)) { + if (S_ISDIR(s.st_mode)) + fslib_mount(fname); else { if (strstr(fname, ".so") || access(fname, X_OK) != 0) // don't duplicate executables, just install the libraries - fslib_duplicate(fname); + fslib_mount(fname); - fslib_copy_libs_parse_as_user(fname); + fslib_mount_libs(fname, 1); // parse as user } } } @@ -338,7 +308,6 @@ static void install_list_entry(const char *lib) { return; } - void fslib_install_list(const char *lib_list) { assert(lib_list); if (arg_debug || arg_debug_private_lib) @@ -444,7 +413,6 @@ void fs_private_lib(void) { fslib_install_list(cfg.shell); // a shell is useless without some basic commands fslib_install_list("/bin/ls,/bin/cat,/bin/mv,/bin/rm"); - } // for the listed libs and directories diff --git a/src/firejail/fs_lib2.c b/src/firejail/fs_lib2.c index d46cfed8614..0e99801b32c 100644 --- a/src/firejail/fs_lib2.c +++ b/src/firejail/fs_lib2.c @@ -21,10 +21,8 @@ #include #include -extern void fslib_duplicate(const char *full_path); -extern void fslib_copy_libs_parse_as_user(const char *full_path); -extern void fslib_copy_libs_parse_as_root(const char *full_path); -extern void fslib_copy_dir(const char *full_path); +extern void fslib_mount_libs(const char *full_path, unsigned user); +extern void fslib_mount(const char *full_path); //*************************************************************** // Standard C library @@ -98,7 +96,7 @@ static void stdc(const char *dirname) { if (asprintf(&fname, "%s/%s", dirname, entry->d_name) == -1) errExit("asprintf"); - fslib_duplicate(fname); + fslib_mount(fname); } } closedir(dir); @@ -119,7 +117,7 @@ void fslib_install_stdc(void) { // install locale if (stat("/usr/lib/locale", &s) == 0) - fslib_copy_dir("/usr/lib/locale"); + fslib_mount("/usr/lib/locale"); fmessage("Standard C library installed in %0.2f ms\n", timetrace_end()); } @@ -129,7 +127,8 @@ void fslib_install_stdc(void) { //*************************************************************** static void fdir(void) { - fslib_copy_dir(LIBDIR "/firejail"); + // firejail directory itself + fslib_mount(LIBDIR "/firejail"); // executables and libraries from firejail directory static const char * const fbin[] = { @@ -143,30 +142,28 @@ static void fdir(void) { NULL, }; - // need to run fldd as root user, unprivileged users have no read permission on executables + // need to parse as root user, unprivileged users have no read permission on executables int i; for (i = 0; fbin[i]; i++) - fslib_copy_libs_parse_as_root(fbin[i]); + fslib_mount_libs(fbin[i], 0); } void fslib_install_firejail(void) { timetrace_start(); // bring in firejail executable libraries, in case we are redirected here // by a firejail symlink from /usr/local/bin/firejail - fslib_copy_libs_parse_as_user(PATH_FIREJAIL); + fslib_mount_libs(PATH_FIREJAIL, 1); // parse as user // bring in firejail directory fdir(); // bring in dhclient libraries if (any_dhcp()) - fslib_copy_libs_parse_as_user(RUN_MNT_DIR "/dhclient"); + fslib_mount_libs(RUN_MNT_DIR "/dhclient", 1); // parse as user -#ifdef HAVE_X11 // bring in xauth libraries if (arg_x11_xorg) - fslib_copy_libs_parse_as_user("/usr/bin/xauth"); -#endif + fslib_mount_libs("/usr/bin/xauth", 1); // parse as user fmessage("Firejail libraries installed in %0.2f ms\n", timetrace_end()); } @@ -315,8 +312,8 @@ void fslib_install_system(void) { if (asprintf(&name, "/usr/lib/x86_64-linux-gnu/%s", ptr->dir1) == -1) errExit("asprintf"); if (access(name, R_OK) == 0) { - fslib_copy_libs_parse_as_user(name); - fslib_copy_dir(name); + fslib_mount_libs(name, 1); // parse as user + fslib_mount(name); } else { free(name); @@ -324,8 +321,8 @@ void fslib_install_system(void) { if (asprintf(&name, "/usr/lib64/%s", ptr->dir1) == -1) errExit("asprintf"); if (access(name, R_OK) == 0) { - fslib_copy_libs_parse_as_user(name); - fslib_copy_dir(name); + fslib_mount_libs(name, 1); // parse as user + fslib_mount(name); } } free(name); @@ -335,8 +332,8 @@ void fslib_install_system(void) { if (asprintf(&name, "/usr/lib/x86_64-linux-gnu/%s", ptr->dir2) == -1) errExit("asprintf"); if (access(name, R_OK) == 0) { - fslib_copy_libs_parse_as_user(name); - fslib_copy_dir(name); + fslib_mount_libs(name, 1); // parse as user + fslib_mount(name); } else { free(name); @@ -344,8 +341,8 @@ void fslib_install_system(void) { if (asprintf(&name, "/usr/lib64/%s", ptr->dir2) == -1) errExit("asprintf"); if (access(name, R_OK) == 0) { - fslib_copy_libs_parse_as_user(name); - fslib_copy_dir(name); + fslib_mount_libs(name, 1); // parse as user + fslib_mount(name); } } free(name); diff --git a/src/lib/ldd_utils.c b/src/lib/ldd_utils.c index 43fee4f21e6..d059c9f5667 100644 --- a/src/lib/ldd_utils.c +++ b/src/lib/ldd_utils.c @@ -23,12 +23,14 @@ #include #include +// todo: resolve overlap with lib_dirs[] array from fs_lib.c const char * const default_lib_paths[] = { "/usr/lib/x86_64-linux-gnu", // Debian & friends "/lib/x86_64-linux-gnu", // CentOS, Fedora + "/usr/lib64", + "/lib64", "/usr/lib", "/lib", - "/lib64", LIBDIR, "/usr/local/lib64", "/usr/local/lib",