Don't insert nullptr shader programs into the shader cache

This can happen when the shader program fails to compile/link, and will
subsequently trigger an assert in the shader cache.

Change-Id: I4c4093d89a8a0f5ed1970012258be5efe8ec29c9
Reviewed-by: Sean Harmer <sean.harmer@kdab.com>

1 files changed, 2 insertions, 1 deletions

diff --git a/src/render/graphicshelpers/graphicscontext.cpp b/src/render/graphicshelpers/graphicscontext.cpp
index 5ac83cac0..84163780d 100644
--- a/src/render/graphicshelpers/graphicscontext.cpp
+++ b/src/render/graphicshelpers/graphicscontext.cpp
@@ -517,7 +517,8 @@ void GraphicsContext::loadShader(Shader *shader, ShaderManager *manager)
         shaderProgram = createShaderProgram(shader);
 
         // Store in cache
-        m_shaderCache.insert(shader->dna(), shader->peerId(), shaderProgram);
+        if (shaderProgram)
+            m_shaderCache.insert(shader->dna(), shader->peerId(), shaderProgram);
     }
 
     // Ensure the Shader node knows about the program interface