Cleanup patch from Vladimir N. Oleynik.
authorEric Andersen <andersen@codepoet.org>
Wed, 31 Jan 2001 17:35:02 +0000 (17:35 -0000)
committerEric Andersen <andersen@codepoet.org>
Wed, 31 Jan 2001 17:35:02 +0000 (17:35 -0000)
    * mkdir: remove 3 lines in source code.
    * mkfs_minix: save 32 bytes, remove 4 bugs.
    * mkswap: save 64 bytes, remove 1 bug.

coreutils/mkdir.c
mkdir.c
mkfs_minix.c
mkswap.c
util-linux/mkfs_minix.c
util-linux/mkswap.c

index 2345d88314dec8e51ca51cc2cf525e49e7082570..07b18713a9a2e2bb72d213c01c03b3831ea48015 100644 (file)
@@ -52,8 +52,7 @@ extern int mkdir_main(int argc, char **argv)
                                /* Find the specified modes */
                                mode = 0;
                                if (parse_mode(*(++argv), &mode) == FALSE) {
-                                       error_msg("Unknown mode: %s\n", *argv);
-                                       return EXIT_FAILURE;
+                                       error_msg_and_die("Unknown mode: %s\n", *argv);
                                }
                                /* Set the umask for this process so it doesn't 
                                 * screw up whatever the user just entered. */
@@ -81,22 +80,19 @@ extern int mkdir_main(int argc, char **argv)
                char buf[BUFSIZ + 1];
 
                if (strlen(*argv) > BUFSIZ - 1) {
-                       error_msg(name_too_long);
-                       return EXIT_FAILURE;
+                       error_msg_and_die(name_too_long);
                }
                strcpy(buf, *argv);
                status = stat(buf, &statBuf);
                if (parentFlag == FALSE && status != -1 && errno != ENOENT) {
-                       error_msg("%s: File exists\n", buf);
-                       return EXIT_FAILURE;
+                       error_msg_and_die("%s: File exists\n", buf);
                }
                if (parentFlag == TRUE) {
                        strcat(buf, "/");
                        create_path(buf, mode);
                } else {
                        if (mkdir(buf, mode) != 0 && parentFlag == FALSE) {
-                               perror(buf);
-                               return EXIT_FAILURE;
+                               perror_msg_and_die(buf);
                        }
                }
                argc--;
diff --git a/mkdir.c b/mkdir.c
index 2345d88314dec8e51ca51cc2cf525e49e7082570..07b18713a9a2e2bb72d213c01c03b3831ea48015 100644 (file)
--- a/mkdir.c
+++ b/mkdir.c
@@ -52,8 +52,7 @@ extern int mkdir_main(int argc, char **argv)
                                /* Find the specified modes */
                                mode = 0;
                                if (parse_mode(*(++argv), &mode) == FALSE) {
-                                       error_msg("Unknown mode: %s\n", *argv);
-                                       return EXIT_FAILURE;
+                                       error_msg_and_die("Unknown mode: %s\n", *argv);
                                }
                                /* Set the umask for this process so it doesn't 
                                 * screw up whatever the user just entered. */
@@ -81,22 +80,19 @@ extern int mkdir_main(int argc, char **argv)
                char buf[BUFSIZ + 1];
 
                if (strlen(*argv) > BUFSIZ - 1) {
-                       error_msg(name_too_long);
-                       return EXIT_FAILURE;
+                       error_msg_and_die(name_too_long);
                }
                strcpy(buf, *argv);
                status = stat(buf, &statBuf);
                if (parentFlag == FALSE && status != -1 && errno != ENOENT) {
-                       error_msg("%s: File exists\n", buf);
-                       return EXIT_FAILURE;
+                       error_msg_and_die("%s: File exists\n", buf);
                }
                if (parentFlag == TRUE) {
                        strcat(buf, "/");
                        create_path(buf, mode);
                } else {
                        if (mkdir(buf, mode) != 0 && parentFlag == FALSE) {
-                               perror(buf);
-                               return EXIT_FAILURE;
+                               perror_msg_and_die(buf);
                        }
                }
                argc--;
