sendexev.c: Fix return of unitialized variable

Well, kinda fix. Upstream handles that differently here, we need a thorough check! Add another FIXME comment because of this.

sendexev.c: Fix return of unitialized variable
50eb5e4a · Ulrich Sibiller · Ulrich Sibiller · 151e0b16 · 50eb5e4a
Commit 50eb5e4a authored Oct 20, 2016 by Ulrich Sibiller Committed by Ulrich Sibiller Oct 20, 2016
Hide whitespace changes
Inline Side-by-side

Showing with 7 additions and 3 deletions

sendexev.c nx-X11/programs/Xserver/Xi/sendexev.c +7 -3

No files found.
--- a/nx-X11/programs/Xserver/Xi/sendexev.c
+++ b/nx-X11/programs/Xserver/Xi/sendexev.c
@@ -149,10 +149,14 @@ ProcXSendExtensionEvent (client)
 	return Success;
 	}

-    // FIXME: ret is not initialized and should not be returned...
-
+    /*
+       the previous code here returned the unitialized variable ret,
+       so using Success we have defined returncode at least. FIXME:
+       Upstream works different here, we must check this!
+    */
    if (stuff->num_events == 0)
-        return ret;
+        /* return ret; */
+        return Success;

    /* The client's event type must be one defined by an extension. */