[PATCH 1/3] mkfs.cramfs: use symbolic exit codes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



For some reason original author had defined symbolic exit codes,
but had not use them. One could argue the symbolic exit codes for
mkfs should go to local exit header which would be included to
all commands that exit with non zero or one return code.

Signed-off-by: Sami Kerola <kerolasa@xxxxxx>
---
 disk-utils/mkfs.cramfs.c |   40 ++++++++++++++++++++--------------------
 1 files changed, 20 insertions(+), 20 deletions(-)

diff --git a/disk-utils/mkfs.cramfs.c b/disk-utils/mkfs.cramfs.c
index 4ffa908..c56aab4 100644
--- a/disk-utils/mkfs.cramfs.c
+++ b/disk-utils/mkfs.cramfs.c
@@ -151,7 +151,7 @@ xmalloc (size_t size) {
 	void *t = malloc(size);
 	if (t == NULL) {
 		perror(NULL);
-		exit(8);	/* out of memory */
+		exit(MKFS_ERROR);	/* out of memory */
 	}
 	return t;
 }
@@ -184,7 +184,7 @@ do_mmap(char *path, unsigned int size, unsigned int mode){
 	start = mmap(NULL, size, PROT_READ, MAP_PRIVATE, fd, 0);
 	if (-1 == (int) (long) start) {
 		perror("mmap");
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 	close(fd);
 
@@ -310,7 +310,7 @@ static unsigned int parse_directory(struct entry *root_entry, const char *name,
 
 	if (dircount < 0) {
 		perror(name);
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 
 	/* process directory */
@@ -340,7 +340,7 @@ static unsigned int parse_directory(struct entry *root_entry, const char *name,
 				  " Please increase MAX_INPUT_NAMELEN in "
 				  "mkcramfs.c and recompile.  Exiting.\n"),
 				namelen, dirent->d_name);
-			exit(8);
+			exit(MKFS_ERROR);
 		}
 		memcpy(endpath, dirent->d_name, namelen + 1);
 
@@ -352,12 +352,12 @@ static unsigned int parse_directory(struct entry *root_entry, const char *name,
 		entry = calloc(1, sizeof(struct entry));
 		if (!entry) {
 			perror(NULL);
-			exit(8);
+			exit(MKFS_ERROR);
 		}
 		entry->name = (unsigned char *)strdup(dirent->d_name);
 		if (!entry->name) {
 			perror(NULL);
-			exit(8);
+			exit(MKFS_ERROR);
 		}
 		if (namelen > 255) {
 			/* Can't happen when reading from ext2fs. */
@@ -469,7 +469,7 @@ static void set_data_offset(struct entry *entry, char *base, unsigned long offse
 	inode_to_host(cramfs_is_big_endian, inode, inode);
 	if (offset >= (1 << (2 + CRAMFS_OFFSET_WIDTH))) {
 		fprintf(stderr, _("filesystem too big.  Exiting.\n"));
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 	inode->offset = (offset >> 2);
 	inode_from_host(cramfs_is_big_endian, inode, inode);
@@ -525,7 +525,7 @@ static unsigned int write_directory_structure(struct entry *entry, char *base, u
 					entry_stack = realloc(entry_stack, stack_size * sizeof(struct entry *));
 					if (!entry_stack) {
 						perror(NULL);
-						exit(8);        /* out of memory */
+						exit(MKFS_ERROR);        /* out of memory */
 					}
 				}
 				entry_stack[stack_entries] = entry;
@@ -636,7 +636,7 @@ do_compress(char *base, unsigned int offset, unsigned char const *name,
 			printf(_("AIEEE: block \"compressed\" to > "
 				 "2*blocklength (%ld)\n"),
 			       len);
-			exit(8);
+			exit(MKFS_ERROR);
 		}
 
 		*(u32 *) (base + offset) = u32_toggle_endianness(cramfs_is_big_endian, curr);
@@ -693,7 +693,7 @@ static unsigned int write_file(char *file, char *base, unsigned int offset)
 	fd = open(file, O_RDONLY);
 	if (fd < 0) {
 		perror(file);
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 	buf = mmap(NULL, image_length, PROT_READ, MAP_PRIVATE, fd, 0);
 	memcpy(base + offset, buf, image_length);
@@ -785,7 +785,7 @@ int main(int argc, char **argv)
 			else if (strcmp(optarg, "host") == 0);	/* default */
 			else 	{
 				perror("invalid endianness given. Must be 'big', 'little', or 'host'");
-				exit(16);
+				exit(MKFS_USAGE);
 			}
 
 			break;
@@ -793,7 +793,7 @@ int main(int argc, char **argv)
 			opt_image = optarg;
 			if (lstat(opt_image, &st) < 0) {
 				perror(opt_image);
-				exit(16);
+				exit(MKFS_USAGE);
 			}
 			image_length = st.st_size; /* may be padded later */
 			fslen_ub += (image_length + 3); /* 3 is for padding */
@@ -811,7 +811,7 @@ int main(int argc, char **argv)
 		case 'V':
 			printf(_("%s (%s)\n"),
 			       progname, PACKAGE_STRING);
-			exit(0);
+			exit(MKFS_OK);
 		case 'v':
 			verbose = 1;
 			break;
@@ -828,14 +828,14 @@ int main(int argc, char **argv)
 
 	if (stat(dirname, &st) < 0) {
 		perror(dirname);
-		exit(16);
+		exit(MKFS_USAGE);
 	}
 	fd = open(outfile, O_WRONLY | O_CREAT | O_TRUNC, 0666);
 
 	root_entry = calloc(1, sizeof(struct entry));
 	if (!root_entry) {
 		perror(NULL);
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 	root_entry->mode = st.st_mode;
 	root_entry->uid = st.st_uid;
@@ -878,7 +878,7 @@ int main(int argc, char **argv)
 
 	if (-1 == (int) (long) rom_image) {
 		perror(_("ROM image map"));
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 
 	/* Skip the first opt_pad bytes for boot loader code */
@@ -925,18 +925,18 @@ int main(int argc, char **argv)
 			_("not enough space allocated for ROM image "
 			  "(%lld allocated, %zu used)\n"),
 			(long long) fslen_ub, offset);
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 
 	written = write(fd, rom_image, offset);
 	if (written < 0) {
 		perror(_("ROM image"));
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 	if (offset != written) {
 		fprintf(stderr, _("ROM image write failed (%zd %zd)\n"),
 			written, offset);
-		exit(8);
+		exit(MKFS_ERROR);
 	}
 
 	/* (These warnings used to come at the start, but they scroll off the
@@ -970,6 +970,6 @@ int main(int argc, char **argv)
 			CRAMFS_OFFSET_WIDTH);
 	if (opt_errors &&
 	    (warn_namelen|warn_skip|warn_size|warn_uid|warn_gid|warn_dev))
-		exit(8);
+		exit(MKFS_ERROR);
 	return 0;
 }
-- 
1.7.3.4

--
To unsubscribe from this list: send the line "unsubscribe util-linux" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[Index of Archives]     [Netdev]     [Ethernet Bridging]     [Linux Wireless]     [Kernel Newbies]     [Security]     [Linux for Hams]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux Admin]     [Samba]

  Powered by Linux