MINOR: sockpair: move send_fd_uxst() error message in caller
Move the ha_alert() in send_fd_uxst() in the callers and add the FD numbers in the message.
This commit is contained in:
parent
f67e8fb92c
commit
708949da49
|
@ -245,7 +245,6 @@ int send_fd_uxst(int fd, int send_fd)
|
||||||
memcpy(fdptr, &send_fd, sizeof(send_fd));
|
memcpy(fdptr, &send_fd, sizeof(send_fd));
|
||||||
|
|
||||||
if (sendmsg(fd, &msghdr, 0) != sizeof(iobuf)) {
|
if (sendmsg(fd, &msghdr, 0) != sizeof(iobuf)) {
|
||||||
ha_warning("Failed to transfer socket\n");
|
|
||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
@ -341,6 +340,7 @@ static int sockpair_connect_server(struct connection *conn, int flags)
|
||||||
/* The new socket is sent on the other side, it should be retrieved and
|
/* The new socket is sent on the other side, it should be retrieved and
|
||||||
* considered as an 'accept' socket on the server side */
|
* considered as an 'accept' socket on the server side */
|
||||||
if (send_fd_uxst(dst_fd, sv[0]) == -1) {
|
if (send_fd_uxst(dst_fd, sv[0]) == -1) {
|
||||||
|
ha_alert("socketpair: Cannot transfer the fd %d over sockpair@%d. Giving up.\n", sv[0], dst_fd);
|
||||||
close(sv[0]);
|
close(sv[0]);
|
||||||
close(sv[1]);
|
close(sv[1]);
|
||||||
conn->err_code = CO_ER_SOCK_ERR;
|
conn->err_code = CO_ER_SOCK_ERR;
|
||||||
|
|
|
@ -322,7 +322,7 @@ int sock_get_old_sockets(const char *unixsocket)
|
||||||
}
|
}
|
||||||
|
|
||||||
if (send_fd_uxst(dst_fd, sv[0]) == -1) {
|
if (send_fd_uxst(dst_fd, sv[0]) == -1) {
|
||||||
ha_alert("socketpair: cannot transfer socket.\n");
|
ha_alert("socketpair: Cannot transfer the fd %d over sockpair@%d. Giving up.\n", sv[0], dst_fd);
|
||||||
close(sv[0]);
|
close(sv[0]);
|
||||||
close(sv[1]);
|
close(sv[1]);
|
||||||
goto out;
|
goto out;
|
||||||
|
|
Loading…
Reference in New Issue