GPUShader: Fix linking working even if one shader compilation failed

Linking without valid shaders works on some drivers. Avoid this case by
forcing linking step to return false.

1 files changed, 5 insertions, 0 deletions

diff --git a/source/blender/gpu/opengl/gl_shader.cc b/source/blender/gpu/opengl/gl_shader.cc
index b2cca2ef45e..3ec818b53a6 100644
--- a/source/blender/gpu/opengl/gl_shader.cc
+++ b/source/blender/gpu/opengl/gl_shader.cc
@@ -151,6 +151,7 @@ GLuint GLShader::create_shader_stage(GLenum gl_stage, MutableSpan<const char *>
   }
   if (!status) {
     glDeleteShader(shader);
+    compilation_failed_ = true;
     return 0;
   }
 
@@ -193,6 +194,10 @@ void GLShader::fragment_shader_from_glsl(MutableSpan<const char *> sources)
 
 bool GLShader::finalize(void)
 {
+  if (compilation_failed_) {
+    return false;
+  }
+
   glLinkProgram(shader_program_);
 
   GLint status;