From de1f09554650ff4f22ff4d6e0ab222a7d08525c8 Mon Sep 17 00:00:00 2001 From: Mattias Andrée Date: Sat, 4 Sep 2021 11:43:51 +0200 Subject: Add missing newline to end of error prints MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: Mattias Andrée --- common.c | 6 +++--- editasroot.c | 26 +++++++++++++------------- 2 files changed, 16 insertions(+), 16 deletions(-) diff --git a/common.c b/common.c index 5379f89..5add5a1 100644 --- a/common.c +++ b/common.c @@ -24,7 +24,7 @@ copy_file(int destfd, const char *destfname, int srcfd, const char *srcfname, in if (r <= 0) { if (!r) break; - fprintf(stderr, "%s: read %s: %s", argv0, srcfname, strerror(errno)); + fprintf(stderr, "%s: read %s: %s\n", argv0, srcfname, strerror(errno)); exit(1); } @@ -37,7 +37,7 @@ copy_file(int destfd, const char *destfname, int srcfd, const char *srcfname, in for (p = ok_off; p < r; p += w) { w = write(destfd, buf, (size_t)(r - p)); if (r <= 0) { - fprintf(stderr, "%s: write %s: %s", argv0, destfname, strerror(errno)); + fprintf(stderr, "%s: write %s: %s\n", argv0, destfname, strerror(errno)); exit(1); } } @@ -46,7 +46,7 @@ copy_file(int destfd, const char *destfname, int srcfd, const char *srcfname, in if (!okp) { w = write(destfd, buf, 1); if (r <= 0) { - fprintf(stderr, "%s: write %s: %s", argv0, destfname, strerror(errno)); + fprintf(stderr, "%s: write %s: %s\n", argv0, destfname, strerror(errno)); exit(1); } } diff --git a/editasroot.c b/editasroot.c index 0a092cd..0999cbd 100644 --- a/editasroot.c +++ b/editasroot.c @@ -80,7 +80,7 @@ get_tmpfile_pattern(void) path = malloc(strlen(tmpdir) + sizeof("/"TEMPFILEPATTERN)); if (!path) { - fprintf(stderr, "%s: malloc: %s", argv0, strerror(ENOMEM)); + fprintf(stderr, "%s: malloc: %s\n", argv0, strerror(ENOMEM)); exit(1); } p = stpcpy(path, tmpdir); @@ -99,7 +99,7 @@ run_child(const char *file, int fd, int close_this) switch (pid = fork()) { case -1: - fprintf(stderr, "%s: fork: %s", argv0, strerror(errno)); + fprintf(stderr, "%s: fork: %s\n", argv0, strerror(errno)); exit(1); case 0: @@ -127,7 +127,7 @@ run_editor(const char *editor, const char *file, int close_this) switch (pid = fork()) { case -1: - fprintf(stderr, "%s: fork: %s", argv0, strerror(errno)); + fprintf(stderr, "%s: fork: %s\n", argv0, strerror(errno)); exit(1); case 0: @@ -141,7 +141,7 @@ run_editor(const char *editor, const char *file, int close_this) } if (waitpid(pid, &status, 0) != pid) { - fprintf(stderr, "%s: waitpid %s 0: %s", argv0, editor, strerror(errno)); + fprintf(stderr, "%s: waitpid %s 0: %s\n", argv0, editor, strerror(errno)); exit(1); } @@ -165,14 +165,14 @@ main(int argc, char *argv[]) usage(); if (atexit(cleanup)) { - fprintf(stderr, "%s: atexit: %s", argv0, strerror(errno)); + fprintf(stderr, "%s: atexit: %s\n", argv0, strerror(errno)); exit(1); } /* Get terminal, needed for attributes and to chekh if VISUAL or EDITOR should be used */ ttyfd = open(_PATH_TTY, O_RDONLY); if (ttyfd < 0) { - fprintf(stderr, "%s: open %s O_RDONLY: %s", argv0, _PATH_TTY, strerror(errno)); + fprintf(stderr, "%s: open %s O_RDONLY: %s\n", argv0, _PATH_TTY, strerror(errno)); exit(1); } @@ -191,7 +191,7 @@ main(int argc, char *argv[]) /* Start copier, with a bidirectional channel to it for copying the file */ if (socketpair(PF_LOCAL, SOCK_SEQPACKET, 0, fds)) { - fprintf(stderr, "%s: socketpair PF_LOCAL SOCK_SEQPACKET 0: %s", argv0, strerror(errno)); + fprintf(stderr, "%s: socketpair PF_LOCAL SOCK_SEQPACKET 0: %s\n", argv0, strerror(errno)); exit(1); } pid = run_child(argv[0], fds[1], fds[0]); @@ -206,13 +206,13 @@ main(int argc, char *argv[]) path = get_tmpfile_pattern(); fd = mkstemp(path); if (fd < 0) { - fprintf(stderr, "%s: mkstemp %s: %s", argv0, path, strerror(errno)); + fprintf(stderr, "%s: mkstemp %s: %s\n", argv0, path, strerror(errno)); exit(1); } unlink_this = path; copy_file(fd, path, fds[0], "", &ok); if (close(fd)) { - fprintf(stderr, "%s: write %s: %s", argv0, path, strerror(errno)); + fprintf(stderr, "%s: write %s: %s\n", argv0, path, strerror(errno)); exit(1); } if (shutdown(fds[0], SHUT_RD)) { @@ -233,16 +233,16 @@ main(int argc, char *argv[]) /* Rewrite file from tmpfile and unlink tmpfile */ fd = open(path, O_RDONLY); if (fd < 0) { - fprintf(stderr, "%s: open %s O_RDONLY: %s", argv0, path, strerror(errno)); + fprintf(stderr, "%s: open %s O_RDONLY: %s\n", argv0, path, strerror(errno)); exit(1); } copy_file(fds[0], "", fd, path, NULL); if (close(fd)) { - fprintf(stderr, "%s: read %s: %s", argv0, path, strerror(errno)); + fprintf(stderr, "%s: read %s: %s\n", argv0, path, strerror(errno)); exit(1); } if (close(fds[0])) { - fprintf(stderr, "%s: write : %s", argv0, strerror(errno)); + fprintf(stderr, "%s: write : %s\n", argv0, strerror(errno)); exit(1); } unlink(unlink_this); @@ -251,7 +251,7 @@ main(int argc, char *argv[]) /* Wait for exit copier to exit */ if (waitpid(pid, &status, 0) != pid) { - fprintf(stderr, "%s: waitpid 0: %s", argv0, strerror(errno)); + fprintf(stderr, "%s: waitpid 0: %s\n", argv0, strerror(errno)); exit(1); } return WIFEXITED(status) ? WEXITSTATUS(status) : 1; -- cgit v1.2.3-70-g09d2