Henri Verbeet : wined3d: Try to make Coverity shut up (CID 863, 905).

Alexandre Julliard julliard at winehq.org
Tue Mar 24 09:01:57 CDT 2009


Module: wine
Branch: master
Commit: 8ce2506aa84e4c7a5185226d023d7bd7a903dd91
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=8ce2506aa84e4c7a5185226d023d7bd7a903dd91

Author: Henri Verbeet <hverbeet at codeweavers.com>
Date:   Tue Mar 24 10:09:21 2009 +0100

wined3d: Try to make Coverity shut up (CID 863, 905).

Comparing gl_shaders against NULL after potentially dereferencing it a few
lines earlier probably confuses Coverity. Checking num_gl_shaders instead
should avoid this.

---

 dlls/wined3d/pixelshader.c  |    3 ++-
 dlls/wined3d/vertexshader.c |    3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/dlls/wined3d/pixelshader.c b/dlls/wined3d/pixelshader.c
index 68dd801..6b4a30b 100644
--- a/dlls/wined3d/pixelshader.c
+++ b/dlls/wined3d/pixelshader.c
@@ -530,7 +530,8 @@ GLuint find_gl_pshader(IWineD3DPixelShaderImpl *shader, const struct ps_compile_
 
     TRACE("No matching GL shader found, compiling a new shader\n");
     if(shader->shader_array_size == shader->num_gl_shaders) {
-        if(shader->gl_shaders) {
+        if (shader->num_gl_shaders)
+        {
             new_size = shader->shader_array_size + max(1, shader->shader_array_size / 2);
             new_array = HeapReAlloc(GetProcessHeap(), 0, shader->gl_shaders,
                                     new_size * sizeof(*shader->gl_shaders));
diff --git a/dlls/wined3d/vertexshader.c b/dlls/wined3d/vertexshader.c
index e8d4b84..4888c2a 100644
--- a/dlls/wined3d/vertexshader.c
+++ b/dlls/wined3d/vertexshader.c
@@ -465,7 +465,8 @@ GLuint find_gl_vshader(IWineD3DVertexShaderImpl *shader, const struct vs_compile
     TRACE("No matching GL shader found, compiling a new shader\n");
 
     if(shader->shader_array_size == shader->num_gl_shaders) {
-        if(shader->gl_shaders) {
+        if (shader->num_gl_shaders)
+        {
             new_size = shader->shader_array_size + max(1, shader->shader_array_size / 2);
             new_array = HeapReAlloc(GetProcessHeap(), 0, shader->gl_shaders,
                                     new_size * sizeof(*shader->gl_shaders));




More information about the wine-cvs mailing list