9pfs: Fix segfault in do_readdir_many caused by struct dirent overread
authorVitaly Chikunov <vt@altlinux.org>
Wed, 16 Feb 2022 18:18:21 +0000 (21:18 +0300)
committerChristian Schoenebeck <qemu_oss@crudebyte.com>
Thu, 17 Feb 2022 15:57:58 +0000 (16:57 +0100)
commite64e27d5cb103b7764f1a05b6eda7e7fedd517c5
tree6f349f7536e1b2e9ee569f7363b9802e82fb8e20
parent494fbbd3ed46a14ef0924651c058b9b0dcb4a7b4
9pfs: Fix segfault in do_readdir_many caused by struct dirent overread

`struct dirent' returned from readdir(3) could be shorter (or longer)
than `sizeof(struct dirent)', thus memcpy of sizeof length will overread
into unallocated page causing SIGSEGV. Example stack trace:

 #0  0x00005555559ebeed v9fs_co_readdir_many (/usr/bin/qemu-system-x86_64 + 0x497eed)
 #1  0x00005555559ec2e9 v9fs_readdir (/usr/bin/qemu-system-x86_64 + 0x4982e9)
 #2  0x0000555555eb7983 coroutine_trampoline (/usr/bin/qemu-system-x86_64 + 0x963983)
 #3  0x00007ffff73e0be0 n/a (n/a + 0x0)

While fixing this, provide a helper for any future `struct dirent' cloning.

Resolves: https://gitlab.com/qemu-project/qemu/-/issues/841
Cc: qemu-stable@nongnu.org
Co-authored-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Reviewed-by: Dmitry V. Levin <ldv@altlinux.org>
Signed-off-by: Vitaly Chikunov <vt@altlinux.org>
Tested-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Reviewed-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
Acked-by: Greg Kurz <groug@kaod.org>
Tested-by: Vitaly Chikunov <vt@altlinux.org>
Message-Id: <20220216181821.3481527-1-vt@altlinux.org>
[C.S. - Fix typo in source comment. ]
Signed-off-by: Christian Schoenebeck <qemu_oss@crudebyte.com>
hw/9pfs/9p-synth.c
hw/9pfs/9p-synth.h
hw/9pfs/codir.c
include/qemu/osdep.h
util/osdep.c