mirror of
https://github.com/systemd/systemd.git
synced 2024-12-25 01:34:28 +03:00
journal-cat: various modernizations
This commit is contained in:
parent
e4603df5cf
commit
939c173f60
@ -100,10 +100,9 @@ static int parse_argv(int argc, char *argv[]) {
|
||||
int k;
|
||||
|
||||
k = parse_boolean(optarg);
|
||||
if (k < 0) {
|
||||
log_error("Failed to parse level prefix value.");
|
||||
return k;
|
||||
}
|
||||
if (k < 0)
|
||||
return log_error_errno(k, "Failed to parse level prefix value.");
|
||||
|
||||
arg_level_prefix = k;
|
||||
break;
|
||||
}
|
||||
@ -119,7 +118,8 @@ static int parse_argv(int argc, char *argv[]) {
|
||||
}
|
||||
|
||||
int main(int argc, char *argv[]) {
|
||||
int r, fd = -1, saved_stderr = -1;
|
||||
_cleanup_close_ int fd = -1, saved_stderr = -1;
|
||||
int r;
|
||||
|
||||
log_parse_environment();
|
||||
log_open();
|
||||
@ -130,8 +130,7 @@ int main(int argc, char *argv[]) {
|
||||
|
||||
fd = sd_journal_stream_fd(arg_identifier, arg_priority, arg_level_prefix);
|
||||
if (fd < 0) {
|
||||
log_error_errno(fd, "Failed to create stream fd: %m");
|
||||
r = fd;
|
||||
r = log_error_errno(fd, "Failed to create stream fd: %m");
|
||||
goto finish;
|
||||
}
|
||||
|
||||
@ -145,25 +144,20 @@ int main(int argc, char *argv[]) {
|
||||
|
||||
if (fd >= 3)
|
||||
safe_close(fd);
|
||||
|
||||
fd = -1;
|
||||
|
||||
if (argc <= optind)
|
||||
execl("/bin/cat", "/bin/cat", NULL);
|
||||
(void) execl("/bin/cat", "/bin/cat", NULL);
|
||||
else
|
||||
execvp(argv[optind], argv + optind);
|
||||
|
||||
(void) execvp(argv[optind], argv + optind);
|
||||
r = -errno;
|
||||
|
||||
/* Let's try to restore a working stderr, so we can print the error message */
|
||||
if (saved_stderr >= 0)
|
||||
dup3(saved_stderr, STDERR_FILENO, 0);
|
||||
(void) dup3(saved_stderr, STDERR_FILENO, 0);
|
||||
|
||||
log_error_errno(r, "Failed to execute process: %m");
|
||||
|
||||
finish:
|
||||
safe_close(fd);
|
||||
safe_close(saved_stderr);
|
||||
|
||||
return r < 0 ? EXIT_FAILURE : EXIT_SUCCESS;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user