[PATCH vkd3d 2/2] vkd3d-shader: Avoid reading past the end of the buffer in vkd3d_string_buffer_trace() (Valgrind).

Zebediah Figura zfigura at codeweavers.com
Thu Oct 15 10:44:45 CDT 2020


Signed-off-by: Zebediah Figura <zfigura at codeweavers.com>
---
 libs/vkd3d-shader/vkd3d_shader_main.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/libs/vkd3d-shader/vkd3d_shader_main.c b/libs/vkd3d-shader/vkd3d_shader_main.c
index 7c3885e9..664110e7 100644
--- a/libs/vkd3d-shader/vkd3d_shader_main.c
+++ b/libs/vkd3d-shader/vkd3d_shader_main.c
@@ -93,15 +93,15 @@ static int VKD3D_PRINTF_FUNC(2, 3) vkd3d_string_buffer_printf(struct vkd3d_strin
 
 static void vkd3d_string_buffer_trace_(const struct vkd3d_string_buffer *buffer, const char *function)
 {
-    const char *p, *q;
+    const char *p, *q, *end = buffer->buffer + buffer->content_size;
 
     if (!TRACE_ON())
         return;
 
-    for (p = buffer->buffer; *p; p = q)
+    for (p = buffer->buffer; p < end; p = q)
     {
-        if (!(q = strchr(p, '\n')))
-            q = p + strlen(p);
+        if (!(q = memchr(p, '\n', end - p)))
+            q = end;
         else
             ++q;
         vkd3d_dbg_printf(VKD3D_DBG_LEVEL_TRACE, function, "%.*s", (int)(q - p), p);
-- 
2.28.0




More information about the wine-devel mailing list