summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRobert Foss <[email protected]>2017-03-01 19:14:39 -0500
committerEmil Velikov <[email protected]>2017-03-02 15:45:19 +0000
commit88becf73022d780cfd0d7dbc5bb3911f8b0d2b11 (patch)
treec06497f0a9a743d153bf17614f642639935c131a
parentaf5f13e58c9dfe3a54487a3b002370c1edd39cf3 (diff)
mesa: Avoid read of uninitialized variable
The is_color_attachement variable is later read when handling two separate error cases, where only one of the cases results in the variable being initialized. This can be avoided by giving the variable a safe default value. Coverity-Id: 1398631 Cc: [email protected] Signed-off-by: Robert Foss <[email protected]> Reviewed-by: Ilia Mirkin <[email protected]> Reviewed-by: Anuj Phogat <[email protected]> Reviewed-by: Marek Olšák <[email protected]> Reviewed-by: Emil Velikov <[email protected]>
-rw-r--r--src/mesa/main/fbobject.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mesa/main/fbobject.c b/src/mesa/main/fbobject.c
index c8ec8e68a19..f73a009d626 100644
--- a/src/mesa/main/fbobject.c
+++ b/src/mesa/main/fbobject.c
@@ -3684,7 +3684,7 @@ _mesa_get_framebuffer_attachment_parameter(struct gl_context *ctx,
GLint *params, const char *caller)
{
const struct gl_renderbuffer_attachment *att;
- bool is_color_attachment;
+ bool is_color_attachment = false;
GLenum err;
/* The error code for an attachment type of GL_NONE differs between APIs.