X-Git-Url: https://git.draconx.ca/gitweb/liblbx.git/blobdiff_plain/5749c5e97a8b984720cef75f328944270874d713..59d43aadba53b164dc529cfef069c5aeb9a65807:/src/lbxtool.c

diff --git a/src/lbxtool.c b/src/lbxtool.c
index 1c32ef4..3c4e436 100644
--- a/src/lbxtool.c
+++ b/src/lbxtool.c
@@ -56,7 +56,7 @@ int filematch(char **argv, const char *name)
 		case FNM_NOMATCH:
 			break;
 		default:
-			errmsg("error matching glob: %s.\n", argv[i]);
+			tool_err(-1, "error matching glob: %s", argv[i]);
 			return 1;
 		}
 	}
@@ -102,8 +102,7 @@ int extract_file(LBXfile *f, const struct lbx_statbuf *stat)
 
 	of = fopen(stat->name, "wb");
 	if (!of) {
-		errmsg("%s: fopen: %s\n",
-			stat->name, strerror(errno));
+		tool_err(0, "%s: fopen", stat->name);
 		return -1;
 	}
 
@@ -111,26 +110,26 @@ int extract_file(LBXfile *f, const struct lbx_statbuf *stat)
 		unsigned char buf[1024];
 
 		rc = lbx_file_read(f, buf, sizeof buf);
-		if (rc == 0) {
-			if (lbx_file_eof(f))
-				ret = 0;
-			break;
-		}
-
-		if (fwrite(buf, rc, 1, of) != 1) {
-			errmsg("%s: fwrite: %s\n", stat->name, strerror(errno));
-			break;
+		if (rc > 0) {
+			/* Write out any data we got. */
+			if (fwrite(buf, rc, 1, of) != 1) {
+				tool_err(0, "%s: fwrite", stat->name);
+				break;
+			}
 		}
 
+		/* Now test for read errors */
 		if (rc < sizeof buf) {
-			if (lbx_file_eof(f))
+			if (!lbx_file_eof(f))
+				tool_err(-1, "error reading archive: %s", lbx_errmsg());
+			else
 				ret = 0;
 			break;
 		}
 	}
 
 	if (fclose(of) == EOF) {
-		errmsg("%s: fclose: %s\n", stat->name, strerror(errno));
+		tool_err(0, "%s: fclose", stat->name);
 		return -1;
 	}
 
@@ -159,7 +158,7 @@ int extract(struct lbx *lbx, int verbose, char **argv)
 
 		file = lbx_file_open(lbx, i);
 		if (!file) {
-			errmsg("%s: %s.\n", stat.name, lbx_errmsg());
+			tool_err(-1, "%s: %s", stat.name, lbx_errmsg());
 			continue;
 		}
 
@@ -233,7 +232,7 @@ int main(int argc, char **argv)
 		lbx = lbx_open(&stdin_handle, &lbx_pipe_fops, NULL, "stdin");
 
 	if (!lbx) {
-		errmsg("%s: %s.\n", file ? file : "stdin", lbx_errmsg());
+		tool_err(-1, "%s: %s", file ? file : "stdin", lbx_errmsg());
 		return EXIT_FAILURE;
 	}
 
@@ -245,7 +244,7 @@ int main(int argc, char **argv)
 		rc = extract(lbx, verbose, &argv[optind]);
 		break;
 	default:
-		errmsg("you must specify a mode.\n", 0);
+		tool_err(-1, "no mode specified");
 	}
 
 	lbx_close(lbx);