From: Eric Andersen Date: Mon, 18 Jul 2005 09:28:36 +0000 (-0000) Subject: Fixup device table based makedevs so it actually works X-Git-Tag: 1_1_0~916 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=e8614dbcd75915e6bf5badb8669efc04e62453f8;p=platform%2Fupstream%2Fbusybox.git Fixup device table based makedevs so it actually works --- diff --git a/include/usage.h b/include/usage.h index 017cb9c..831a9a7 100644 --- a/include/usage.h +++ b/include/usage.h @@ -1672,13 +1672,37 @@ #ifdef CONFIG_FEATURE_MAKEDEVS_TABLE #define makedevs_trivial_usage \ - "[-r rootdir] [device_table]" + "[-d device_table] rootdir" #define makedevs_full_usage \ - "Creates a batch of special files as specified in a device table\n" \ - "The device table has one line per device group, each group is of\n" \ - "the format\n" \ - "\ttype mode user group major minor start increment count\n" \ - "a '-' may be used for blank entries\n" + "Creates a range of special files as specified in a device table.\n" \ + "Device table entries take the form of:\n" \ + "type mode user group major minor start increment count\n\n" \ + " \n" \ + "Where name is the file name, type can be one of:\n" \ + " f A regular file\n" \ + " d Directory\n" \ + " c Character special device file\n" \ + " b Block special device file\n" \ + " p Fifo (named pipe)\n" \ + "uid is the user id for the target file, gid is the group id for the\n" \ + "target file. The rest of the entries (major, minor, etc) apply to\n" \ + "to device special files. A '-' may be used for blank entries.\n\n" \ +#define makedevs_example_usage \ + "For example:\n" \ + " \n" \ + "/dev d 755 0 0 - - - - -\n" \ + "/dev/console c 666 0 0 5 1 - - -\n" \ + "/dev/null c 666 0 0 1 3 0 0 -\n" \ + "/dev/zero c 666 0 0 1 5 0 0 -\n" \ + "/dev/hda b 640 0 0 3 0 0 0 -\n" \ + "/dev/hda b 640 0 0 3 1 1 1 15\n\n" \ + "Will Produce:\n" \ + "/dev\n" \ + "/dev/console\n" \ + "/dev/null\n" \ + "/dev/zero\n" \ + "/dev/hda\n" \ + "/dev/hda[0-15]" #endif #ifdef CONFIG_FEATURE_MD5_SHA1_SUM_CHECK diff --git a/miscutils/makedevs.c b/miscutils/makedevs.c index 2c9a0a9..eb3e368 100644 --- a/miscutils/makedevs.c +++ b/miscutils/makedevs.c @@ -91,35 +91,42 @@ int makedevs_main(int argc, char **argv) * */ -static const struct option makedevs_long_options[] = { - {"root", 1, NULL, 'r'}, - {0, 0, 0, 0} -}; - extern int makedevs_main(int argc, char **argv) { - FILE *table; int opt; - char *rootdir = "./"; - char *line; + FILE *table = stdin; + char *rootdir = NULL; + char *line = NULL; + int linenum = 0; int ret = EXIT_SUCCESS; - bb_opt_complementaly = "d~r"; - bb_applet_long_options = makedevs_long_options; - opt = bb_getopt_ulflags(argc, argv, "d:r:", &rootdir, &rootdir); + while ((opt = getopt(argc, argv, "d:")) != -1) { + switch(opt) { + case 'd': + table = bb_xfopen((line=optarg), "r"); + break; + default: + bb_show_usage(); + } + } - if (optind + 1 == argc) { - table = bb_xfopen(argv[optind], "r"); - } else { - table = stdin; + if (optind >= argc || (rootdir=argv[optind])==NULL) { + bb_error_msg_and_die("root directory not speficied"); } - if (chdir(rootdir) == -1) { - bb_perror_msg_and_die("Couldnt chdor to %s", rootdir); + if (chdir(rootdir) != 0) { + bb_perror_msg_and_die("Couldnt chdir to %s", rootdir); } umask(0); + printf("rootdir=%s\n", rootdir); + if (line) { + printf("table='%s'\n", line); + } else { + printf("table=\n"); + } + while ((line = bb_get_chomped_line_from_file(table))) { char type; unsigned int mode = 0755; @@ -135,11 +142,16 @@ extern int makedevs_main(int argc, char **argv) uid_t uid; gid_t gid; + linenum++; + if ((2 > sscanf(line, "%40s %c %o %40s %40s %u %u %u %u %u", name, - &type, &mode, user, group, &major, - &minor, &start, &increment, &count)) || - ((major | minor | start | count | increment) > 255)) { - bb_error_msg("Ignoring invalid line\n%s\n", line); + &type, &mode, user, group, &major, + &minor, &start, &increment, &count)) || + ((major | minor | start | count | increment) > 255)) + { + if (*line=='\0' || *line=='#' || isspace(*line)) + continue; + bb_error_msg("line %d invalid: '%s'\n", linenum, line); ret = EXIT_FAILURE; continue; } @@ -159,9 +171,9 @@ extern int makedevs_main(int argc, char **argv) full_name = concat_path_file(rootdir, name); if (type == 'd') { - bb_make_directory(full_name, mode | S_IFDIR, 0); + bb_make_directory(full_name, mode | S_IFDIR, FILEUTILS_RECUR); if (chown(full_name, uid, gid) == -1) { - bb_perror_msg("chown failed for %s", full_name); + bb_perror_msg("line %d: chown failed for %s", linenum, full_name); ret = EXIT_FAILURE; goto loop; } @@ -177,7 +189,7 @@ extern int makedevs_main(int argc, char **argv) else if (type == 'b') { mode |= S_IFBLK; } else { - bb_error_msg("Unsupported file type %c", type); + bb_error_msg("line %d: Unsupported file type %c", linenum, type); ret = EXIT_FAILURE; goto loop; } @@ -191,11 +203,11 @@ extern int makedevs_main(int argc, char **argv) sprintf(full_name_inc, "%s%d", full_name, i); rdev = (major << 8) + minor + (i * increment - start); if (mknod(full_name_inc, mode, rdev) == -1) { - bb_perror_msg("Couldnt create node %s", full_name_inc); + bb_perror_msg("line %d: Couldnt create node %s", linenum, full_name_inc); ret = EXIT_FAILURE; } else if (chown(full_name_inc, uid, gid) == -1) { - bb_perror_msg("chown failed for %s", full_name_inc); + bb_perror_msg("line %d: chown failed for %s", linenum, full_name_inc); ret = EXIT_FAILURE; } } @@ -203,11 +215,11 @@ extern int makedevs_main(int argc, char **argv) } else { rdev = (major << 8) + minor; if (mknod(full_name, mode, rdev) == -1) { - bb_perror_msg("Couldnt create node %s", full_name); + bb_perror_msg("line %d: Couldnt create node %s", linenum, full_name); ret = EXIT_FAILURE; } else if (chown(full_name, uid, gid) == -1) { - bb_perror_msg("chown failed for %s", full_name); + bb_perror_msg("line %d: chown failed for %s", linenum, full_name); ret = EXIT_FAILURE; } } @@ -220,6 +232,7 @@ loop: return 0; } + #else # error makdedevs configuration error, either leaf or table must be selected #endif