winex11.drv: Don't compare error event serial if it's zero.

Sometimes error event serial is zero. For example, NVIDIA driver may send X_GLXCopyContext errors with the event serial set to zero. When it's zero, previously the error handler passes the error to the default error handler and cause application crashes. Signed-off-by: Zhiyi Zhang <zzhang@codeweavers.com>

winex11.drv: Don't compare error event serial if it's zero.
d328af75 · Zhiyi Zhang · Alexandre Julliard · 18ae96e5 · d328af75
Commit d328af75 authored May 17, 2022 by Zhiyi Zhang Committed by Alexandre Julliard Jul 13, 2022
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

x11drv_main.c dlls/winex11.drv/x11drv_main.c +1 -1

No files found.
--- a/dlls/winex11.drv/x11drv_main.c
+++ b/dlls/winex11.drv/x11drv_main.c
@@ -304,7 +304,7 @@ int X11DRV_check_error(void)
 static int error_handler( Display *display, XErrorEvent *error_evt )
 {
    if (err_callback && display == err_callback_display &&
-        (long)(error_evt->serial - err_serial) >= 0)
+        (!error_evt->serial || error_evt->serial >= err_serial))
    {
        if ((err_callback_result = err_callback( display, error_evt, err_callback_arg )))
        {