index e1ede6caa3357ba3c8ec4920aceb39498d5f7677..21965d3b1c0adc77bc19dec7354b8bd1480300ee 100644 (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 unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1))
 
-/*
- * 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()
+static __attribute__ ((noreturn)) void show_usage()
 {
        usage(mkfs_minix_usage);
 }
@@ -291,7 +279,7 @@ static void check_mount(void)
        if (!mnt)
                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)
@@ -348,28 +336,28 @@ static void write_tables(void)
        Super.s_state &= ~MINIX_ERROR_FS;
 
        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))
-               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))
-               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))
-               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))
-               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))
-               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))
-               die("unable to write inodes");
+               error_msg_and_die("unable to write inodes\n");
 
 }
 
 static void write_block(int blk, char *buffer)
 {
        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))
-               die("write failed in write_block");
+               error_msg_and_die("write failed in write_block\n");
 }
 
 static int get_free_block(void)
@@ -377,7 +365,7 @@ static int get_free_block(void)
        int blk;
 
        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)
                blk = good_blocks_table[used_good_blocks - 1] + 1;
        else
@@ -385,7 +373,7 @@ static int get_free_block(void)
        while (blk < ZONES && zone_in_use(blk))
                blk++;
        if (blk >= ZONES)
-               die("not enough good blocks");
+               error_msg_and_die("not enough good blocks\n");
        good_blocks_table[used_good_blocks] = blk;
        used_good_blocks++;
        return blk;
@@ -451,7 +439,7 @@ static void make_bad_inode(void)
                                goto end_bad;
                }
        }
-       die("too many bad blocks");
+       error_msg_and_die("too many bad blocks\n");
   end_bad:
        if (ind)
                write_block(ind, (char *) ind_block);
@@ -501,7 +489,7 @@ static void make_bad_inode2(void)
                }
        }
        /* Could make triple indirect block here */
-       die("too many bad blocks");
+       error_msg_and_die("too many bad blocks\n");
   end_bad:
        if (ind)
                write_block(ind, (char *) ind_block);
@@ -602,7 +590,7 @@ static void setup_tables(void)
         * /sbin/mkfs.minix -i 200 test.fs
         * */
        if (i >= 999) {
-               die("unable to allocate buffers for maps");
+               error_msg_and_die("unable to allocate buffers for maps\n");
        }
        FIRSTZONE = NORM_FIRSTZONE;
        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. */
        if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) !=
                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) {
                if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) !=
                        currently_testing * BLOCK_SIZE)
-                       die("seek failed in check_blocks");
+                       error_msg_and_die("seek failed in check_blocks\n");
                try = TEST_BUFFER_BLOCKS;
                if (currently_testing + try > ZONES)
                        try = ZONES - currently_testing;
@@ -682,7 +670,7 @@ static void check_blocks(void)
                if (got == try)
                        continue;
                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);
                badblocks++;
                currently_testing++;
@@ -702,7 +690,7 @@ char *filename;
 
        listfile = fopen(filename, "r");
        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)) {
                fscanf(listfile, "%ld\n", &blockno);
@@ -724,10 +712,10 @@ extern int mkfs_minix_main(int argc, char **argv)
        int stopIt=FALSE;
 
        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
        if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE)
-               die("bad inode size");
+               error_msg_and_die("bad inode size\n");
 #endif
        
        /* Parse options */
@@ -844,13 +832,13 @@ goodbye:
        strcpy(tmp + 2, ".badblocks");
        DEV = open(device_name, O_RDWR);
        if (DEV < 0)
-               die("unable to open %s");
+               error_msg_and_die("unable to open %s\n", device_name);
        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))
                check = 0;
        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();
        if (check)
                check_blocks();
