diff options
| author | Keith Packard <keithp@keithp.com> | 2014-12-09 09:30:58 -0800 |
|---|---|---|
| committer | Alan Coopersmith <alan.coopersmith@oracle.com> | 2014-12-09 11:27:26 -0800 |
| commit | 61b17c0f10307e25e51e30e6fb1d3e3127f82d86 (patch) | |
| tree | 5cc76b49c0ab11efdc506f1fa66189ee3df2b571 /glx | |
| parent | b20912c3d45cbbde3c443e6c3d9e189092fe65e1 (diff) | |
glx: Can't mix declarations and code in X.org sources [CVE-2014-8098 pt. 9]
We're using compiler compatibility settings which generate warnings
when a variable is declared after the first statement.
Signed-off-by: Keith Packard <keithp@keithp.com>
Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Signed-off-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'glx')
| -rw-r--r-- | glx/clientinfo.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/glx/clientinfo.c b/glx/clientinfo.c index c5fef3074..74ad91991 100644 --- a/glx/clientinfo.c +++ b/glx/clientinfo.c @@ -36,13 +36,14 @@ set_client_info(__GLXclientState * cl, xGLXSetClientInfoARBReq * req, ClientPtr client = cl->client; char *gl_extensions; char *glx_extensions; + int size; REQUEST_AT_LEAST_SIZE(xGLXSetClientInfoARBReq); /* Verify that the size of the packet matches the size inferred from the * sizes specified for the various fields. */ - int size = sz_xGLXSetClientInfoARBReq; + size = sz_xGLXSetClientInfoARBReq; size = safe_add(size, safe_mul(req->numVersions, bytes_per_version)); size = safe_add(size, safe_pad(req->numGLExtensionBytes)); size = safe_add(size, safe_pad(req->numGLXExtensionBytes)); |