From b82ac7b08fd7c70d8cd439e633f1fedfdbb156f0 Mon Sep 17 00:00:00 2001 From: "Enrico Weigelt, metux IT consult" Date: Mon, 15 Jul 2024 10:28:39 +0200 Subject: [PATCH] Xext: vidmode: ProcVidModeModModeLine(): move len variable into branch scope Semantically these are separate values in each branch any only used there, so it's a bit more clean to move the declaration into the branches. Signed-off-by: Enrico Weigelt, metux IT consult --- Xext/vidmode.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/Xext/vidmode.c b/Xext/vidmode.c index 1280a4b94..af0f5b3f2 100644 --- a/Xext/vidmode.c +++ b/Xext/vidmode.c @@ -748,8 +748,6 @@ VidModeModModeLine(ClientPtr client, xXF86VidModeModModeLineReq *stuff); static int ProcVidModeModModeLine(ClientPtr client) { - int len; - /* limited to local-only connections */ if (!VidModeAllowNonLocal && !client->local) return VidModeErrorBase + XF86VidModeClientNotLocal; @@ -759,7 +757,7 @@ ProcVidModeModModeLine(ClientPtr client) if (ClientMajorVersion(client) < 2) { REQUEST(xXF86OldVidModeModModeLineReq) REQUEST_AT_LEAST_SIZE(xXF86OldVidModeModModeLineReq); - len = + int len = client->req_len - bytes_to_int32(sizeof(xXF86OldVidModeModModeLineReq)); if (len != stuff->privsize) @@ -786,7 +784,7 @@ ProcVidModeModModeLine(ClientPtr client) else { REQUEST(xXF86VidModeModModeLineReq); REQUEST_AT_LEAST_SIZE(xXF86VidModeModModeLineReq); - len = + int len = client->req_len - bytes_to_int32(sizeof(xXF86VidModeModModeLineReq)); if (len != stuff->privsize)