mirror of
https://passt.top/passt
synced 2025-05-24 18:15:36 +02:00
treewide: Replace strerror() calls
Now that we have logging functions embedding perror() functionality, we can make _some_ calls more terse by using them. In many places, the strerror() calls are still more convenient because, for example, they are used in flow debugging functions, or because the return code variable of interest is not 'errno'. While at it, convert a few error messages from a scant perror style to proper failure descriptions. Signed-off-by: Stefano Brivio <sbrivio@redhat.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
92a22fef93
commit
dba7f0f5ce
11 changed files with 53 additions and 77 deletions
12
passt.c
12
passt.c
|
@ -227,15 +227,11 @@ int main(int argc, char **argv)
|
|||
__openlog("pasta", 0, LOG_DAEMON);
|
||||
|
||||
sa.sa_handler = pasta_child_handler;
|
||||
if (sigaction(SIGCHLD, &sa, NULL)) {
|
||||
die("Couldn't install signal handlers: %s",
|
||||
strerror(errno));
|
||||
}
|
||||
if (sigaction(SIGCHLD, &sa, NULL))
|
||||
die_perror("Couldn't install signal handlers");
|
||||
|
||||
if (signal(SIGPIPE, SIG_IGN) == SIG_ERR) {
|
||||
die("Couldn't set disposition for SIGPIPE: %s",
|
||||
strerror(errno));
|
||||
}
|
||||
if (signal(SIGPIPE, SIG_IGN) == SIG_ERR)
|
||||
die_perror("Couldn't set disposition for SIGPIPE");
|
||||
|
||||
c.mode = MODE_PASTA;
|
||||
} else if (strstr(name, "passt")) {
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue