[PATCH] shell: fix weston_output_mode_switch() usage
Hardening
rdp.effort at gmail.com
Tue Sep 23 06:34:07 PDT 2014
Le 23/09/2014 09:37, Pekka Paalanen a écrit :
> On Mon, 22 Sep 2014 11:01:10 -0500
> Derek Foreman <derekf at osg.samsung.com> wrote:
>
>> On 22/09/14 06:41 AM, Pekka Paalanen wrote:
>>> On Fri, 19 Sep 2014 14:43:23 -0500
>>> Derek Foreman <derekf at osg.samsung.com> wrote:
>>>
>>>> Calling weston_output_mode_switch() with WESTON_MODE_SWITCH_RESTORE_NATIVE
>>>> will result in the mode being set "back" to the passed in mode - so the
[...]
> No, its really all clients. Even the fullscreen client does not know if
> the temporary mode change it requested actually happened.
>
>>> Therefore: pushed. Feel free to refactor the code in a follow-up,
>>> IMHO. ;-)
>>
>> Ok, I'll try to get to that in the next couple of days.
>
> Thank you.
>
> Hardening, could you keep an eye here and make sure we don't break
> anything for the RDP backend or such?
>
Yes of course. IIRC at the time we had already suggested to split and
rework the function.
> I believe some of the details here are only to cater for RDP, like
> non-enumerable modes.
>
I may be wrong but I remember a conversation where someone was telling
that DRM also have arbitrary modes...
Regards.
--
David FORT
website: http://www.hardening-consulting.com/
More information about the wayland-devel
mailing list