Cleanup patch from Vladimir N. Oleynik.

* mkdir: remove 3 lines in source code.
    * mkfs_minix: save 32 bytes, remove 4 bugs.
    * mkswap: save 64 bytes, remove 1 bug.
This commit is contained in:
Eric Andersen 2001-01-31 17:35:02 +00:00
parent 53cfb7e231
commit 65225df2dc
6 changed files with 76 additions and 120 deletions

View File

@ -52,8 +52,7 @@ extern int mkdir_main(int argc, char **argv)
/* Find the specified modes */ /* Find the specified modes */
mode = 0; mode = 0;
if (parse_mode(*(++argv), &mode) == FALSE) { if (parse_mode(*(++argv), &mode) == FALSE) {
error_msg("Unknown mode: %s\n", *argv); error_msg_and_die("Unknown mode: %s\n", *argv);
return EXIT_FAILURE;
} }
/* Set the umask for this process so it doesn't /* Set the umask for this process so it doesn't
* screw up whatever the user just entered. */ * screw up whatever the user just entered. */
@ -81,22 +80,19 @@ extern int mkdir_main(int argc, char **argv)
char buf[BUFSIZ + 1]; char buf[BUFSIZ + 1];
if (strlen(*argv) > BUFSIZ - 1) { if (strlen(*argv) > BUFSIZ - 1) {
error_msg(name_too_long); error_msg_and_die(name_too_long);
return EXIT_FAILURE;
} }
strcpy(buf, *argv); strcpy(buf, *argv);
status = stat(buf, &statBuf); status = stat(buf, &statBuf);
if (parentFlag == FALSE && status != -1 && errno != ENOENT) { if (parentFlag == FALSE && status != -1 && errno != ENOENT) {
error_msg("%s: File exists\n", buf); error_msg_and_die("%s: File exists\n", buf);
return EXIT_FAILURE;
} }
if (parentFlag == TRUE) { if (parentFlag == TRUE) {
strcat(buf, "/"); strcat(buf, "/");
create_path(buf, mode); create_path(buf, mode);
} else { } else {
if (mkdir(buf, mode) != 0 && parentFlag == FALSE) { if (mkdir(buf, mode) != 0 && parentFlag == FALSE) {
perror(buf); perror_msg_and_die(buf);
return EXIT_FAILURE;
} }
} }
argc--; argc--;

12
mkdir.c
View File

@ -52,8 +52,7 @@ extern int mkdir_main(int argc, char **argv)
/* Find the specified modes */ /* Find the specified modes */
mode = 0; mode = 0;
if (parse_mode(*(++argv), &mode) == FALSE) { if (parse_mode(*(++argv), &mode) == FALSE) {
error_msg("Unknown mode: %s\n", *argv); error_msg_and_die("Unknown mode: %s\n", *argv);
return EXIT_FAILURE;
} }
/* Set the umask for this process so it doesn't /* Set the umask for this process so it doesn't
* screw up whatever the user just entered. */ * screw up whatever the user just entered. */
@ -81,22 +80,19 @@ extern int mkdir_main(int argc, char **argv)
char buf[BUFSIZ + 1]; char buf[BUFSIZ + 1];
if (strlen(*argv) > BUFSIZ - 1) { if (strlen(*argv) > BUFSIZ - 1) {
error_msg(name_too_long); error_msg_and_die(name_too_long);
return EXIT_FAILURE;
} }
strcpy(buf, *argv); strcpy(buf, *argv);
status = stat(buf, &statBuf); status = stat(buf, &statBuf);
if (parentFlag == FALSE && status != -1 && errno != ENOENT) { if (parentFlag == FALSE && status != -1 && errno != ENOENT) {
error_msg("%s: File exists\n", buf); error_msg_and_die("%s: File exists\n", buf);
return EXIT_FAILURE;
} }
if (parentFlag == TRUE) { if (parentFlag == TRUE) {
strcat(buf, "/"); strcat(buf, "/");
create_path(buf, mode); create_path(buf, mode);
} else { } else {
if (mkdir(buf, mode) != 0 && parentFlag == FALSE) { if (mkdir(buf, mode) != 0 && parentFlag == FALSE) {
perror(buf); perror_msg_and_die(buf);
return EXIT_FAILURE;
} }
} }
argc--; argc--;

