mirror of
https://passt.top/passt
synced 2025-06-19 05:25:34 +02:00
passt-repair: Don't use perror(), accept ECONNRESET as termination
If we use glibc's perror(), we need to allow dup() and fcntl() in our seccomp profiles, which are a bit too much for this simple helper. On top of that, we would probably need a wrapper to avoid allocation for translated messages. While at it: ECONNRESET is just a close() from passt, treat it like EOF. Signed-off-by: Stefano Brivio <sbrivio@redhat.com> Reviewed-by: David Gibson <david@gibson.dropbear.id.au>
This commit is contained in:
parent
a5cca995de
commit
a0b7f56b3a
1 changed files with 7 additions and 3 deletions
|
@ -95,7 +95,7 @@ int main(int argc, char **argv)
|
|||
}
|
||||
|
||||
if ((s = socket(AF_UNIX, SOCK_STREAM, 0)) < 0) {
|
||||
perror("Failed to create AF_UNIX socket");
|
||||
fprintf(stderr, "Failed to create AF_UNIX socket: %i\n", errno);
|
||||
_exit(1);
|
||||
}
|
||||
|
||||
|
@ -108,8 +108,12 @@ int main(int argc, char **argv)
|
|||
loop:
|
||||
ret = recvmsg(s, &msg, 0);
|
||||
if (ret < 0) {
|
||||
perror("Failed to receive message");
|
||||
_exit(1);
|
||||
if (errno == ECONNRESET) {
|
||||
ret = 0;
|
||||
} else {
|
||||
fprintf(stderr, "Failed to read message: %i\n", errno);
|
||||
_exit(1);
|
||||
}
|
||||
}
|
||||
|
||||
if (!ret) /* Done */
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue