aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDag-Erling Smørgrav <des@FreeBSD.org>2022-08-08 19:06:35 +0000
committerDag-Erling Smørgrav <des@FreeBSD.org>2022-08-08 19:06:35 +0000
commit441202c00bdf6edbf6b6bc1b1923bebd8e2d8e45 (patch)
tree1c1fbe26c24956b255cd35e60d7feeefc18985d8
parent521abc32e22a74981360aa5883bb383aea9ee243 (diff)
downloadsrc-441202c00bdf6edbf6b6bc1b1923bebd8e2d8e45.tar.gz
src-441202c00bdf6edbf6b6bc1b1923bebd8e2d8e45.zip
apply: clean up error handling.
Sponsored by: Klara, Inc.
-rw-r--r--usr.bin/apply/apply.c15
1 files changed, 8 insertions, 7 deletions
diff --git a/usr.bin/apply/apply.c b/usr.bin/apply/apply.c
index 3cba41c37e64..fc90af6d3dff 100644
--- a/usr.bin/apply/apply.c
+++ b/usr.bin/apply/apply.c
@@ -176,23 +176,24 @@ main(int argc, char *argv[])
*/
for (rval = 0; argc > nargs; argc -= nargs, argv += nargs) {
sbuf_clear(cmdbuf);
- sbuf_cat(cmdbuf, "exec ");
+ if (sbuf_cat(cmdbuf, "exec ") != 0)
+ err(1, "sbuf");
/* Expand command argv references. */
for (p = cmd; *p != '\0'; ++p) {
if (ISMAGICNO(p)) {
- if (sbuf_cat(cmdbuf, argv[(++p)[0] - '0'])
- == -1)
- errc(1, ENOMEM, "sbuf");
+ if (sbuf_cat(cmdbuf, argv[*++p - '0']) != 0)
+ err(1, "sbuf");
} else {
- if (sbuf_putc(cmdbuf, *p) == -1)
- errc(1, ENOMEM, "sbuf");
+ if (sbuf_putc(cmdbuf, *p) != 0)
+ err(1, "sbuf");
}
if (sbuf_len(cmdbuf) > arg_max)
errc(1, E2BIG, NULL);
}
/* Terminate the command string. */
- sbuf_finish(cmdbuf);
+ if (sbuf_finish(cmdbuf) != 0)
+ err(1, "sbuf");
/* Run the command. */
if (debug)