[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Qemu-devel] [PULL 08/19] linux-user: allow NULL arguments to mount
From: |
riku . voipio |
Subject: |
[Qemu-devel] [PULL 08/19] linux-user: allow NULL arguments to mount |
Date: |
Mon, 23 Jun 2014 16:26:23 +0300 |
From: Paul Burton <address@hidden>
Calls to the mount syscall can legitimately provide NULL as the value
for the source of filesystemtype arguments, which QEMU would previously
reject & return -EFAULT to the target program. An example of this is
remounting an already mounted filesystem with different properties.
Instead of rejecting such syscalls with -EFAULT, pass NULL along to the
kernel as the target program expects.
Additionally this patch fixes a potential memory leak when DEBUG_REMAP
is enabled and lock_user_string fails on the target or filesystemtype
arguments but a prior argument was non-NULL and already locked.
Since the patch already touched most lines of the TARGET_NR_mount case,
it fixes the indentation & coding style for good measure.
Signed-off-by: Paul Burton <address@hidden>
Signed-off-by: Riku Voipio <address@hidden>
---
linux-user/syscall.c | 75 +++++++++++++++++++++++++++++++++++++---------------
1 file changed, 53 insertions(+), 22 deletions(-)
diff --git a/linux-user/syscall.c b/linux-user/syscall.c
index 3971cb5..4e48af6 100644
--- a/linux-user/syscall.c
+++ b/linux-user/syscall.c
@@ -5614,29 +5614,60 @@ abi_long do_syscall(void *cpu_env, int num, abi_long
arg1,
break;
#endif
case TARGET_NR_mount:
- {
- /* need to look at the data field */
- void *p2, *p3;
- p = lock_user_string(arg1);
- p2 = lock_user_string(arg2);
- p3 = lock_user_string(arg3);
- if (!p || !p2 || !p3)
- ret = -TARGET_EFAULT;
- else {
- /* FIXME - arg5 should be locked, but it isn't
clear how to
- * do that since it's not guaranteed to be a
NULL-terminated
- * string.
- */
- if ( ! arg5 )
- ret = get_errno(mount(p, p2, p3, (unsigned
long)arg4, NULL));
- else
- ret = get_errno(mount(p, p2, p3, (unsigned
long)arg4, g2h(arg5)));
- }
+ {
+ /* need to look at the data field */
+ void *p2, *p3;
+
+ if (arg1) {
+ p = lock_user_string(arg1);
+ if (!p) {
+ goto efault;
+ }
+ } else {
+ p = NULL;
+ }
+
+ p2 = lock_user_string(arg2);
+ if (!p2) {
+ if (arg1) {
+ unlock_user(p, arg1, 0);
+ }
+ goto efault;
+ }
+
+ if (arg3) {
+ p3 = lock_user_string(arg3);
+ if (!p3) {
+ if (arg1) {
unlock_user(p, arg1, 0);
- unlock_user(p2, arg2, 0);
- unlock_user(p3, arg3, 0);
- break;
- }
+ }
+ unlock_user(p2, arg2, 0);
+ goto efault;
+ }
+ } else {
+ p3 = NULL;
+ }
+
+ /* FIXME - arg5 should be locked, but it isn't clear how to
+ * do that since it's not guaranteed to be a NULL-terminated
+ * string.
+ */
+ if (!arg5) {
+ ret = mount(p, p2, p3, (unsigned long)arg4, NULL);
+ } else {
+ ret = mount(p, p2, p3, (unsigned long)arg4, g2h(arg5));
+ }
+ ret = get_errno(ret);
+
+ if (arg1) {
+ unlock_user(p, arg1, 0);
+ }
+ unlock_user(p2, arg2, 0);
+ if (arg3) {
+ unlock_user(p3, arg3, 0);
+ }
+ }
+ break;
#ifdef TARGET_NR_umount
case TARGET_NR_umount:
if (!(p = lock_user_string(arg1)))
--
2.0.0
- [Qemu-devel] [PULL 13/19] linux-user: support timerfd_{create, gettime, settime} syscalls, (continued)
- [Qemu-devel] [PULL 13/19] linux-user: support timerfd_{create, gettime, settime} syscalls, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 01/19] Add support for MAP_NORESERVE mmap flag., riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 17/19] linux-user: support the unshare syscall, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 12/19] linux-user: allow NULL tv argument for settimeofday, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 11/19] linux-user: respect timezone for settimeofday, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 05/19] linux-user: support SO_ACCEPTCONN getsockopt option, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 16/19] linux-user: support the setns syscall, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 07/19] linux-user: support SO_PASSSEC setsockopt option, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 14/19] linux-user: support ioprio_{get, set} syscalls, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 15/19] linux-user: support {name_to, open_by}_handle_at syscalls, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 08/19] linux-user: allow NULL arguments to mount,
riku . voipio <=
- [Qemu-devel] [PULL 06/19] linux-user: support SO_{SND, RCV}BUFFORCE setsockopt options, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 19/19] linux-user: support the SIOCGIFINDEX ioctl, riku . voipio, 2014/06/23
- [Qemu-devel] [PULL 18/19] linux-user: support the KDSIGACCEPT ioctl, riku . voipio, 2014/06/23
- Re: [Qemu-devel] [PULL 00/19] linux-user changes for 2.1, Peter Maydell, 2014/06/23