hurd: Only deallocate addrport when it's valid

Signed-off-by: Sergey Bugaev <bugaevc@gmail.com>
Message-Id: <20230423160548.126576-3-bugaevc@gmail.com>
This commit is contained in:
Sergey Bugaev 2023-04-23 19:05:47 +03:00 committed by Samuel Thibault
parent 70b9173caa
commit 4c39333050
5 changed files with 10 additions and 6 deletions

View File

@ -54,7 +54,8 @@ __recv (int fd, void *buf, size_t n, int flags)
if (err)
return __hurd_sockfail (fd, flags, err);
__mach_port_deallocate (__mach_task_self (), addrport);
if (MACH_PORT_VALID (addrport))
__mach_port_deallocate (__mach_task_self (), addrport);
__vm_deallocate (__mach_task_self (), (vm_address_t) cdata, clen);
if (bufp != buf)

View File

@ -94,7 +94,8 @@ __recvfrom (int fd, void *buf, size_t n, int flags, __SOCKADDR_ARG addrarg,
else if (addr_len != NULL)
*addr_len = 0;
__mach_port_deallocate (__mach_task_self (), addrport);
if (MACH_PORT_VALID (addrport))
__mach_port_deallocate (__mach_task_self (), addrport);
/* Toss control data; we don't care. */
__vm_deallocate (__mach_task_self (), (vm_address_t) cdata, clen);

View File

@ -135,7 +135,8 @@ __libc_recvmsg (int fd, struct msghdr *message, int flags)
else if (message->msg_name != NULL)
message->msg_namelen = 0;
__mach_port_deallocate (__mach_task_self (), aport);
if (MACH_PORT_VALID (aport))
__mach_port_deallocate (__mach_task_self (), aport);
if (buf == data)
buf += len;

View File

@ -195,8 +195,9 @@ __libc_sendmsg (int fd, const struct msghdr *message, int flags)
message->msg_controllen,
&amount);
LIBC_CANCEL_RESET (cancel_oldtype);
__mach_port_deallocate (__mach_task_self (),
aport);
if (MACH_PORT_VALID (aport))
__mach_port_deallocate (__mach_task_self (),
aport);
}
err;
}));

View File

@ -94,7 +94,7 @@ __sendto (int fd,
err;
}));
if (aport != MACH_PORT_NULL)
if (MACH_PORT_VALID (aport))
__mach_port_deallocate (__mach_task_self (), aport);
return err ? __hurd_sockfail (fd, flags, err) : wrote;