aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorZach Reizner <zachr@google.com>2021-03-26 22:11:16 -0400
committerLaurent Vivier <laurent@vivier.eu>2021-03-27 16:48:45 +0100
commit4a1e6bce2308b720d79d5ea0a3d24501c89bd80c (patch)
tree5a37c203e8888dd5eb15f2f1e26be5e74d9c6e94
parent23fff7a17f47420797ac6480147941612152a9ad (diff)
downloadqemu-4a1e6bce2308b720d79d5ea0a3d24501c89bd80c.zip
qemu-4a1e6bce2308b720d79d5ea0a3d24501c89bd80c.tar.gz
qemu-4a1e6bce2308b720d79d5ea0a3d24501c89bd80c.tar.bz2
linux-user: allow NULL msg in recvfrom
The kernel allows a NULL msg in recvfrom so that he size of the next message may be queried before allocating a correctly sized buffer. This change allows the syscall translator to pass along the NULL msg pointer instead of returning early with EFAULT. Signed-off-by: Zach Reizner <zachr@google.com> Reviewed-by: Laurent Vivier <laurent@vivier.eu> Message-Id: <CAFNex=DvFCq=AQf+=19fTfw-T8eZZT=3NnFFm2JMFvVr5QgQyA@mail.gmail.com> Signed-off-by: Laurent Vivier <laurent@vivier.eu>
-rw-r--r--linux-user/syscall.c11
1 files changed, 8 insertions, 3 deletions
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 1e50857..294779c 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -3679,9 +3679,14 @@ static abi_long do_recvfrom(int fd, abi_ulong msg, size_t len, int flags,
void *host_msg;
abi_long ret;
- host_msg = lock_user(VERIFY_WRITE, msg, len, 0);
- if (!host_msg)
- return -TARGET_EFAULT;
+ if (!msg) {
+ host_msg = NULL;
+ } else {
+ host_msg = lock_user(VERIFY_WRITE, msg, len, 0);
+ if (!host_msg) {
+ return -TARGET_EFAULT;
+ }
+ }
if (target_addr) {
if (get_user_u32(addrlen, target_addrlen)) {
ret = -TARGET_EFAULT;