X-Git-Url: http://git.annexia.org/?p=libguestfs.git;a=blobdiff_plain;f=src%2Finspect_fs_unix.c;fp=src%2Finspect_fs_unix.c;h=b68dde1ef6a6d80f57ce8a2fd2a23d7faf074ef5;hp=340fe6afe85e3d891b753bb0afd3709ed467295c;hb=35c8e4fa7f5c90cc8371028e2bfc0562272380e3;hpb=7c89d9ef6b37314fb55aabd0c389a9d652acdb0f diff --git a/src/inspect_fs_unix.c b/src/inspect_fs_unix.c index 340fe6a..b68dde1 100644 --- a/src/inspect_fs_unix.c +++ b/src/inspect_fs_unix.c @@ -107,7 +107,6 @@ compile_regexps (void) COMPILE (re_scientific_linux_no_minor, "Scientific Linux.*release (\\d+)", 0); COMPILE (re_major_minor, "(\\d+)\\.(\\d+)", 0); - COMPILE (re_aug_seq, "/\\d+$", 0); COMPILE (re_xdev, "^/dev/(h|s|v|xv)d([a-z]+)(\\d*)$", 0); COMPILE (re_cciss, "^/dev/(cciss/c\\d+d\\d+)(?:p(\\d+))?$", 0); COMPILE (re_freebsd, "^/dev/ad(\\d+)s(\\d+)([a-z])$", 0); @@ -128,7 +127,6 @@ free_regexps (void) pcre_free (re_scientific_linux); pcre_free (re_scientific_linux_no_minor); pcre_free (re_major_minor); - pcre_free (re_aug_seq); pcre_free (re_xdev); pcre_free (re_cciss); pcre_free (re_freebsd); @@ -724,45 +722,43 @@ check_hostname_freebsd (guestfs_h *g, struct inspect_fs *fs) static int check_fstab (guestfs_h *g, struct inspect_fs *fs) { - char **lines = guestfs_aug_ls (g, "/files/etc/fstab"); - if (lines == NULL) goto error; + char **entries, **entry; + char augpath[256]; + char *spec, *mp; + int r; + + entries = guestfs_aug_match (g, "/files/etc/fstab/*[label() != '#comment']"); + if (entries == NULL) goto error; - if (lines[0] == NULL) { + if (entries[0] == NULL) { error (g, _("could not parse /etc/fstab or empty file")); goto error; } - size_t i; - char augpath[256]; - for (i = 0; lines[i] != NULL; ++i) { - /* Ignore comments. Only care about sequence lines which - * match m{/\d+$}. - */ - if (match (g, lines[i], re_aug_seq)) { - snprintf (augpath, sizeof augpath, "%s/spec", lines[i]); - char *spec = guestfs_aug_get (g, augpath); - if (spec == NULL) goto error; - - snprintf (augpath, sizeof augpath, "%s/file", lines[i]); - char *mp = guestfs_aug_get (g, augpath); - if (mp == NULL) { - free (spec); - goto error; - } + for (entry = entries; *entry != NULL; entry++) { + snprintf (augpath, sizeof augpath, "%s/spec", *entry); + spec = guestfs_aug_get (g, augpath); + if (spec == NULL) goto error; - int r = add_fstab_entry (g, fs, spec, mp); + snprintf (augpath, sizeof augpath, "%s/file", *entry); + mp = guestfs_aug_get (g, augpath); + if (mp == NULL) { free (spec); - free (mp); - - if (r == -1) goto error; + goto error; } + + r = add_fstab_entry (g, fs, spec, mp); + free (spec); + free (mp); + + if (r == -1) goto error; } - guestfs___free_string_list (lines); + guestfs___free_string_list (entries); return 0; error: - if (lines) guestfs___free_string_list (lines); + if (entries) guestfs___free_string_list (entries); return -1; }