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