View File

@ -255,19 +255,7 @@ static inline int bit(char * a,unsigned int i)
#define mark_zone(x) (setbit(zone_map,(x)-FIRSTZONE+1)) #define mark_zone(x) (setbit(zone_map,(x)-FIRSTZONE+1))
#define unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1)) #define unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1))
/* static __attribute__ ((noreturn)) void show_usage()
* Volatile to let gcc know that this doesn't return. When trying
* to compile this under minix, volatile gives a warning, as
* exit() isn't defined as volatile under minix.
*/
static volatile void die(char *str)
{
error_msg("%s\n", str);
exit(8);
}
static volatile void show_usage() __attribute__ ((noreturn));
static volatile void show_usage()
{ {
usage(mkfs_minix_usage); usage(mkfs_minix_usage);
} }
@ -291,7 +279,7 @@ static void check_mount(void)
if (!mnt) if (!mnt)
return; return;
die("%s is mounted; will not make a filesystem here!"); error_msg_and_die("%s is mounted; will not make a filesystem here!\n", device_name);
} }
static long valid_offset(int fd, int offset) static long valid_offset(int fd, int offset)
@ -348,28 +336,28 @@ static void write_tables(void)
Super.s_state &= ~MINIX_ERROR_FS; Super.s_state &= ~MINIX_ERROR_FS;
if (lseek(DEV, 0, SEEK_SET)) if (lseek(DEV, 0, SEEK_SET))
die("seek to boot block failed in write_tables"); error_msg_and_die("seek to boot block failed in write_tables\n");
if (512 != write(DEV, boot_block_buffer, 512)) if (512 != write(DEV, boot_block_buffer, 512))
die("unable to clear boot sector"); error_msg_and_die("unable to clear boot sector\n");
if (BLOCK_SIZE != lseek(DEV, BLOCK_SIZE, SEEK_SET)) if (BLOCK_SIZE != lseek(DEV, BLOCK_SIZE, SEEK_SET))
die("seek failed in write_tables"); error_msg_and_die("seek failed in write_tables\n");
if (BLOCK_SIZE != write(DEV, super_block_buffer, BLOCK_SIZE)) if (BLOCK_SIZE != write(DEV, super_block_buffer, BLOCK_SIZE))
die("unable to write super-block"); error_msg_and_die("unable to write super-block\n");
if (IMAPS * BLOCK_SIZE != write(DEV, inode_map, IMAPS * BLOCK_SIZE)) if (IMAPS * BLOCK_SIZE != write(DEV, inode_map, IMAPS * BLOCK_SIZE))
die("unable to write inode map"); error_msg_and_die("unable to write inode map\n");
if (ZMAPS * BLOCK_SIZE != write(DEV, zone_map, ZMAPS * BLOCK_SIZE)) if (ZMAPS * BLOCK_SIZE != write(DEV, zone_map, ZMAPS * BLOCK_SIZE))
die("unable to write zone map"); error_msg_and_die("unable to write zone map\n");
if (INODE_BUFFER_SIZE != write(DEV, inode_buffer, INODE_BUFFER_SIZE)) if (INODE_BUFFER_SIZE != write(DEV, inode_buffer, INODE_BUFFER_SIZE))
die("unable to write inodes"); error_msg_and_die("unable to write inodes\n");
} }
static void write_block(int blk, char *buffer) static void write_block(int blk, char *buffer)
{ {
if (blk * BLOCK_SIZE != lseek(DEV, blk * BLOCK_SIZE, SEEK_SET)) if (blk * BLOCK_SIZE != lseek(DEV, blk * BLOCK_SIZE, SEEK_SET))
die("seek failed in write_block"); error_msg_and_die("seek failed in write_block\n");
if (BLOCK_SIZE != write(DEV, buffer, BLOCK_SIZE)) if (BLOCK_SIZE != write(DEV, buffer, BLOCK_SIZE))
die("write failed in write_block"); error_msg_and_die("write failed in write_block\n");
} }
static int get_free_block(void) static int get_free_block(void)
@ -377,7 +365,7 @@ static int get_free_block(void)
int blk; int blk;
if (used_good_blocks + 1 >= MAX_GOOD_BLOCKS) if (used_good_blocks + 1 >= MAX_GOOD_BLOCKS)
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
if (used_good_blocks) if (used_good_blocks)
blk = good_blocks_table[used_good_blocks - 1] + 1; blk = good_blocks_table[used_good_blocks - 1] + 1;
else else
@ -385,7 +373,7 @@ static int get_free_block(void)
while (blk < ZONES && zone_in_use(blk)) while (blk < ZONES && zone_in_use(blk))
blk++; blk++;
if (blk >= ZONES) if (blk >= ZONES)
die("not enough good blocks"); error_msg_and_die("not enough good blocks\n");
good_blocks_table[used_good_blocks] = blk; good_blocks_table[used_good_blocks] = blk;
used_good_blocks++; used_good_blocks++;
return blk; return blk;
@ -451,7 +439,7 @@ static void make_bad_inode(void)
goto end_bad; goto end_bad;
} }
} }
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
end_bad: end_bad:
if (ind) if (ind)
write_block(ind, (char *) ind_block); write_block(ind, (char *) ind_block);
@ -501,7 +489,7 @@ static void make_bad_inode2(void)
} }
} }
/* Could make triple indirect block here */ /* Could make triple indirect block here */
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
end_bad: end_bad:
if (ind) if (ind)
write_block(ind, (char *) ind_block); write_block(ind, (char *) ind_block);
@ -602,7 +590,7 @@ static void setup_tables(void)
* /sbin/mkfs.minix -i 200 test.fs * /sbin/mkfs.minix -i 200 test.fs
* */ * */
if (i >= 999) { if (i >= 999) {
die("unable to allocate buffers for maps"); error_msg_and_die("unable to allocate buffers for maps\n");
} }
FIRSTZONE = NORM_FIRSTZONE; FIRSTZONE = NORM_FIRSTZONE;
inode_map = xmalloc(IMAPS * BLOCK_SIZE); inode_map = xmalloc(IMAPS * BLOCK_SIZE);
@ -633,7 +621,7 @@ long do_check(char *buffer, int try, unsigned int current_block)
/* Seek to the correct loc. */ /* Seek to the correct loc. */
if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) != if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) !=
current_block * BLOCK_SIZE) { current_block * BLOCK_SIZE) {
die("seek failed during testing of blocks"); error_msg_and_die("seek failed during testing of blocks\n");
} }
@ -673,7 +661,7 @@ static void check_blocks(void)
while (currently_testing < ZONES) { while (currently_testing < ZONES) {
if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) != if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) !=
currently_testing * BLOCK_SIZE) currently_testing * BLOCK_SIZE)
die("seek failed in check_blocks"); error_msg_and_die("seek failed in check_blocks\n");
try = TEST_BUFFER_BLOCKS; try = TEST_BUFFER_BLOCKS;
if (currently_testing + try > ZONES) if (currently_testing + try > ZONES)
try = ZONES - currently_testing; try = ZONES - currently_testing;
@ -682,7 +670,7 @@ static void check_blocks(void)
if (got == try) if (got == try)
continue; continue;
if (currently_testing < FIRSTZONE) if (currently_testing < FIRSTZONE)
die("bad blocks before data-area: cannot make fs"); error_msg_and_die("bad blocks before data-area: cannot make fs\n");
mark_zone(currently_testing); mark_zone(currently_testing);
badblocks++; badblocks++;
currently_testing++; currently_testing++;
@ -702,7 +690,7 @@ char *filename;
listfile = fopen(filename, "r"); listfile = fopen(filename, "r");
if (listfile == (FILE *) NULL) { if (listfile == (FILE *) NULL) {
die("can't open file of bad blocks"); error_msg_and_die("can't open file of bad blocks\n");
} }
while (!feof(listfile)) { while (!feof(listfile)) {
fscanf(listfile, "%ld\n", &blockno); fscanf(listfile, "%ld\n", &blockno);
@ -724,10 +712,10 @@ extern int mkfs_minix_main(int argc, char **argv)
int stopIt=FALSE; int stopIt=FALSE;
if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE) if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE)
die("bad inode size"); error_msg_and_die("bad inode size\n");
#ifdef BB_FEATURE_MINIX2 #ifdef BB_FEATURE_MINIX2
if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE) if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE)
die("bad inode size"); error_msg_and_die("bad inode size\n");
#endif #endif
/* Parse options */ /* Parse options */
@ -844,13 +832,13 @@ goodbye:
strcpy(tmp + 2, ".badblocks"); strcpy(tmp + 2, ".badblocks");
DEV = open(device_name, O_RDWR); DEV = open(device_name, O_RDWR);
if (DEV < 0) if (DEV < 0)
die("unable to open %s"); error_msg_and_die("unable to open %s\n", device_name);
if (fstat(DEV, &statbuf) < 0) if (fstat(DEV, &statbuf) < 0)
die("unable to stat %s"); error_msg_and_die("unable to stat %s\n", device_name);
if (!S_ISBLK(statbuf.st_mode)) if (!S_ISBLK(statbuf.st_mode))
check = 0; check = 0;
else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340) else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340)
die("will not try to make filesystem on '%s'"); error_msg_and_die("will not try to make filesystem on '%s'\n", device_name);
setup_tables(); setup_tables();
if (check) if (check)
check_blocks(); check_blocks();

