Re: [PATCH 1/1] win32u: Fix data race in NtUserGetProcessDpiAwarenessContext.

[Zebediah Figura]

On 10/3/22 07:45, Jinoh Kang wrote:
> @@ -5326,14 +5326,17 @@ BOOL WINAPI NtUserSetProcessDpiAwarenessContext( ULONG awareness, ULONG unknown
>    */
>   ULONG WINAPI NtUserGetProcessDpiAwarenessContext( HANDLE process )
>   {
> +    DPI_AWARENESS val;
> +
>       if (process && process != GetCurrentProcess())
>       {
>           WARN( "not supported on other process %p\n", process );
>           return NTUSER_DPI_UNAWARE;
>       }
>
> -    if (!dpi_awareness) return NTUSER_DPI_UNAWARE;
> -    return dpi_awareness;
> +    val = ReadAcquire( &dpi_awareness );
> +    if (!val) return NTUSER_DPI_UNAWARE;
> +    return val;

Why does this need acquire semantics?