From 573d70ae25cf3b83b5ceee5f76d6d571b06f4b62 Mon Sep 17 00:00:00 2001
From: "Enrico Weigelt, metux IT consult" <info@metux.net>
Date: Thu, 20 Feb 2025 14:12:44 +0100
Subject: [PATCH] xwin: don't need NULL check before free()

free() is safe against NULL arguments.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
Part-of: <https://gitlab.freedesktop.org/xorg/xserver/-/merge_requests/1818>
---
 hw/xwin/winclipboard/xevents.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/hw/xwin/winclipboard/xevents.c b/hw/xwin/winclipboard/xevents.c
index cac17153f..6a846902e 100644
--- a/hw/xwin/winclipboard/xevents.c
+++ b/hw/xwin/winclipboard/xevents.c
@@ -637,8 +637,7 @@ winClipboardFlushXEvents(HWND hwnd,
             if (xtpText_value) {
                 free(xtpText_value);
             }
-            if (pszConvertData)
-                free(pszConvertData);
+            free(pszConvertData);
             if (hGlobal && pszGlobalData)
                 GlobalUnlock(hGlobal);