On 09/09/2013 08:34 PM, Vinson Lee wrote:
> Fixes "Missing varargs init or cleanup" defect reported by Coverity.
> 
> Signed-off-by: Vinson Lee <v...@freedesktop.org>
> ---
>  src/glsl/builtin_functions.cpp | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/glsl/builtin_functions.cpp b/src/glsl/builtin_functions.cpp
> index 5d8f171..39127e6 100644
> --- a/src/glsl/builtin_functions.cpp
> +++ b/src/glsl/builtin_functions.cpp
> @@ -1850,6 +1850,7 @@ builtin_builder::add_function(const char *name, ...)
>  
>        f->add_signature(sig);
>     }
> +   va_end(ap);
>  
>     shader->symbols->add_function(f);
>  }
> 

Oooooops.   Thanks for catching this!

Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to