View File

@ -173,12 +173,6 @@ static int bit_test_and_clear(unsigned int *addr, unsigned int nr)
} }
static void die(const char *str)
{
error_msg("%s\n", str);
exit(EXIT_FAILURE);
}
static void page_ok(int page) static void page_ok(int page)
{ {
if (version == 0) if (version == 0)
@ -191,7 +185,7 @@ static void page_bad(int page)
bit_test_and_clear(signature_page, page); bit_test_and_clear(signature_page, page);
else { else {
if (badpages == MAX_BADPAGES) if (badpages == MAX_BADPAGES)
die("too many bad pages"); error_msg_and_die("too many bad pages\n");
p->badpages[badpages] = page; p->badpages[badpages] = page;
} }
badpages++; badpages++;
@ -212,7 +206,7 @@ static void check_blocks(void)
} }
if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) != if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) !=
current_page * pagesize) current_page * pagesize)
die("seek failed in check_blocks"); error_msg_and_die("seek failed in check_blocks\n");
if ((do_seek = (pagesize != read(DEV, buffer, pagesize)))) { if ((do_seek = (pagesize != read(DEV, buffer, pagesize)))) {
page_bad(current_page++); page_bad(current_page++);
continue; continue;
@ -369,7 +363,7 @@ int mkswap_main(int argc, char **argv)
if (!S_ISBLK(statbuf.st_mode)) if (!S_ISBLK(statbuf.st_mode))
check = 0; check = 0;
else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340) else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340)
die("Will not try to make swapdevice on '%s'"); error_msg_and_die("Will not try to make swapdevice on '%s'\n", device_name);
#ifdef __sparc__ #ifdef __sparc__
if (!force && version == 0) { if (!force && version == 0) {
@ -378,7 +372,7 @@ int mkswap_main(int argc, char **argv)
unsigned short *q, sum; unsigned short *q, sum;
if (read(DEV, buffer, 512) != 512) if (read(DEV, buffer, 512) != 512)
die("fatal: first page unreadable"); error_msg_and_die("fatal: first page unreadable\n");
if (buffer[508] == 0xDA && buffer[509] == 0xBE) { if (buffer[508] == 0xDA && buffer[509] == 0xBE) {
q = (unsigned short *) (buffer + 510); q = (unsigned short *) (buffer + 510);
for (sum = 0; q >= (unsigned short *) buffer;) for (sum = 0; q >= (unsigned short *) buffer;)
@ -397,7 +391,7 @@ int mkswap_main(int argc, char **argv)
if (version == 0 || check) if (version == 0 || check)
check_blocks(); check_blocks();
if (version == 0 && !bit_test_and_clear(signature_page, 0)) if (version == 0 && !bit_test_and_clear(signature_page, 0))
die("fatal: first page unreadable"); error_msg_and_die("fatal: first page unreadable\n");
if (version == 1) { if (version == 1) {
p->version = version; p->version = version;
p->last_page = PAGES - 1; p->last_page = PAGES - 1;
@ -406,23 +400,23 @@ int mkswap_main(int argc, char **argv)
goodpages = PAGES - badpages - 1; goodpages = PAGES - badpages - 1;
if (goodpages <= 0) if (goodpages <= 0)
die("Unable to set up swap-space: unreadable"); error_msg_and_die("Unable to set up swap-space: unreadable\n");
printf("Setting up swapspace version %d, size = %ld bytes\n", printf("Setting up swapspace version %d, size = %ld bytes\n",
version, (long) (goodpages * pagesize)); version, (long) (goodpages * pagesize));
write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2"); write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2");
offset = ((version == 0) ? 0 : 1024); offset = ((version == 0) ? 0 : 1024);
if (lseek(DEV, offset, SEEK_SET) != offset) if (lseek(DEV, offset, SEEK_SET) != offset)
die("unable to rewind swap-device"); error_msg_and_die("unable to rewind swap-device\n");
if (write(DEV, (char *) signature_page + offset, pagesize - offset) if (write(DEV, (char *) signature_page + offset, pagesize - offset)
!= pagesize - offset) != pagesize - offset)
die("unable to write signature page"); error_msg_and_die("unable to write signature page\n");
/* /*
* A subsequent swapon() will fail if the signature * A subsequent swapon() will fail if the signature
* is not actually on disk. (This is a kernel bug.) * is not actually on disk. (This is a kernel bug.)
*/ */
if (fsync(DEV)) if (fsync(DEV))
die("fsync failed"); error_msg_and_die("fsync failed\n");
return EXIT_SUCCESS; return EXIT_SUCCESS;
} }

View File

@ -255,19 +255,7 @@ static inline int bit(char * a,unsigned int i)
#define mark_zone(x) (setbit(zone_map,(x)-FIRSTZONE+1)) #define mark_zone(x) (setbit(zone_map,(x)-FIRSTZONE+1))
#define unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1)) #define unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1))
/* static __attribute__ ((noreturn)) void show_usage()
* Volatile to let gcc know that this doesn't return. When trying
* to compile this under minix, volatile gives a warning, as
* exit() isn't defined as volatile under minix.
*/
static volatile void die(char *str)
{
error_msg("%s\n", str);
exit(8);
}
static volatile void show_usage() __attribute__ ((noreturn));
static volatile void show_usage()
{ {
usage(mkfs_minix_usage); usage(mkfs_minix_usage);
} }
@ -291,7 +279,7 @@ static void check_mount(void)
if (!mnt) if (!mnt)
return; return;
die("%s is mounted; will not make a filesystem here!"); error_msg_and_die("%s is mounted; will not make a filesystem here!\n", device_name);
} }
static long valid_offset(int fd, int offset) static long valid_offset(int fd, int offset)
@ -348,28 +336,28 @@ static void write_tables(void)
Super.s_state &= ~MINIX_ERROR_FS; Super.s_state &= ~MINIX_ERROR_FS;
if (lseek(DEV, 0, SEEK_SET)) if (lseek(DEV, 0, SEEK_SET))
die("seek to boot block failed in write_tables"); error_msg_and_die("seek to boot block failed in write_tables\n");
if (512 != write(DEV, boot_block_buffer, 512)) if (512 != write(DEV, boot_block_buffer, 512))
die("unable to clear boot sector"); error_msg_and_die("unable to clear boot sector\n");
if (BLOCK_SIZE != lseek(DEV, BLOCK_SIZE, SEEK_SET)) if (BLOCK_SIZE != lseek(DEV, BLOCK_SIZE, SEEK_SET))
die("seek failed in write_tables"); error_msg_and_die("seek failed in write_tables\n");
if (BLOCK_SIZE != write(DEV, super_block_buffer, BLOCK_SIZE)) if (BLOCK_SIZE != write(DEV, super_block_buffer, BLOCK_SIZE))
die("unable to write super-block"); error_msg_and_die("unable to write super-block\n");
if (IMAPS * BLOCK_SIZE != write(DEV, inode_map, IMAPS * BLOCK_SIZE)) if (IMAPS * BLOCK_SIZE != write(DEV, inode_map, IMAPS * BLOCK_SIZE))
die("unable to write inode map"); error_msg_and_die("unable to write inode map\n");
if (ZMAPS * BLOCK_SIZE != write(DEV, zone_map, ZMAPS * BLOCK_SIZE)) if (ZMAPS * BLOCK_SIZE != write(DEV, zone_map, ZMAPS * BLOCK_SIZE))
die("unable to write zone map"); error_msg_and_die("unable to write zone map\n");
if (INODE_BUFFER_SIZE != write(DEV, inode_buffer, INODE_BUFFER_SIZE)) if (INODE_BUFFER_SIZE != write(DEV, inode_buffer, INODE_BUFFER_SIZE))
die("unable to write inodes"); error_msg_and_die("unable to write inodes\n");
} }
static void write_block(int blk, char *buffer) static void write_block(int blk, char *buffer)
{ {
if (blk * BLOCK_SIZE != lseek(DEV, blk * BLOCK_SIZE, SEEK_SET)) if (blk * BLOCK_SIZE != lseek(DEV, blk * BLOCK_SIZE, SEEK_SET))
die("seek failed in write_block"); error_msg_and_die("seek failed in write_block\n");
if (BLOCK_SIZE != write(DEV, buffer, BLOCK_SIZE)) if (BLOCK_SIZE != write(DEV, buffer, BLOCK_SIZE))
die("write failed in write_block"); error_msg_and_die("write failed in write_block\n");
} }
static int get_free_block(void) static int get_free_block(void)
@ -377,7 +365,7 @@ static int get_free_block(void)
int blk; int blk;
if (used_good_blocks + 1 >= MAX_GOOD_BLOCKS) if (used_good_blocks + 1 >= MAX_GOOD_BLOCKS)
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
if (used_good_blocks) if (used_good_blocks)
blk = good_blocks_table[used_good_blocks - 1] + 1; blk = good_blocks_table[used_good_blocks - 1] + 1;
else else
@ -385,7 +373,7 @@ static int get_free_block(void)
while (blk < ZONES && zone_in_use(blk)) while (blk < ZONES && zone_in_use(blk))
blk++; blk++;
if (blk >= ZONES) if (blk >= ZONES)
die("not enough good blocks"); error_msg_and_die("not enough good blocks\n");
good_blocks_table[used_good_blocks] = blk; good_blocks_table[used_good_blocks] = blk;
used_good_blocks++; used_good_blocks++;
return blk; return blk;
@ -451,7 +439,7 @@ static void make_bad_inode(void)
goto end_bad; goto end_bad;
} }
} }
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
end_bad: end_bad:
if (ind) if (ind)
write_block(ind, (char *) ind_block); write_block(ind, (char *) ind_block);
@ -501,7 +489,7 @@ static void make_bad_inode2(void)
} }
} }
/* Could make triple indirect block here */ /* Could make triple indirect block here */
die("too many bad blocks"); error_msg_and_die("too many bad blocks\n");
end_bad: end_bad:
if (ind) if (ind)
write_block(ind, (char *) ind_block); write_block(ind, (char *) ind_block);
@ -602,7 +590,7 @@ static void setup_tables(void)
* /sbin/mkfs.minix -i 200 test.fs * /sbin/mkfs.minix -i 200 test.fs
* */ * */
if (i >= 999) { if (i >= 999) {
die("unable to allocate buffers for maps"); error_msg_and_die("unable to allocate buffers for maps\n");
} }
FIRSTZONE = NORM_FIRSTZONE; FIRSTZONE = NORM_FIRSTZONE;
inode_map = xmalloc(IMAPS * BLOCK_SIZE); inode_map = xmalloc(IMAPS * BLOCK_SIZE);
@ -633,7 +621,7 @@ long do_check(char *buffer, int try, unsigned int current_block)
/* Seek to the correct loc. */ /* Seek to the correct loc. */
if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) != if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) !=
current_block * BLOCK_SIZE) { current_block * BLOCK_SIZE) {
die("seek failed during testing of blocks"); error_msg_and_die("seek failed during testing of blocks\n");
} }
@ -673,7 +661,7 @@ static void check_blocks(void)
while (currently_testing < ZONES) { while (currently_testing < ZONES) {
if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) != if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) !=
currently_testing * BLOCK_SIZE) currently_testing * BLOCK_SIZE)
die("seek failed in check_blocks"); error_msg_and_die("seek failed in check_blocks\n");
try = TEST_BUFFER_BLOCKS; try = TEST_BUFFER_BLOCKS;
if (currently_testing + try > ZONES) if (currently_testing + try > ZONES)
try = ZONES - currently_testing; try = ZONES - currently_testing;
@ -682,7 +670,7 @@ static void check_blocks(void)
if (got == try) if (got == try)
continue; continue;
if (currently_testing < FIRSTZONE) if (currently_testing < FIRSTZONE)
die("bad blocks before data-area: cannot make fs"); error_msg_and_die("bad blocks before data-area: cannot make fs\n");
mark_zone(currently_testing); mark_zone(currently_testing);
badblocks++; badblocks++;
currently_testing++; currently_testing++;
@ -702,7 +690,7 @@ char *filename;
listfile = fopen(filename, "r"); listfile = fopen(filename, "r");
if (listfile == (FILE *) NULL) { if (listfile == (FILE *) NULL) {
die("can't open file of bad blocks"); error_msg_and_die("can't open file of bad blocks\n");
} }
while (!feof(listfile)) { while (!feof(listfile)) {
fscanf(listfile, "%ld\n", &blockno); fscanf(listfile, "%ld\n", &blockno);
@ -724,10 +712,10 @@ extern int mkfs_minix_main(int argc, char **argv)
int stopIt=FALSE; int stopIt=FALSE;
if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE) if (INODE_SIZE * MINIX_INODES_PER_BLOCK != BLOCK_SIZE)
die("bad inode size"); error_msg_and_die("bad inode size\n");
#ifdef BB_FEATURE_MINIX2 #ifdef BB_FEATURE_MINIX2
if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE) if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE)
die("bad inode size"); error_msg_and_die("bad inode size\n");
#endif #endif
/* Parse options */ /* Parse options */
@ -844,13 +832,13 @@ goodbye:
strcpy(tmp + 2, ".badblocks"); strcpy(tmp + 2, ".badblocks");
DEV = open(device_name, O_RDWR); DEV = open(device_name, O_RDWR);
if (DEV < 0) if (DEV < 0)
die("unable to open %s"); error_msg_and_die("unable to open %s\n", device_name);
if (fstat(DEV, &statbuf) < 0) if (fstat(DEV, &statbuf) < 0)
die("unable to stat %s"); error_msg_and_die("unable to stat %s\n", device_name);
if (!S_ISBLK(statbuf.st_mode)) if (!S_ISBLK(statbuf.st_mode))
check = 0; check = 0;
else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340) else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340)
die("will not try to make filesystem on '%s'"); error_msg_and_die("will not try to make filesystem on '%s'\n", device_name);
setup_tables(); setup_tables();
if (check) if (check)
check_blocks(); check_blocks();