index 5b908daa34a29c147b397cca8d964dd76a7a6f07..8a3c900f31d02a579065c89f712ddc9d84830fd9 100644 (file)
--- a/mkswap.c
+++ b/mkswap.c
@@ -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)
 {
        if (version == 0)
@@ -191,7 +185,7 @@ static void page_bad(int page)
                bit_test_and_clear(signature_page, page);
        else {
                if (badpages == MAX_BADPAGES)
-                       die("too many bad pages");
+                       error_msg_and_die("too many bad pages\n");
                p->badpages[badpages] = page;
        }
        badpages++;
@@ -212,7 +206,7 @@ static void check_blocks(void)
                }
                if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) !=
                        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)))) {
                        page_bad(current_page++);
                        continue;
@@ -369,7 +363,7 @@ int mkswap_main(int argc, char **argv)
        if (!S_ISBLK(statbuf.st_mode))
                check = 0;
        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__
        if (!force && version == 0) {
@@ -378,7 +372,7 @@ int mkswap_main(int argc, char **argv)
                unsigned short *q, sum;
 
                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) {
                        q = (unsigned short *) (buffer + 510);
                        for (sum = 0; q >= (unsigned short *) buffer;)
@@ -397,7 +391,7 @@ int mkswap_main(int argc, char **argv)
        if (version == 0 || check)
                check_blocks();
        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) {
                p->version = version;
                p->last_page = PAGES - 1;
@@ -406,23 +400,23 @@ int mkswap_main(int argc, char **argv)
 
        goodpages = PAGES - badpages - 1;
        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",
                   version, (long) (goodpages * pagesize));
        write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2");
 
        offset = ((version == 0) ? 0 : 1024);
        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)
                != 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
         * is not actually on disk. (This is a kernel bug.)
         */
        if (fsync(DEV))
-               die("fsync failed");
+               error_msg_and_die("fsync failed\n");
        return EXIT_SUCCESS;
 }
index e1ede6caa3357ba3c8ec4920aceb39498d5f7677..21965d3b1c0adc77bc19dec7354b8bd1480300ee 100644 (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 unmark_zone(x) (clrbit(zone_map,(x)-FIRSTZONE+1))
 
-/*
- * 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()
+static __attribute__ ((noreturn)) void show_usage()
 {
        usage(mkfs_minix_usage);
 }
@@ -291,7 +279,7 @@ static void check_mount(void)
        if (!mnt)
                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)
@@ -348,28 +336,28 @@ static void write_tables(void)
        Super.s_state &= ~MINIX_ERROR_FS;
 
        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))
-               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))
-               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))
-               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))
-               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))
-               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))
-               die("unable to write inodes");
+               error_msg_and_die("unable to write inodes\n");
 
 }
 
 static void write_block(int blk, char *buffer)
 {
        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))
-               die("write failed in write_block");
+               error_msg_and_die("write failed in write_block\n");
 }
 
 static int get_free_block(void)
@@ -377,7 +365,7 @@ static int get_free_block(void)
        int blk;
 
        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)
                blk = good_blocks_table[used_good_blocks - 1] + 1;
        else
@@ -385,7 +373,7 @@ static int get_free_block(void)
        while (blk < ZONES && zone_in_use(blk))
                blk++;
        if (blk >= ZONES)
-               die("not enough good blocks");
+               error_msg_and_die("not enough good blocks\n");
        good_blocks_table[used_good_blocks] = blk;
        used_good_blocks++;
        return blk;
@@ -451,7 +439,7 @@ static void make_bad_inode(void)
                                goto end_bad;
                }
        }
-       die("too many bad blocks");
+       error_msg_and_die("too many bad blocks\n");
   end_bad:
        if (ind)
                write_block(ind, (char *) ind_block);
@@ -501,7 +489,7 @@ static void make_bad_inode2(void)
                }
        }
        /* Could make triple indirect block here */
-       die("too many bad blocks");
+       error_msg_and_die("too many bad blocks\n");
   end_bad:
        if (ind)
                write_block(ind, (char *) ind_block);
@@ -602,7 +590,7 @@ static void setup_tables(void)
         * /sbin/mkfs.minix -i 200 test.fs
         * */
        if (i >= 999) {
-               die("unable to allocate buffers for maps");
+               error_msg_and_die("unable to allocate buffers for maps\n");
        }
        FIRSTZONE = NORM_FIRSTZONE;
        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. */
        if (lseek(DEV, current_block * BLOCK_SIZE, SEEK_SET) !=
                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) {
                if (lseek(DEV, currently_testing * BLOCK_SIZE, SEEK_SET) !=
                        currently_testing * BLOCK_SIZE)
-                       die("seek failed in check_blocks");
+                       error_msg_and_die("seek failed in check_blocks\n");
                try = TEST_BUFFER_BLOCKS;
                if (currently_testing + try > ZONES)
                        try = ZONES - currently_testing;
@@ -682,7 +670,7 @@ static void check_blocks(void)
                if (got == try)
                        continue;
                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);
                badblocks++;
                currently_testing++;
@@ -702,7 +690,7 @@ char *filename;
 
        listfile = fopen(filename, "r");
        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)) {
                fscanf(listfile, "%ld\n", &blockno);
@@ -724,10 +712,10 @@ extern int mkfs_minix_main(int argc, char **argv)
        int stopIt=FALSE;
 
        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
        if (INODE_SIZE2 * MINIX2_INODES_PER_BLOCK != BLOCK_SIZE)
-               die("bad inode size");
+               error_msg_and_die("bad inode size\n");
 #endif
        
        /* Parse options */
@@ -844,13 +832,13 @@ goodbye:
        strcpy(tmp + 2, ".badblocks");
        DEV = open(device_name, O_RDWR);
        if (DEV < 0)
-               die("unable to open %s");
+               error_msg_and_die("unable to open %s\n", device_name);
        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))
                check = 0;
        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();
        if (check)
                check_blocks();
