[Piglit] [PATCH] ext_framebuffer_multisample: Initialize Test member variables.

Brian Paul brianp at vmware.com
Mon Dec 17 06:51:14 PST 2012


On 12/16/2012 05:15 PM, Vinson Lee wrote:
> Fixes uninitialized scalar field defect reported by Coverity.
>
> Signed-off-by: Vinson Lee<vlee at freedesktop.org>
> ---
>   tests/spec/ext_framebuffer_multisample/common.cpp | 7 ++++++-
>   1 file changed, 6 insertions(+), 1 deletion(-)
>
> diff --git a/tests/spec/ext_framebuffer_multisample/common.cpp b/tests/spec/ext_framebuffer_multisample/common.cpp
> index f25fb53..02a8382 100644
> --- a/tests/spec/ext_framebuffer_multisample/common.cpp
> +++ b/tests/spec/ext_framebuffer_multisample/common.cpp
> @@ -1342,7 +1342,12 @@ Test::Test(TestPattern *pattern, ManifestProgram *manifest_program,
>   	  manifest_program(manifest_program),
>   	  test_resolve(test_resolve),
>   	  blit_type(blit_type),
> -	  srgb(srgb)
> +	  num_samples(0),
> +	  pattern_width(0),
> +	  pattern_height(0),
> +	  supersample_factor(0),
> +	  srgb(srgb),
> +	  downsample_prog()
>   {
>   }
>

Reviewed-by: Brian Paul <brianp at vmware.com>


More information about the Piglit mailing list