On Mon, 2016-01-18 at 14:33 +0000, Frediano Ziglio wrote: > This make happy address sanitizer during make check. > Otherwise memory leak detector can keep in and make tests fails. > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> Acked-by: Pavel Grunt <pgrunt@xxxxxxxxxx> > --- > server/tests/test-qxl-parsing.c | 10 +++++++++- > 1 file changed, 9 insertions(+), 1 deletion(-) > > diff --git a/server/tests/test-qxl-parsing.c b/server/tests/test-qxl- > parsing.c > index d44289f..a3b2e6c 100644 > --- a/server/tests/test-qxl-parsing.c > +++ b/server/tests/test-qxl-parsing.c > @@ -65,6 +65,8 @@ int main(int argc, char **argv) > QXLCursor *cursor; > QXLDataChunk *chunks[2]; > > + void *surface_mem; > + > memset(&qxl, 0, sizeof(qxl)); > > qxl.surface_id = 123; > @@ -75,7 +77,8 @@ int main(int argc, char **argv) > qxl.u.surface_create.width = 128; > qxl.u.surface_create.stride = 512; > qxl.u.surface_create.height = 128; > - qxl.u.surface_create.data = to_physical(malloc(0x10000)); > + surface_mem = malloc(0x10000); > + qxl.u.surface_create.data = to_physical(surface_mem); > if (red_get_surface_cmd(&mem_info, 0, &cmd, to_physical(&qxl))) > failure(); > > @@ -117,6 +120,7 @@ int main(int argc, char **argv) > > if (red_get_cursor_cmd(&mem_info, 0, &red_cursor_cmd, > to_physical(&cursor_cmd))) > failure(); > + free(red_cursor_cmd.u.set.shape.data); > free(cursor); > > /* a circular list of empty chunks should not be a problems */ > @@ -173,5 +177,9 @@ int main(int argc, char **argv) > free(cursor); > free(chunks[0]); > > + free(mem_info.mem_slots[0]); > + free(mem_info.mem_slots); > + free(surface_mem); > + > return exit_code; > } _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/spice-devel