David Fernandez Gonzalez 48b9a8dabc VMCI: Fix use-after-free when removing resource in vmci_resource_remove() ... When removing a resource from vmci_resource_table in vmci_resource_remove(), the search is performed using the resource handle by comparing context and resource fields. It is possible though to create two resources with different types but same handle (same context and resource fields). When trying to remove one of the resources, vmci_resource_remove() may not remove the intended one, but the object will still be freed as in the case of the datagram type in vmci_datagram_destroy_handle(). vmci_resource_table will still hold a pointer to this freed resource leading to a use-after-free vulnerability. BUG: KASAN: use-after-free in vmci_handle_is_equal include/linux/vmw_vmci_defs.h:142 [inline] BUG: KASAN: use-after-free in vmci_resource_remove+0x3a1/0x410 drivers/misc/vmw_vmci/vmci_resource.c:147 Read of size 4 at addr ffff88801c16d800 by task syz-executor197/1592 Call Trace: <TASK> __dump_stack lib/dump_stack.c:88 [inline] dump_stack_lvl+0x82/0xa9 lib/dump_stack.c:106 print_address_description.constprop.0+0x21/0x366 mm/kasan/report.c:239 __kasan_report.cold+0x7f/0x132 mm/kasan/report.c:425 kasan_report+0x38/0x51 mm/kasan/report.c:442 vmci_handle_is_equal include/linux/vmw_vmci_defs.h:142 [inline] vmci_resource_remove+0x3a1/0x410 drivers/misc/vmw_vmci/vmci_resource.c:147 vmci_qp_broker_detach+0x89a/0x11b9 drivers/misc/vmw_vmci/vmci_queue_pair.c:2182 ctx_free_ctx+0x473/0xbe1 drivers/misc/vmw_vmci/vmci_context.c:444 kref_put include/linux/kref.h:65 [inline] vmci_ctx_put drivers/misc/vmw_vmci/vmci_context.c:497 [inline] vmci_ctx_destroy+0x170/0x1d6 drivers/misc/vmw_vmci/vmci_context.c:195 vmci_host_close+0x125/0x1ac drivers/misc/vmw_vmci/vmci_host.c:143 __fput+0x261/0xa34 fs/file_table.c:282 task_work_run+0xf0/0x194 kernel/task_work.c:164 tracehook_notify_resume include/linux/tracehook.h:189 [inline] exit_to_user_mode_loop+0x184/0x189 kernel/entry/common.c:187 exit_to_user_mode_prepare+0x11b/0x123 kernel/entry/common.c:220 __syscall_exit_to_user_mode_work kernel/entry/common.c:302 [inline] syscall_exit_to_user_mode+0x18/0x42 kernel/entry/common.c:313 do_syscall_64+0x41/0x85 arch/x86/entry/common.c:86 entry_SYSCALL_64_after_hwframe+0x6e/0x0 This change ensures the type is also checked when removing the resource from vmci_resource_table in vmci_resource_remove(). Fixes: bc63dedb7d ("VMCI: resource object implementation.") Cc: stable@vger.kernel.org Reported-by: George Kennedy <george.kennedy@oracle.com> Signed-off-by: David Fernandez Gonzalez <david.fernandez.gonzalez@oracle.com> Link: https://lore.kernel.org/r/20240828154338.754746-1-david.fernandez.gonzalez@oracle.com Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>		2024-09-03 13:16:42 +02:00
..
Kconfig	VMCI: Add support for ARM64	2022-04-24 17:32:14 +02:00
Makefile
vmci_context.c	misc: vmw_vmci: Rename kvfree_rcu() to kvfree_rcu_mightsleep()	2023-04-05 13:48:03 +00:00
vmci_context.h
vmci_datagram.c	VMCI: Fix possible memcpy() run-time warning in vmci_datagram_invoke_guest_handler()	2024-03-01 16:03:32 -08:00
vmci_datagram.h
vmci_doorbell.c
vmci_doorbell.h
vmci_driver.c
vmci_driver.h
vmci_event.c	vmci: prevent speculation leaks by sanitizing event in event_deliver()	2024-05-03 07:28:53 +02:00
vmci_event.h
vmci_guest.c	Char/Misc and other driver subsystem changes for 6.10-rc1	2024-05-22 12:26:46 -07:00
vmci_handle_array.c	VMCI: Remove handle_arr_calc_size()	2023-12-15 17:27:04 +01:00
vmci_handle_array.h	Char/Misc and other Driver changes for 6.8-rc1	2024-01-17 16:47:17 -08:00
vmci_host.c	vmci_host: fix a race condition in vmci_host_poll() causing GPF	2023-04-05 19:37:35 +02:00
vmci_queue_pair.c	use less confusing names for iov_iter direction initializers	2022-11-25 13:01:55 -05:00
vmci_queue_pair.h
vmci_resource.c	VMCI: Fix use-after-free when removing resource in vmci_resource_remove()	2024-09-03 13:16:42 +02:00
vmci_resource.h
vmci_route.c
vmci_route.h