KonstantinSeurer
/
mesa
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

winsys/virgl/vtest: Correct off-by-one error in resource allocation 

The resource bo array must already extended when the target index is
equal to the current size of the array.

Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
This commit is contained in:
Gert Wollny 2018-09-04 09:36:53 +02:00 committed by Gert Wollny
parent 5341260f62
commit 218ff0d510
1 changed files with 9 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/gallium/winsys/virgl/vtest/virgl_vtest_winsys.c
View File

@ -459,13 +459,18 @@ static void virgl_vtest_add_res(struct virgl_vtest_winsys *vtws,
{
unsigned hash = res->res_handle & (sizeof(cbuf->is_handle_added)-1);
if (cbuf->cres > cbuf->nres) {
cbuf->nres += 256;
cbuf->res_bo = realloc(cbuf->res_bo, cbuf->nres * sizeof(struct virgl_hw_buf*));
if (!cbuf->res_bo) {
if (cbuf->cres >= cbuf->nres) {
unsigned new_nres = cbuf->nres + 256;
struct virgl_hw_res **new_re_bo = REALLOC(cbuf->res_bo,
cbuf->nres * sizeof(struct virgl_hw_buf*),
new_nres * sizeof(struct virgl_hw_buf*));
if (!new_re_bo) {
fprintf(stderr,"failure to add relocation %d, %d\n", cbuf->cres, cbuf->nres);
return;
}
cbuf->res_bo = new_re_bo;
cbuf->nres = new_nres;
}
cbuf->res_bo[cbuf->cres] = NULL;