Re: [PATCH v1 4/6] virtio console: Harden control message handling

Greg Kroah-Hartman
2023-01-20T05:34:13Z

On Thu, Jan 19, 2023 at 03:57:19PM +0200, Alexander Shishkin wrote:
> In handle_control_message(), we look at the ->event field twice, which
> gives a malicious VMM a window in which to switch it from PORT_ADD to
> PORT_REMOVE, triggering a null dereference further down the line:

How is the other VMM have full control over the full message here?
Shouldn't this all have been copied into our local memory if we are
going to be poking around in it? Like I mentioned in my other review,
copy it all once and then parse it. Don't try to mess with individual
fields one at a time otherwise that way lies madness...

thanks,

greg k-h