[PATCH 0/1] MR303: win32u: Don't report cloned monitors in EnumDisplayMonitors().

Zhiyi Zhang (@zhiyi) wine at gitlab.winehq.org
Thu Jun 23 10:06:38 CDT 2022


Zhiyi Zhang (@zhiyi) commented about dlls/win32u/sysparams.c:
>                  break;
>              }
>  
> +            is_cloned = FALSE;
> +            LIST_FOR_EACH_ENTRY(monitor2, &monitors, struct monitor, entry)
> +            {
> +                if (EqualRect(&monitor2->rc_monitor, &monitor->rc_monitor))
> +                {
> +                    is_cloned = TRUE;
> +                    break;
> +                }
> +            }
> +
> +            if (is_cloned)
> +                continue;
If the monitor to be added is the last monitor and is a cloned replica, this will leak the monitor pointer.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/303#note_2552



More information about the wine-devel mailing list