fstab-generator: fix trivial leak

[elogind.git] / src / fstab-generator / fstab-generator.c

diff --git a/src/fstab-generator/fstab-generator.c b/src/fstab-generator/fstab-generator.c
index 34cd720517d7ea260d1330b0110534dc37fe0aea..418e8b171dbd5ea118625905c69d3196359c7593 100644 (file)
--- a/src/fstab-generator/fstab-generator.c
+++ b/src/fstab-generator/fstab-generator.c
@@ -35,6 +35,7 @@
 #include "fileio.h"
 #include "generator.h"
 #include "strv.h"
 #include "fileio.h"
 #include "generator.h"
 #include "strv.h"

+#include "virt.h"

 
 static const char *arg_dest = "/tmp";
 static bool arg_fstab_enabled = true;
 
 static const char *arg_dest = "/tmp";
 static bool arg_fstab_enabled = true;


@@ -43,6 +44,7 @@ static char *arg_root_fstype = NULL;
 static char *arg_root_options = NULL;
 static int arg_root_rw = -1;
 
 static char *arg_root_options = NULL;
 static int arg_root_rw = -1;
 

+

 static int mount_find_pri(struct mntent *me, int *ret) {
         char *end, *pri;
         unsigned long r;
 static int mount_find_pri(struct mntent *me, int *ret) {
         char *end, *pri;
         unsigned long r;


@@ -77,6 +79,11 @@ static int add_swap(const char *what, struct mntent *me) {
         assert(what);
         assert(me);
 
         assert(what);
         assert(me);
 

+        if (detect_container(NULL) > 0) {
+                log_info("Running in a container, ignoring fstab swap entry for %s.", what);
+                return 0;
+        }
+

         r = mount_find_pri(me, &pri);
         if (r < 0) {
                 log_error("Failed to parse priority");
         r = mount_find_pri(me, &pri);
         if (r < 0) {
                 log_error("Failed to parse priority");


@@ -122,6 +129,11 @@ static int add_swap(const char *what, struct mntent *me) {
                 return -errno;
         }
 
                 return -errno;
         }
 

+        /* use what as where, to have a nicer error message */
+        r = generator_write_timeouts(arg_dest, what, what, me->mnt_opts, NULL);
+        if (r < 0)
+                return r;
+

         if (!noauto) {
                 lnk = strjoin(arg_dest, "/" SPECIAL_SWAP_TARGET ".wants/", name, NULL);
                 if (!lnk)
         if (!noauto) {
                 lnk = strjoin(arg_dest, "/" SPECIAL_SWAP_TARGET ".wants/", name, NULL);
                 if (!lnk)


@@ -167,7 +179,8 @@ static int add_mount(
 
         _cleanup_free_ char
                 *name = NULL, *unit = NULL, *lnk = NULL,
 
         _cleanup_free_ char
                 *name = NULL, *unit = NULL, *lnk = NULL,

-                *automount_name = NULL, *automount_unit = NULL;
+                *automount_name = NULL, *automount_unit = NULL,
+                *filtered = NULL;

         _cleanup_fclose_ FILE *f = NULL;
         int r;
 
         _cleanup_fclose_ FILE *f = NULL;
         int r;
 


@@ -239,8 +252,12 @@ static int add_mount(
         if (!isempty(fstype) && !streq(fstype, "auto"))
                 fprintf(f, "Type=%s\n", fstype);
 
         if (!isempty(fstype) && !streq(fstype, "auto"))
                 fprintf(f, "Type=%s\n", fstype);
 

-        if (!isempty(opts) && !streq(opts, "defaults"))
-                fprintf(f, "Options=%s\n", opts);
+        r = generator_write_timeouts(arg_dest, what, where, opts, &filtered);
+        if (r < 0)
+                return r;
+
+        if (!isempty(filtered) && !streq(filtered, "defaults"))
+                fprintf(f, "Options=%s\n", filtered);

 
         fflush(f);
         if (ferror(f)) {
 
         fflush(f);
         if (ferror(f)) {


@@ -341,6 +358,11 @@ static int parse_fstab(bool initrd) {
                 if (!what)
                         return log_oom();
 
                 if (!what)
                         return log_oom();
 

+                if (detect_container(NULL) > 0 && is_device_path(what)) {
+                        log_info("Running in a container, ignoring fstab device entry for %s.", what);
+                        continue;
+                }
+

                 where = initrd ? strappend("/sysroot/", me->mnt_dir) : strdup(me->mnt_dir);
                 if (!where)
                         return log_oom();
                 where = initrd ? strappend("/sysroot/", me->mnt_dir) : strdup(me->mnt_dir);
                 if (!where)
                         return log_oom();


@@ -391,7 +413,8 @@ static int parse_fstab(bool initrd) {
 }
 
 static int add_root_mount(void) {
 }
 
 static int add_root_mount(void) {

-        _cleanup_free_ char *o = NULL, *what = NULL;
+        _cleanup_free_ char *what = NULL;
+        const char *opts;

 
         if (isempty(arg_root_what)) {
                 log_debug("Could not find a root= entry on the kernel commandline.");
 
         if (isempty(arg_root_what)) {
                 log_debug("Could not find a root= entry on the kernel commandline.");


@@ -405,23 +428,19 @@ static int add_root_mount(void) {
         }
 
         if (!arg_root_options)
         }
 
         if (!arg_root_options)

-                o = strdup(arg_root_rw > 0 ? "rw" : "ro");
-        else {
-                if (arg_root_rw >= 0 ||
-                    (!mount_test_option(arg_root_options, "ro") &&
-                     !mount_test_option(arg_root_options, "rw")))
-                        o = strjoin(arg_root_options, ",", arg_root_rw > 0 ? "rw" : "ro", NULL);
-                else
-                        o = strdup(arg_root_options);
-        }
-        if (!o)
-                return log_oom();
+                opts = arg_root_rw > 0 ? "rw" : "ro";
+        else if (arg_root_rw >= 0 ||
+                 (!mount_test_option(arg_root_options, "ro") &&
+                  !mount_test_option(arg_root_options, "rw")))
+                opts = strappenda3(arg_root_options, ",", arg_root_rw > 0 ? "rw" : "ro");
+        else
+                opts = arg_root_options;

 
         log_debug("Found entry what=%s where=/sysroot type=%s", what, strna(arg_root_fstype));
         return add_mount(what,
                          "/sysroot",
                          arg_root_fstype,
 
         log_debug("Found entry what=%s where=/sysroot type=%s", what, strna(arg_root_fstype));
         return add_mount(what,
                          "/sysroot",
                          arg_root_fstype,

-                         o,
+                         opts,

                          1,
                          false,
                          false,
                          1,
                          false,
                          false,


@@ -475,8 +494,6 @@ static int parse_proc_cmdline_item(const char *key, const char *value) {
                 arg_root_rw = true;
         else if (streq(key, "ro") && !value)
                 arg_root_rw = false;
                 arg_root_rw = true;
         else if (streq(key, "ro") && !value)
                 arg_root_rw = false;

-        else if (startswith(key, "fstab.") || startswith(key, "rd.fstab."))
-                log_warning("Unknown kernel switch %s. Ignoring.", key);

 
         return 0;
 }
 
         return 0;
 }


@@ -509,6 +526,8 @@ int main(int argc, char *argv[]) {
         if (arg_fstab_enabled) {
                 int k;
 
         if (arg_fstab_enabled) {
                 int k;
 

+                log_debug("Parsing /etc/fstab");
+

                 /* Parse the local /etc/fstab, possibly from the initrd */
                 k = parse_fstab(false);
                 if (k < 0)
                 /* Parse the local /etc/fstab, possibly from the initrd */
                 k = parse_fstab(false);
                 if (k < 0)


@@ -516,11 +535,15 @@ int main(int argc, char *argv[]) {
 
                 /* If running in the initrd also parse the /etc/fstab from the host */
                 if (in_initrd()) {
 
                 /* If running in the initrd also parse the /etc/fstab from the host */
                 if (in_initrd()) {

+                        log_debug("Parsing /sysroot/etc/fstab");
+

                         k = parse_fstab(true);
                         if (k < 0)
                                 r = k;
                 }
         }
 
                         k = parse_fstab(true);
                         if (k < 0)
                                 r = k;
                 }
         }
 

+        free(arg_root_what);
+

         return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
 }
         return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
 }