View File

@ -173,12 +173,6 @@ static int bit_test_and_clear(unsigned int *addr, unsigned int nr)
} }
static void die(const char *str)
{
error_msg("%s\n", str);
exit(EXIT_FAILURE);
}
static void page_ok(int page) static void page_ok(int page)
{ {
if (version == 0) if (version == 0)
@ -191,7 +185,7 @@ static void page_bad(int page)
bit_test_and_clear(signature_page, page); bit_test_and_clear(signature_page, page);
else { else {
if (badpages == MAX_BADPAGES) if (badpages == MAX_BADPAGES)
die("too many bad pages"); error_msg_and_die("too many bad pages\n");
p->badpages[badpages] = page; p->badpages[badpages] = page;
} }
badpages++; badpages++;
@ -212,7 +206,7 @@ static void check_blocks(void)
} }
if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) != if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) !=
current_page * pagesize) current_page * pagesize)
die("seek failed in check_blocks"); error_msg_and_die("seek failed in check_blocks\n");
if ((do_seek = (pagesize != read(DEV, buffer, pagesize)))) { if ((do_seek = (pagesize != read(DEV, buffer, pagesize)))) {
page_bad(current_page++); page_bad(current_page++);
continue; continue;
@ -369,7 +363,7 @@ int mkswap_main(int argc, char **argv)
if (!S_ISBLK(statbuf.st_mode)) if (!S_ISBLK(statbuf.st_mode))
check = 0; check = 0;
else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340) else if (statbuf.st_rdev == 0x0300 || statbuf.st_rdev == 0x0340)
die("Will not try to make swapdevice on '%s'"); error_msg_and_die("Will not try to make swapdevice on '%s'\n", device_name);
#ifdef __sparc__ #ifdef __sparc__
if (!force && version == 0) { if (!force && version == 0) {
@ -378,7 +372,7 @@ int mkswap_main(int argc, char **argv)
unsigned short *q, sum; unsigned short *q, sum;
if (read(DEV, buffer, 512) != 512) if (read(DEV, buffer, 512) != 512)
die("fatal: first page unreadable"); error_msg_and_die("fatal: first page unreadable\n");
if (buffer[508] == 0xDA && buffer[509] == 0xBE) { if (buffer[508] == 0xDA && buffer[509] == 0xBE) {
q = (unsigned short *) (buffer + 510); q = (unsigned short *) (buffer + 510);
for (sum = 0; q >= (unsigned short *) buffer;) for (sum = 0; q >= (unsigned short *) buffer;)
@ -397,7 +391,7 @@ int mkswap_main(int argc, char **argv)
if (version == 0 || check) if (version == 0 || check)
check_blocks(); check_blocks();
if (version == 0 && !bit_test_and_clear(signature_page, 0)) if (version == 0 && !bit_test_and_clear(signature_page, 0))
die("fatal: first page unreadable"); error_msg_and_die("fatal: first page unreadable\n");
if (version == 1) { if (version == 1) {
p->version = version; p->version = version;
p->last_page = PAGES - 1; p->last_page = PAGES - 1;
@ -406,23 +400,23 @@ int mkswap_main(int argc, char **argv)
goodpages = PAGES - badpages - 1; goodpages = PAGES - badpages - 1;
if (goodpages <= 0) if (goodpages <= 0)
die("Unable to set up swap-space: unreadable"); error_msg_and_die("Unable to set up swap-space: unreadable\n");
printf("Setting up swapspace version %d, size = %ld bytes\n", printf("Setting up swapspace version %d, size = %ld bytes\n",
version, (long) (goodpages * pagesize)); version, (long) (goodpages * pagesize));
write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2"); write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2");
offset = ((version == 0) ? 0 : 1024); offset = ((version == 0) ? 0 : 1024);
if (lseek(DEV, offset, SEEK_SET) != offset) if (lseek(DEV, offset, SEEK_SET) != offset)
die("unable to rewind swap-device"); error_msg_and_die("unable to rewind swap-device\n");
if (write(DEV, (char *) signature_page + offset, pagesize - offset) if (write(DEV, (char *) signature_page + offset, pagesize - offset)
!= pagesize - offset) != pagesize - offset)
die("unable to write signature page"); error_msg_and_die("unable to write signature page\n");
/* /*
* A subsequent swapon() will fail if the signature * A subsequent swapon() will fail if the signature
* is not actually on disk. (This is a kernel bug.) * is not actually on disk. (This is a kernel bug.)
*/ */
if (fsync(DEV)) if (fsync(DEV))
die("fsync failed"); error_msg_and_die("fsync failed\n");
return EXIT_SUCCESS; return EXIT_SUCCESS;
} }