From 6203998905893e537f8d6769b83d775cf9dcb132 Mon Sep 17 00:00:00 2001
From: "Enrico Weigelt, metux IT consult" <info@metux.net>
Date: Wed, 28 Aug 2024 13:28:50 +0200
Subject: [PATCH] (submit/glx-cleanups-vla) glx: DoQueryContext(): use fixed
 size array instead of variable length

Our array here really is fixed, but it's size is determined by a variable
(that's assigned a fix values), unncessarily making it a VLA (even making
it const doesn't change that), so giving false alarms when compiling w/
-Wvla or -Werror=vla.

Replacing the variable by #define trivially fixes this.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 glx/glxcmds.c | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/glx/glxcmds.c b/glx/glxcmds.c
index 635f145c2..8e7a29a3f 100644
--- a/glx/glxcmds.c
+++ b/glx/glxcmds.c
@@ -1654,13 +1654,14 @@ __glXDisp_SwapBuffers(__GLXclientState * cl, GLbyte * pc)
     return Success;
 }
 
+#define GLX_QUERY_NPROPS 5
+
 static int
 DoQueryContext(__GLXclientState * cl, GLXContextID gcId)
 {
     ClientPtr client = cl->client;
     __GLXcontext *ctx;
-    int nProps = 5;
-    int sendBuf[nProps * 2];
+    int sendBuf[GLX_QUERY_NPROPS * 2];
     int nReplyBytes;
     int err;
 
@@ -1670,8 +1671,8 @@ DoQueryContext(__GLXclientState * cl, GLXContextID gcId)
     xGLXQueryContextInfoEXTReply reply = {
         .type = X_Reply,
         .sequenceNumber = client->sequence,
-        .length = nProps << 1,
-        .n = nProps
+        .length = GLX_QUERY_NPROPS  << 1,
+        .n = GLX_QUERY_NPROPS,
     };
 
     nReplyBytes = reply.length << 2;
@@ -1706,6 +1707,8 @@ DoQueryContext(__GLXclientState * cl, GLXContextID gcId)
     return Success;
 }
 
+#undef GLX_QUERY_NPROPS
+
 int
 __glXDisp_QueryContextInfoEXT(__GLXclientState * cl, GLbyte * pc)
 {