Commit f4c07e77 authored by Jean-Paul Saman's avatar Jean-Paul Saman

xvba_video.c: do not leak but crash

Do not leak subpicture associations, but rather crash so that the bug can be investigated and fixed.
parent 47c4998c
...@@ -216,11 +216,12 @@ int surface_add_association( ...@@ -216,11 +216,12 @@ int surface_add_association(
return 0; return 0;
if (obj_surface->assocs[i]->subpicture == assoc->subpicture) { if (obj_surface->assocs[i]->subpicture == assoc->subpicture) {
/* XXX: this should not happen, but replace it in the interim */ /* XXX: this should not happen, but replace it in the interim */
ASSERT(obj_surface->assocs[i]->surface == assoc->surface); //ASSERT(obj_surface->assocs[i]->surface == assoc->surface);
SubpictureAssociationP a = obj_surface->assocs[i]; //SubpictureAssociationP a = obj_surface->assocs[i];
free(a); /* XXX: release previous association */ //free(a); /* XXX: release previous association */
obj_surface->assocs[i] = assoc; //obj_surface->assocs[i] = assoc;
return 0; abort();
return -1;
} }
} }
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment