Mesa (master): nouveau: don't keep stale pointer to free'd data

Ilia Mirkin imirkin at kemper.freedesktop.org
Sat Aug 16 21:55:02 UTC 2014


Module: Mesa
Branch: master
Commit: ef130b6050fc91c12a220f19d8bdd659712f7fb9
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=ef130b6050fc91c12a220f19d8bdd659712f7fb9

Author: Ilia Mirkin <imirkin at alum.mit.edu>
Date:   Sat Aug 16 02:46:01 2014 -0400

nouveau: don't keep stale pointer to free'd data

If ->sys is non-null, we might decide that it's where the data is
stored.

Reviewed-by: Francisco Jerez <currojerez at riseup.net>
Signed-off-by: Ilia Mirkin <imirkin at alum.mit.edu>
Cc: <mesa-stable at lists.freedesktop.org>

---

 src/mesa/drivers/dri/nouveau/nouveau_bufferobj.c |    1 +
 1 file changed, 1 insertion(+)

diff --git a/src/mesa/drivers/dri/nouveau/nouveau_bufferobj.c b/src/mesa/drivers/dri/nouveau/nouveau_bufferobj.c
index 9a3d417..2bb633e 100644
--- a/src/mesa/drivers/dri/nouveau/nouveau_bufferobj.c
+++ b/src/mesa/drivers/dri/nouveau/nouveau_bufferobj.c
@@ -86,6 +86,7 @@ nouveau_bufferobj_data(struct gl_context *ctx, GLenum target, GLsizeiptrARB size
 	/* Free previous storage */
 	nouveau_bo_ref(NULL, &nbo->bo);
 	free(nbo->sys);
+	nbo->sys = NULL;
 
 	if (target == GL_ELEMENT_ARRAY_BUFFER_ARB ||
 	    (size < 512 && usage == GL_DYNAMIC_DRAW_ARB) ||




More information about the mesa-commit mailing list