Mesa (8.0): glx/tests: Fix off-by-one error in allocating extension string buffer

Ian Romanick idr at kemper.freedesktop.org
Tue Jul 3 19:40:41 UTC 2012


Module: Mesa
Branch: 8.0
Commit: 313d48a11006646ea0beb2f39b3233e3cc17eb35
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=313d48a11006646ea0beb2f39b3233e3cc17eb35

Author: Ian Romanick <ian.d.romanick at intel.com>
Date:   Mon Jun  4 13:28:16 2012 -0700

glx/tests: Fix off-by-one error in allocating extension string buffer

NOTE: This is a candidate for the 8.0 release branch.

Signed-off-by: Ian Romanick <ian.d.romanick at intel.com>
Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=50621
Bugzilla: https://bugs.gentoo.org/show_bug.cgi?id=418161
Reviewed-by: Brian Paul <brianp at vmware.com>
Reviewed-by: Kenneth Graunke <kenneth at whitecape.org>
Cc: Markus Oehme <oehme.markus at gmx.de>
(cherry picked from commit 5fb178ee43fbc364b150fe6c6f0f79e8d8b0b179)

---

 tests/glx/fake_glx_screen.h |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/tests/glx/fake_glx_screen.h b/tests/glx/fake_glx_screen.h
index 2151522..6aa6cb6 100644
--- a/tests/glx/fake_glx_screen.h
+++ b/tests/glx/fake_glx_screen.h
@@ -40,7 +40,7 @@ public:
       this->display = glx_dpy;
       this->dpy = (glx_dpy != NULL) ? glx_dpy->dpy : NULL;
 
-      this->serverGLXexts = new char[strlen(ext)];
+      this->serverGLXexts = new char[strlen(ext) + 1];
       strcpy((char *) this->serverGLXexts, ext);
    }
 




More information about the mesa-commit mailing list