diff options
| author | Zach Reizner <zachr@google.com> | 2021-03-26 22:11:16 -0400 |
|---|---|---|
| committer | Laurent Vivier <laurent@vivier.eu> | 2021-03-27 16:48:45 +0100 |
| commit | 4a1e6bce2308b720d79d5ea0a3d24501c89bd80c (patch) | |
| tree | 5a37c203e8888dd5eb15f2f1e26be5e74d9c6e94 /linux-user | |
| parent | 23fff7a17f47420797ac6480147941612152a9ad (diff) | |
| download | qemu-4a1e6bce2308b720d79d5ea0a3d24501c89bd80c.zip | |
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>
Diffstat (limited to 'linux-user')
| -rw-r--r-- | linux-user/syscall.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/linux-user/syscall.c b/linux-user/syscall.c index 1e508576c7..294779c86f 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; |