diff --git a/e2fsprogs/e2p/e2p.h b/e2fsprogs/e2p/e2p.h index 78930ec17..57bcec31f 100644 --- a/e2fsprogs/e2p/e2p.h +++ b/e2fsprogs/e2p/e2p.h @@ -1,3 +1,4 @@ +#include "busybox.h" #include /* Needed by dirent.h on netbsd */ #include #include @@ -30,8 +31,9 @@ int getversion (int fd, unsigned long * version); int iterate_on_dir (const char * dir_name, int (*func) (const char *, struct dirent *, void *), void * private); -void list_super(struct ext2_super_block * s); +/*void list_super(struct ext2_super_block * s);*/ void list_super2(struct ext2_super_block * s, FILE *f); +#define list_super(s) list_super2(s, stdout) void print_fs_errors (FILE * f, unsigned short errors); void print_flags (FILE * f, unsigned long flags, unsigned options); void print_fs_state (FILE * f, unsigned short state); diff --git a/e2fsprogs/e2p/feature.c b/e2fsprogs/e2p/feature.c index ef11d93b7..f4e920a98 100644 --- a/e2fsprogs/e2p/feature.c +++ b/e2fsprogs/e2p/feature.c @@ -56,10 +56,10 @@ static struct feature feature_list[] = { const char *e2p_feature2string(int compat, unsigned int mask) { - struct feature *f; + struct feature *f; static char buf[20]; - char fchar; - int fnum; + char fchar; + int fnum; for (f = feature_list; f->string; f++) { if ((compat == f->compat) && @@ -67,7 +67,7 @@ const char *e2p_feature2string(int compat, unsigned int mask) return f->string; } switch (compat) { - case E2P_FEATURE_COMPAT: + case E2P_FEATURE_COMPAT: fchar = 'C'; break; case E2P_FEATURE_INCOMPAT: @@ -81,15 +81,15 @@ const char *e2p_feature2string(int compat, unsigned int mask) break; } for (fnum = 0; mask >>= 1; fnum++); - sprintf(buf, "FEATURE_%c%d", fchar, fnum); + sprintf(buf, "FEATURE_%c%d", fchar, fnum); return buf; } int e2p_string2feature(char *string, int *compat_type, unsigned int *mask) { - struct feature *f; - char *eptr; - int num; + struct feature *f; + char *eptr; + int num; for (f = feature_list; f->string; f++) { if (!strcasecmp(string, f->string)) { @@ -128,14 +128,14 @@ int e2p_string2feature(char *string, int *compat_type, unsigned int *mask) return 0; } -static char *skip_over_blanks(char *cp) +static inline char *skip_over_blanks(char *cp) { while (*cp && isspace(*cp)) cp++; return cp; } -static char *skip_over_word(char *cp) +static inline char *skip_over_word(char *cp) { while (*cp && !isspace(*cp) && *cp != ',') cp++; @@ -187,4 +187,3 @@ int e2p_edit_feature(const char *str, __u32 *compat_array, __u32 *ok_array) } return 0; } - diff --git a/e2fsprogs/e2p/hashstr.c b/e2fsprogs/e2p/hashstr.c index b257eb26c..7c0552f09 100644 --- a/e2fsprogs/e2p/hashstr.c +++ b/e2fsprogs/e2p/hashstr.c @@ -17,20 +17,20 @@ #include "e2p.h" struct hash { - int num; - const char *string; + int num; + const char *string; }; static struct hash hash_list[] = { - { EXT2_HASH_LEGACY, "legacy" }, - { EXT2_HASH_HALF_MD4, "half_md4" }, - { EXT2_HASH_TEA, "tea" }, - { 0, 0 }, + { EXT2_HASH_LEGACY, "legacy" }, + { EXT2_HASH_HALF_MD4, "half_md4" }, + { EXT2_HASH_TEA, "tea" }, + { 0, 0 }, }; const char *e2p_hash2string(int num) { - struct hash *p; + struct hash *p; static char buf[20]; for (p = hash_list; p->string; p++) { @@ -46,9 +46,9 @@ const char *e2p_hash2string(int num) */ int e2p_string2hash(char *string) { - struct hash *p; - char *eptr; - int num; + struct hash *p; + char *eptr; + int num; for (p = hash_list; p->string; p++) { if (!strcasecmp(string, p->string)) { @@ -67,4 +67,3 @@ int e2p_string2hash(char *string) return -1; return num; } - diff --git a/e2fsprogs/e2p/ls.c b/e2fsprogs/e2p/ls.c index e8d9d482d..71f8f5126 100644 --- a/e2fsprogs/e2p/ls.c +++ b/e2fsprogs/e2p/ls.c @@ -20,28 +20,18 @@ #include "e2p.h" -static void print_user (unsigned short uid, FILE *f) +static void print_user(unsigned short uid, FILE *f) { - struct passwd *pw; - - fprintf(f, "%u ", uid); - pw = getpwuid (uid); - if (pw == NULL) - fprintf(f, "(user unknown)\n"); - else - fprintf(f, "(user %s)\n", pw->pw_name); + struct passwd *pw = getpwuid(uid); + fprintf(f, "%u (user %s)\n", uid, + (pw == NULL ? "unknown" : pw->pw_name)); } -static void print_group (unsigned short gid, FILE *f) +static void print_group(unsigned short gid, FILE *f) { - struct group *gr; - - fprintf(f, "%u ", gid); - gr = getgrgid (gid); - if (gr == NULL) - fprintf(f, "(group unknown)\n"); - else - fprintf(f, "(group %s)\n", gr->gr_name); + struct group *gr = getgrgid(gid); + fprintf(f, "%u (group %s)\n", gid, + (gr == NULL ? "unknown" : gr->gr_name)); } #define MONTH_INT (86400 * 30) @@ -167,10 +157,12 @@ void list_super2(struct ext2_super_block * sb, FILE *f) strncpy(buf, sb->s_last_mounted, sizeof(sb->s_last_mounted)); } else strcpy(buf, ""); - fprintf(f, "Last mounted on: %s\n", buf); - fprintf(f, "Filesystem UUID: %s\n", e2p_uuid2str(sb->s_uuid)); - fprintf(f, "Filesystem magic number: 0x%04X\n", sb->s_magic); - fprintf(f, "Filesystem revision #: %d", sb->s_rev_level); + fprintf(f, + "Last mounted on: %s\n" + "Filesystem UUID: %s\n" + "Filesystem magic number: 0x%04X\n" + "Filesystem revision #: %d", + buf, e2p_uuid2str(sb->s_uuid), sb->s_magic, sb->s_rev_level); if (sb->s_rev_level == EXT2_GOOD_OLD_REV) { fprintf(f, " (original)\n"); #ifdef EXT2_DYNAMIC_REV @@ -183,28 +175,34 @@ void list_super2(struct ext2_super_block * sb, FILE *f) print_mntopts(sb, f); fprintf(f, "Filesystem state: "); print_fs_state (f, sb->s_state); - fprintf(f, "\n"); - fprintf(f, "Errors behavior: "); + fprintf(f, "\nErrors behavior: "); print_fs_errors(f, sb->s_errors); - fprintf(f, "\n"); str = e2p_os2string(sb->s_creator_os); - fprintf(f, "Filesystem OS type: %s\n", str); + fprintf(f, + "\n" + "Filesystem OS type: %s\n" + "Inode count: %u\n" + "Block count: %u\n" + "Reserved block count: %u\n" + "Free blocks: %u\n" + "Free inodes: %u\n" + "First block: %u\n" + "Block size: %u\n" + "Fragment size: %u\n", + str, sb->s_inodes_count, sb->s_blocks_count, sb->s_r_blocks_count, + sb->s_free_blocks_count, sb->s_free_inodes_count, + sb->s_first_data_block, EXT2_BLOCK_SIZE(sb), EXT2_FRAG_SIZE(sb)); free(str); - fprintf(f, "Inode count: %u\n", sb->s_inodes_count); - fprintf(f, "Block count: %u\n", sb->s_blocks_count); - fprintf(f, "Reserved block count: %u\n", sb->s_r_blocks_count); - fprintf(f, "Free blocks: %u\n", sb->s_free_blocks_count); - fprintf(f, "Free inodes: %u\n", sb->s_free_inodes_count); - fprintf(f, "First block: %u\n", sb->s_first_data_block); - fprintf(f, "Block size: %u\n", EXT2_BLOCK_SIZE(sb)); - fprintf(f, "Fragment size: %u\n", EXT2_FRAG_SIZE(sb)); if (sb->s_reserved_gdt_blocks) fprintf(f, "Reserved GDT blocks: %u\n", sb->s_reserved_gdt_blocks); - fprintf(f, "Blocks per group: %u\n", sb->s_blocks_per_group); - fprintf(f, "Fragments per group: %u\n", sb->s_frags_per_group); - fprintf(f, "Inodes per group: %u\n", sb->s_inodes_per_group); - fprintf(f, "Inode blocks per group: %u\n", inode_blocks_per_group); + fprintf(f, + "Blocks per group: %u\n" + "Fragments per group: %u\n" + "Inodes per group: %u\n" + "Inode blocks per group: %u\n", + sb->s_blocks_per_group, sb->s_frags_per_group, + sb->s_inodes_per_group, inode_blocks_per_group); if (sb->s_first_meta_bg) fprintf(f, "First meta block group: %u\n", sb->s_first_meta_bg); @@ -216,13 +214,17 @@ void list_super2(struct ext2_super_block * sb, FILE *f) fprintf(f, "Last mount time: %s", sb->s_mtime ? ctime(&tm) : "n/a\n"); tm = sb->s_wtime; - fprintf(f, "Last write time: %s", ctime(&tm)); - fprintf(f, "Mount count: %u\n", sb->s_mnt_count); - fprintf(f, "Maximum mount count: %d\n", sb->s_max_mnt_count); + fprintf(f, + "Last write time: %s" + "Mount count: %u\n" + "Maximum mount count: %d\n", + ctime(&tm), sb->s_mnt_count, sb->s_max_mnt_count); tm = sb->s_lastcheck; - fprintf(f, "Last checked: %s", ctime(&tm)); - fprintf(f, "Check interval: %u (%s)\n", sb->s_checkinterval, - interval_string(sb->s_checkinterval)); + fprintf(f, + "Last checked: %s" + "Check interval: %u (%s)\n", + ctime(&tm), + sb->s_checkinterval, interval_string(sb->s_checkinterval)); if (sb->s_checkinterval) { time_t next; @@ -235,8 +237,10 @@ void list_super2(struct ext2_super_block * sb, FILE *f) fprintf(f, "Reserved blocks gid: "); print_group(sb->s_def_resgid, f); if (sb->s_rev_level >= EXT2_DYNAMIC_REV) { - fprintf(f, "First inode: %d\n", sb->s_first_ino); - fprintf(f, "Inode size: %d\n", sb->s_inode_size); + fprintf(f, + "First inode: %d\n" + "Inode size: %d\n", + sb->s_first_ino, sb->s_inode_size); } if (!e2p_is_null_uuid(sb->s_journal_uuid)) fprintf(f, "Journal UUID: %s\n", @@ -259,18 +263,9 @@ void list_super2(struct ext2_super_block * sb, FILE *f) e2p_uuid2str(sb->s_hash_seed)); if (sb->s_jnl_backup_type) { fprintf(f, "Journal backup: "); - switch (sb->s_jnl_backup_type) { - case 1: + if (sb->s_jnl_backup_type == 1) fprintf(f, "inode blocks\n"); - break; - default: + else fprintf(f, "type %u\n", sb->s_jnl_backup_type); - } } } - -void list_super (struct ext2_super_block * s) -{ - list_super2(s, stdout); -} - diff --git a/e2fsprogs/e2p/ostype.c b/e2fsprogs/e2p/ostype.c index fe6597dd5..a079b57b1 100644 --- a/e2fsprogs/e2p/ostype.c +++ b/e2fsprogs/e2p/ostype.c @@ -23,17 +23,17 @@ const char *os_tab[] = */ char *e2p_os2string(int os_type) { - const char *os; - char *ret; + const char *os; + char *ret; if (os_type <= EXT2_OS_LITES) os = os_tab[os_type]; else os = "(unknown os)"; - ret = malloc(strlen(os)+1); - strcpy(ret, os); - return ret; + ret = xmalloc(strlen(os)+1); + strcpy(ret, os); + return ret; } /* @@ -41,8 +41,8 @@ char *e2p_os2string(int os_type) */ int e2p_string2os(char *str) { - const char **cpp; - int i = 0; + const char **cpp; + int i = 0; for (cpp = os_tab; *cpp; cpp++, i++) { if (!strcasecmp(str, *cpp)) diff --git a/e2fsprogs/e2p/pe.c b/e2fsprogs/e2p/pe.c index 4cce69117..07fb88694 100644 --- a/e2fsprogs/e2p/pe.c +++ b/e2fsprogs/e2p/pe.c @@ -18,20 +18,14 @@ #include "e2p.h" -void print_fs_errors (FILE * f, unsigned short errors) +void print_fs_errors(FILE *f, unsigned short errors) { - switch (errors) - { - case EXT2_ERRORS_CONTINUE: - fprintf (f, "Continue"); - break; - case EXT2_ERRORS_RO: - fprintf (f, "Remount read-only"); - break; - case EXT2_ERRORS_PANIC: - fprintf (f, "Panic"); - break; - default: - fprintf (f, "Unknown (continue)"); + char *disp = NULL; + switch (errors) { + case EXT2_ERRORS_CONTINUE: disp = "Continue"; break; + case EXT2_ERRORS_RO: disp = "Remount read-only"; break; + case EXT2_ERRORS_PANIC: disp = "Panic"; break; + default: disp = "Unknown (continue)"; } + fprintf(f, disp); } diff --git a/e2fsprogs/e2p/pf.c b/e2fsprogs/e2p/pf.c index 48d75a9ae..d1a1c4caa 100644 --- a/e2fsprogs/e2p/pf.c +++ b/e2fsprogs/e2p/pf.c @@ -71,4 +71,3 @@ void print_flags (FILE * f, unsigned long flags, unsigned options) if (long_opt && first) fputs("---", f); } - diff --git a/e2fsprogs/e2p/ps.c b/e2fsprogs/e2p/ps.c index bec8b4195..b7c0ee2f1 100644 --- a/e2fsprogs/e2p/ps.c +++ b/e2fsprogs/e2p/ps.c @@ -18,12 +18,9 @@ #include "e2p.h" -void print_fs_state (FILE * f, unsigned short state) +void print_fs_state(FILE *f, unsigned short state) { - if (state & EXT2_VALID_FS) - fprintf (f, " clean"); - else - fprintf (f, " not clean"); + fprintf(f, (state & EXT2_VALID_FS ? " clean" : " not clean")); if (state & EXT2_ERROR_FS) fprintf (f, " with errors"); } diff --git a/e2fsprogs/e2p/uuid.c b/e2fsprogs/e2p/uuid.c index fef3b91e7..c16a76842 100644 --- a/e2fsprogs/e2p/uuid.c +++ b/e2fsprogs/e2p/uuid.c @@ -70,10 +70,8 @@ void e2p_uuid_to_str(void *uu, char *out) const char *e2p_uuid2str(void *uu) { static char buf[80]; - if (e2p_is_null_uuid(uu)) return ""; e2p_uuid_to_str(uu, buf); return buf; } -