fdo-mirrors/mesa
1
0
Fork 0
mirror of https://gitlab.freedesktop.org/mesa/mesa.git synced 2026-05-06 13:48:06 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 1

rbug: fix transmitted texture sizes

Browse source 
The rbug wire format defines the texture size parameters to be uint32_t sized
and uses memcpy to move the function parameters to the message structure.
This caused totally wrong transmitted texture sizes since the height and depth
paramterds have been changed to uint16_t in the gallium API. Fix this by doing
an explicit conversion to the correct representation before packing into the
wire message.

Fixes: e6428092f5 (gallium: decrease the size of pipe_resource - 64 -> 48 bytes)
Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
(cherry picked from commit 6174cba748)
This commit is contained in:
Lucas Stach 2019-09-16 14:48:27 +02:00 • committed by Dylan Baker
parent ef906b4636
commit 306e82acb6
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/gallium/auxiliary/rbug/rbug_texture.c
View file

@ -283,9 +283,9 @@ int rbug_send_texture_info_reply(struct rbug_connection *__con,
uint32_t format,
uint32_t *width,
uint32_t width_len,
uint16_t *height,
uint16_t *h16,
uint32_t height_len,
uint16_t *depth,
uint16_t *d16,
uint32_t depth_len,
uint32_t blockw,
uint32_t blockh,
@ -299,6 +299,8 @@ int rbug_send_texture_info_reply(struct rbug_connection *__con,
uint32_t __pos = 0;
uint8_t *__data = NULL;
int __ret = 0;
uint32_t *height = alloca(sizeof(uint32_t) * height_len);
uint32_t *depth = alloca(sizeof(uint32_t) * height_len);
LEN(8); /* header */
LEN(4); /* serial */
@ -321,6 +323,11 @@ int rbug_send_texture_info_reply(struct rbug_connection *__con,
if (!__data)
return -ENOMEM;
for (int i = 0; i < height_len; i++)
height[i] = h16[i];
for (int i = 0; i < depth_len; i++)
depth[i] = d16[i];
WRITE(4, int32_t, ((int32_t)RBUG_OP_TEXTURE_INFO_REPLY));
WRITE(4, uint32_t, ((uint32_t)(__len / 4)));
WRITE(4, uint32_t, serial); /* serial */