index 5b908daa34a29c147b397cca8d964dd76a7a6f07..8a3c900f31d02a579065c89f712ddc9d84830fd9 100644 (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)
 {
        if (version == 0)
@@ -191,7 +185,7 @@ static void page_bad(int page)
                bit_test_and_clear(signature_page, page);
        else {
                if (badpages == MAX_BADPAGES)
-                       die("too many bad pages");
+                       error_msg_and_die("too many bad pages\n");
                p->badpages[badpages] = page;
        }
        badpages++;
@@ -212,7 +206,7 @@ static void check_blocks(void)
                }
                if (do_seek && lseek(DEV, current_page * pagesize, SEEK_SET) !=
                        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)))) {
                        page_bad(current_page++);
                        continue;
@@ -369,7 +363,7 @@ int mkswap_main(int argc, char **argv)
        if (!S_ISBLK(statbuf.st_mode))
                check = 0;
        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__
        if (!force && version == 0) {
@@ -378,7 +372,7 @@ int mkswap_main(int argc, char **argv)
                unsigned short *q, sum;
 
                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) {
                        q = (unsigned short *) (buffer + 510);
                        for (sum = 0; q >= (unsigned short *) buffer;)
@@ -397,7 +391,7 @@ int mkswap_main(int argc, char **argv)
        if (version == 0 || check)
                check_blocks();
        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) {
                p->version = version;
                p->last_page = PAGES - 1;
@@ -406,23 +400,23 @@ int mkswap_main(int argc, char **argv)
 
        goodpages = PAGES - badpages - 1;
        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",
                   version, (long) (goodpages * pagesize));
        write_signature((version == 0) ? "SWAP-SPACE" : "SWAPSPACE2");
 
        offset = ((version == 0) ? 0 : 1024);
        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)
                != 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
         * is not actually on disk. (This is a kernel bug.)
         */
        if (fsync(DEV))
-               die("fsync failed");
+               error_msg_and_die("fsync failed\n");
        return EXIT_SUCCESS;
 }