[Spice-devel] [PATCH spice-gtk 1/2] channel-display: Make monitors array contain monitors in id order

[Marc-André Lureau]

Hi

----- Mensaje original -----
> > Ah, I think this is correct as the current model doesn't allow
> > qxl-0
> > and qxl-2 outputs enabled, while qxl-1 is disabled.
> 
> That is not entirely true, currently one cannot do that through an
> agent message, but one can "easily" do it using the xrandr cmdline
> utility. So the code is already broken, the xorg driver and
> qemu/server
> code can send a QXLMonitorsConfig with non continues id range in
> there already, it just requires manual configuration inside the
> guest.

If I remember correctly, that change in the guest/agent will trigger a new monitor config and the client is reconfigured, and it will always have continues monitor id in this case. Can you reproduce an invalid access by playing with 3 monitors and xrandr? I don't recall any problem with that.