Re: [Qemu-stable] [PATCH for v2.3.1] exec: Respect as_translate_internal length clamp

[Paolo Bonzini]

On 27/01/2016 13:06, P J P wrote:
> +-- On Wed, 27 Jan 2016, Paolo Bonzini wrote --+
> | This is not enough.  This patch caused a huge fallout back when it was 
> applied.
> | 
> | You need these too, I think:
> | 
> |     exec: skip MMIO regions correctly in 
> cpu_physical_memory_write_rom_internal
> |     exec: clamp accesses against the MemoryRegionSection
> |     exec: do not clamp accesses to MMIO regions
> |     exec: Respect as_translate_internal length clamp
> |     ioport: reserve the whole range of an I/O port in the AddressSpace
> |     ioport: loosen assertions on emulation of 16-bit ports
> |     ioport: remove wrong comment
> |     ide: there is only one data port
> |     gus: clean up MemoryRegionPortio
> |     sb16: remove useless mixer_write_indexw
> 
>   Ie. all these also need to be pulled into versions <= v2.3.1, right? 
> Otherwise the segfault issue seems to be cured by commit - '23820dbfc79d1c9'

In exchange for that, you get a few interesting bugs. :)

Paolo

> | but you'll need to look at the archives for all the conversations going on.
> 
>   Okay, I'll go through the archives and confirm here.
> 
> Thank you.
> --
> Prasad J Pandit / Red Hat Product Security Team
> 47AF CE69 3A90 54AA 9045 1053 DD13 3D32 FE5B 041F
>