[Mesa-dev] [PATCH] mesa: Fix memory leak in out-of-memory path.

Fixes Coverity resource leak defect. --- src/mesa/main/pack.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/pack.c b/src/mesa/main/pack.c index ecdeaf5..539a06c 100644 --- a/src/mesa/main/pack.c +++ b/src/mesa/main/pack.c @@ -1983,6 +1983,7 @@

[Mesa-dev] [PATCH] radeon: Fix variable initialization typo.

Fixes Coverity uninitialized scalar variable defect. --- src/mesa/drivers/dri/radeon/radeon_common.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/mesa/drivers/dri/radeon/radeon_common.c b/src/mesa/drivers/dri/radeon/radeon_common.c index 2ef3faf..2b87f96 100644

[Mesa-dev] [PATCH] pp: Reorder calloc to avoid memory leak on error path.

Fixes Coverity resource leak defect. --- src/gallium/auxiliary/postprocess/pp_mlaa.c |7 --- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/src/gallium/auxiliary/postprocess/pp_mlaa.c b/src/gallium/auxiliary/postprocess/pp_mlaa.c index f514e69..51bc02e 100644 ---

[Mesa-dev] [PATCH] g3dvl: Fix memory leaks on error paths.

Fixes Coverity resource leak defect. --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c |5 - 1 files changed, 4 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c b/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c index d4b8ae0..2442d78 100644 ---

[Mesa-dev] [PATCH] st/dri: Fix memory leak on error path.

Fixes Coverity resource leak defect. --- src/gallium/state_trackers/dri/drm/dri2.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/gallium/state_trackers/dri/drm/dri2.c b/src/gallium/state_trackers/dri/drm/dri2.c index a6bf3c1..4e3f106 100644 ---

[Mesa-dev] [PATCH] st/mesa: Fix memory leak on error path.

Fixes Coverity resource leak defect. --- src/mesa/state_tracker/st_program.c |4 +++- 1 files changed, 3 insertions(+), 1 deletions(-) diff --git a/src/mesa/state_tracker/st_program.c b/src/mesa/state_tracker/st_program.c index 146e77f..8cd039c 100644 ---

[Mesa-dev] [PATCH] u_vbuf: Silence uninitialized variable warnings.

Fixes these GCC warnings. u_vbuf.c: In function ‘u_vbuf_draw_begin’: u_vbuf.c:839:20: warning: ‘max_index’ may be used uninitialized in this function [-Wuninitialized] u_vbuf.c:838:20: warning: ‘min_index’ may be used uninitialized in this function [-Wuninitialized] Signed-off-by: Vinson Lee v

Re: [Mesa-dev] [PATCH 2/6] mesa: fix potential mem leak in generate_mipmap_compressed()

); return; } -- 1.7.3.4 ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev Reviewed-by: Vinson Lee v...@vmware.com ___ mesa

[Mesa-dev] [PATCH] gallivm: Fix build with llvm-3.1svn.

llvm-3.1svn r145714 moved global variables into a new TargetOptions class. TargetMachine constructor now needs a TargetOptions object as well. Signed-off-by: Vinson Lee v...@vmware.com --- src/gallium/auxiliary/gallivm/lp_bld_debug.cpp | 14 +- src/gallium/auxiliary/gallivm

[Mesa-dev] [PATCH] mesa: Fix memory leak on error path.

Fixes Coverity resource leak defect. Signed-off-by: Vinson Lee v...@vmware.com --- src/mesa/program/prog_parameter_layout.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/mesa/program/prog_parameter_layout.c b/src/mesa/program/prog_parameter_layout.c index 28fca3b

[Mesa-dev] [PATCH] st/mesa: Fix memory leak in out-of-memory path.

Fixes Coverity resource leak defect. Signed-off-by: Vinson Lee v...@vmware.com --- src/mesa/state_tracker/st_atom_pixeltransfer.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/src/mesa/state_tracker/st_atom_pixeltransfer.c b/src/mesa/state_tracker

Re: [Mesa-dev] [PATCH] Fix build with LLVM = r145623.

-Original Message- This is a workaround for https://bugs.freedesktop.org/show_bug.cgi?id=43861. Actually the issue which makes -pedantic failing should be solved. --- configure.ac |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/configure.ac

[Mesa-dev] [PATCH] mesa: Use __builtin_ffsll on Mac OS X.

Fixes this GCC warning. arrayobj.c: In function '_mesa_update_array_object_max_element': arrayobj.c:310: warning: implicit declaration of function 'ffsll' Signed-off-by: Vinson Lee v...@vmware.com --- src/mesa/main/imports.h |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

[Mesa-dev] [PATCH] mesa: Silence GCC unused variable warnings.

Silences these GCC warnings. warning: unused variable 'texelBytes' Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/texstore.c | 34 ++ 1 files changed, 34 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texstore.c b/src/mesa/main

[Mesa-dev] [PATCH] mesa: Silence GCC unused variable warnings.

Silences these GCC warnings. warning: unused variable 'texelBytes' Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/texstore.c | 34 ++ 1 files changed, 34 insertions(+), 0 deletions(-) diff --git a/src/mesa/main/texstore.c b/src/mesa/main

[Mesa-dev] [PATCH] mesa: Remove 'texelBytes' declarations that are only used in assertions.

This patch also silences these GCC warnings. warning: unused variable 'texelBytes' Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/texstore.c | 68 -- 1 files changed, 0 insertions(+), 68 deletions(-) diff --git a/src/mesa/main

[Mesa-dev] [PATCH] draw: Remove unused variables.

Fix this GCC warning. draw_pipe_clip.c: In function ‘interp’: draw_pipe_clip.c:122:13: warning: variable ‘clip_dist’ set but not used [-Wunused-but-set-variable] Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/draw/draw_pipe_clip.c |4 1 files changed, 0

Re: [Mesa-dev] [PATCH] Implementing varying packing

-Original Message- Subject: [Mesa-dev] [PATCH] Implementing varying packing This optimisation pass will look for and pack together float, vec2, vec3 varyings in fragment shaders and transform the vertex shader accordingly. It might improve performance depending on the hardware.

[Mesa-dev] [PATCH] nouveau: Fix typos of ParseSourceList in SConscript files.

--- src/gallium/drivers/nouveau/SConscript |2 +- src/gallium/drivers/nv50/SConscript|2 +- src/gallium/drivers/nvc0/SConscript|2 +- src/gallium/drivers/nvfx/SConscript|2 +- 4 files changed, 4 insertions(+), 4 deletions(-) diff --git

[Mesa-dev] [PATCH] st/mesa: Initialize variable.

ptr is uninitialized if ib in NULL. Fixes Coverity uninitialized pointer read defect. --- src/mesa/state_tracker/st_draw.c |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/src/mesa/state_tracker/st_draw.c b/src/mesa/state_tracker/st_draw.c index 20ba993..a2bff04 100644

[Mesa-dev] [PATCH] g3dvl: Remove unused loop variable.

Fixes Coverity uninitialized scalar variable defect. --- src/gallium/auxiliary/vl/vl_mpeg12_decoder.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c b/src/gallium/auxiliary/vl/vl_mpeg12_decoder.c index 7e7fa99..d4b8ae0

[Mesa-dev] [PATCH] pp: Fix memory leak on error path.

Fixes Coverity resource leak defect. --- src/gallium/auxiliary/postprocess/pp_program.c |3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/src/gallium/auxiliary/postprocess/pp_program.c b/src/gallium/auxiliary/postprocess/pp_program.c index c287af5..1a8a584 100644 ---

[Mesa-dev] [PATCH] tgsi: Fix memory leak in out-of-memory path.

Fixes Coverity resource leak defect. --- src/gallium/auxiliary/tgsi/tgsi_exec.c | 17 ++--- 1 files changed, 10 insertions(+), 7 deletions(-) diff --git a/src/gallium/auxiliary/tgsi/tgsi_exec.c b/src/gallium/auxiliary/tgsi/tgsi_exec.c index b4eea54..1fb7f8f 100644 ---

[Mesa-dev] [PATCH] i965: Fix assignment instead of comparison in asserts.

Fixes side effect in assertion defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i965/brw_fs_emit.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri/i965/brw_fs_emit.cpp b/src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH] configure.ac: Do not check for rt on Mac OS X.

There is no rt library on Mac OS X. Signed-off-by: Vinson Lee v...@freedesktop.org Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=58872 --- configure.ac | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 9cc5c4a

[Mesa-dev] [PATCH] glapi: Do not use backtrace on MinGW.

execinfo.h is not available on MinGW. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mapi/glapi/gen/gl_gentable.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mapi/glapi/gen/gl_gentable.py b/src/mapi/glapi/gen/gl_gentable.py index a00c9c2..a3c9898 100644

Re: [Mesa-dev] [PATCH] build: Fix no previous prototype for glReadBufferNV with --disable-asm

glapi/glapitemp.h +#endif #endif /* USE_X86_ASM */ -- 1.7.8.6 Tested-by: Vinson Lee v...@freedesktop.org ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 1/6] swrast: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/swrast/swrast.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/drivers/dri/swrast/swrast.c b/src/mesa/drivers/dri/swrast/swrast.c index 9638271..b380de3 100644

[Mesa-dev] [PATCH 2/6] nvc0: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nvc0/nvc0_vbo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nvc0/nvc0_vbo.c b/src/gallium/drivers/nvc0/nvc0_vbo.c index 61a6341..3ae437a 100644

[Mesa-dev] [PATCH 3/6] nv50: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nv50/nv50_vbo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nv50/nv50_vbo.c b/src/gallium/drivers/nv50/nv50_vbo.c index 6e81b7b..d050062 100644

[Mesa-dev] [PATCH 4/6] nv30: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nv30/nv30_vbo.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nv30/nv30_vbo.c b/src/gallium/drivers/nv30/nv30_vbo.c index 128457f..d7a774f 100644

[Mesa-dev] [PATCH 5/6] egl/dri2: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/egl/drivers/dri2/egl_dri2.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/egl/drivers/dri2/egl_dri2.c b/src/egl/drivers/dri2/egl_dri2.c index 351fbf4..ab06247 100644 --- a/src/egl

[Mesa-dev] [PATCH 6/6] r600g: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/r600_hw_context.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/r600_hw_context.c b/src/gallium/drivers/r600

[Mesa-dev] [PATCH] glsl: Initialize ast_parameter_declarator member variables.

Fixes uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/ast.h | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/glsl/ast.h b/src/glsl/ast.h index 1a28963..fcc6b45 100644 --- a/src/glsl/ast.h

[Mesa-dev] [PATCH] glsl: Initialize all tfeedback_candidate_generator member variables.

Fixes uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/link_varyings.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/glsl/link_varyings.cpp b/src/glsl/link_varyings.cpp index e2cb46e..b1317c8

[Mesa-dev] [PATCH] nv30: Fix memory leak.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nv30/nv30_screen.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/nv30/nv30_screen.c b/src/gallium/drivers/nv30/nv30_screen.c index f3faf8a..7825697

[Mesa-dev] [PATCH] intel: Ensure variable intel is used in i915 builds.

Fixes unused pointer value defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/intel/intel_tex_copy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/intel/intel_tex_copy.c b/src/mesa/drivers/dri/intel

[Mesa-dev] [PATCH] glsl: Ensure glsl_type constructors initialize gl_type.

Fixes uninitialized scalar field defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/glsl_types.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/glsl/glsl_types.cpp b/src/glsl/glsl_types.cpp index 4a2c879..3b066d0 100644 --- a/src/glsl

[Mesa-dev] [PATCH] configure.ac: Do not check for rt on Mac OS X.

There is no rt library on Mac OS X. Signed-off-by: Vinson Lee v...@freedesktop.org Bugzilla: https://bugs.freedesktop.org/show_bug.cgi?id=58872 --- configure.ac | 14 ++ 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/configure.ac b/configure.ac index 9cc5c4a

[Mesa-dev] [PATCH v2] configure.ac: Do not error out if librt is not available.

Some platforms, such as Mac OS X, do not have librt. Signed-off-by: Vinson Lee v...@freedesktop.org --- configure.ac | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 6735fdc..9b9a0ad 100644 --- a/configure.ac +++ b/configure.ac @@ -500,8

Re: [Mesa-dev] [PATCH v2] configure.ac: Do not error out if librt is not available.

On Sat, Feb 9, 2013 at 11:38 AM, Matt Turner matts...@gmail.com wrote: On Fri, Feb 8, 2013 at 7:19 PM, Vinson Lee v...@freedesktop.org wrote: Some platforms, such as Mac OS X, do not have librt. But Mac OS X (apparently from my quick Google search) doesn't clock_gettime. clock_gettime

[Mesa-dev] [PATCH] glsl: Initialize ast_function member variables.

Fixes uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/glsl_parser_extras.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glsl/glsl_parser_extras.cpp b/src/glsl/glsl_parser_extras.cpp index 7d826e3

[Mesa-dev] [PATCH] glsl: Initialize ir_texture member variable.

Fixes uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/ir.h | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/glsl/ir.h b/src/glsl/ir.h index efd80da..1e09988 100644 --- a/src/glsl/ir.h +++ b/src/glsl

[Mesa-dev] [PATCH] glsl: Initialize parcel_out_uniform_storage member variables.

Fixes uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/link_uniforms.cpp | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp index d457e4d..5e391ee

[Mesa-dev] [PATCH] configure.ac: Do not check for clock_gettime on MinGW.

MinGW does not have clock_gettime. Signed-off-by: Vinson Lee v...@freedesktop.org --- configure.ac | 2 ++ 1 file changed, 2 insertions(+) diff --git a/configure.ac b/configure.ac index 16c2f8c..57d3a70 100644 --- a/configure.ac +++ b/configure.ac @@ -502,6 +502,8 @@ AC_SUBST([DLOPEN_LIBS

[Mesa-dev] [PATCH v2] glsl: Initialize parcel_out_uniform_storage member variables.

Fixes uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/link_uniforms.cpp | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp index d457e4d..b5bfe13

[Mesa-dev] [PATCH] radeonsi: Fix memory leak in si_shader_select.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/radeonsi/si_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index d20e3ff

[Mesa-dev] [PATCH] r600g: Fix memory leak in r600_shader_select.

Signed-off-by: Vinson Lee v...@freedesktop.org Reported-by: Michel Dänzer mic...@daenzer.net --- src/gallium/drivers/r600/r600_state_common.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/r600/r600_state_common.c b/src/gallium/drivers/r600/r600_state_common.c index

[Mesa-dev] [PATCH] st/xvmc/tests: Ensure colorkey is initialized.

Fixes uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/state_trackers/xvmc/tests/xvmc_bench.c | 4 1 file changed, 4 insertions(+) diff --git a/src/gallium/state_trackers/xvmc/tests/xvmc_bench.c b/src/gallium

[Mesa-dev] [PATCH 2/2] st/vdpau: Fix memory leak in vlVdpBitmapSurfaceCreate.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/state_trackers/vdpau/bitmap.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/vdpau/bitmap.c b/src/gallium/state_trackers/vdpau/bitmap.c index ecc5b15

[Mesa-dev] [PATCH 1/2] st/vdpau: Fix memory leak in vlVdpOutputSurfaceCreate.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/state_trackers/vdpau/output.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/state_trackers/vdpau/output.c b/src/gallium/state_trackers/vdpau/output.c index a835126

[Mesa-dev] [PATCH v2] vl: Fix off-by-one error in device_name_length allocation.

Fixes out-of-bounds write reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/vl/vl_winsys_dri.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c b/src/gallium/auxiliary/vl/vl_winsys_dri.c

[Mesa-dev] [PATCH] st/vega: Fix memory leak in combine_shaders.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/state_trackers/vega/shaders_cache.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/state_trackers/vega/shaders_cache.c b/src/gallium/state_trackers

[Mesa-dev] [PATCH] radeonsi: Fix memory leak in si_set_constant_buffer.

Fixes resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/radeonsi/si_state.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeonsi/si_state.c b/src/gallium/drivers/radeonsi/si_state.c index 769ba0c

[Mesa-dev] [PATCH] mesa: Use correct functions for enum conversion.

Fixes mixing enum types defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/errors.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/mesa/main/errors.c b/src/mesa/main/errors.c index 97f1b8a..684f235 100644 --- a/src/mesa

[Mesa-dev] [PATCH] llvmpipe: Fix assertions with assignment instead of comparison.

Fixes assign instead of compare defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/llvmpipe/lp_state_fs.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/gallium/drivers/llvmpipe/lp_state_fs.c b/src/gallium/drivers

[Mesa-dev] [PATCH] tgsi: Ensure struct tgsi_ind_register field Index is initialized.

Fixes uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/tgsi/tgsi_build.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/auxiliary/tgsi/tgsi_build.c b/src/gallium/auxiliary/tgsi/tgsi_build.c index

[Mesa-dev] [PATCH] ilo: Add missing break statement in aos_tex TGSI_OPCODE_TEX2 case.

Fixes Missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/shader/toy_tgsi.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/ilo/shader/toy_tgsi.c b/src/gallium/drivers/ilo/shader/toy_tgsi.c index

[Mesa-dev] [PATCH v2] amd_performance_monitor: Fix multi-statement macro 'report'.

Fixes Nesting level does not match indentation defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- tests/spec/amd_performance_monitor/api.c | 6 -- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/tests/spec/amd_performance_monitor/api.c b/tests/spec

[Mesa-dev] [PATCH] ilo: Initialize read_back in transfer_map_sys.

Fixes Uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/ilo_resource.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/ilo/ilo_resource.c b/src/gallium/drivers/ilo

[Mesa-dev] [PATCH] scons: Use LLVM shared library if found.

This patch fixes SCons builds on Fedora 18. Signed-off-by: Vinson Lee v...@freedesktop.org --- scons/llvm.py | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/scons/llvm.py b/scons/llvm.py index 7cd609c..432ece6 100644 --- a/scons/llvm.py +++ b/scons/llvm.py @@ -198,7

[Mesa-dev] [PATCH] st/mesa: Remove unused variable 'texImage'.

All uses of 'texImage' were removed in commit 77a405dba7f70f8a47655e90774a5ecf5c88a6ed. Fixes Unused pointer value defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/state_tracker/st_cb_fbo.c | 4 1 file changed, 4 deletions(-) diff --git a/src/mesa

[Mesa-dev] [PATCH] gallivm: Fix build with LLVM = 3.4 r181680.

Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/gallivm/lp_bld_debug.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp b/src/gallium/auxiliary/gallivm/lp_bld_debug.cpp index 86617d4..1c886ea 100644

Re: [Mesa-dev] [PATCH] scons: Use LLVM shared library if found.

- This patch fixes SCons builds on Fedora 18. Signed-off-by: Vinson Lee v...@freedesktop.org --- scons/llvm.py | 10 +- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/scons/llvm.py b/scons/llvm.py index 7cd609c..432ece6 100644 --- a/scons/llvm.py +++ b/scons

[Mesa-dev] [PATCH] radeon: Initialize variables in radeon_llvm_context_init.

'type' was not fully initialized when calling lp_build_context_init. Fixes Uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/radeon/radeon_setup_tgsi_llvm.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src

[Mesa-dev] [PATCH] ilo: Initialize need_flush in draw_vbo.

need_flush was uninitialized if hw3d-new_batch was true. Fixes Uninitialized scalar variable defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/ilo_3d.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers

[Mesa-dev] [PATCH] glsl linker: Initialize member variable interface_namespace.

Fixes Uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/lower_named_interface_blocks.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glsl/lower_named_interface_blocks.cpp b/src/glsl

[Mesa-dev] [PATCH] mesa: Add missing break statement in _mesa_choose_tex_format.

Fixes Missing break in switch defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/texformat.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/mesa/main/texformat.c b/src/mesa/main/texformat.c index fda5d74..0ae79ea 100644 --- a/src/mesa/main

[Mesa-dev] [PATCH] ilo: Ensure array 'shaders' in fully initialized.

Fixes Uninitialized pointer read defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/ilo_state.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/ilo/ilo_state.c b/src/gallium/drivers/ilo/ilo_state.c

Re: [Mesa-dev] [PATCH] ilo: Ensure array 'shaders' in fully initialized.

On Fri, May 31, 2013 at 12:35 AM, Chia-I Wu olva...@gmail.com wrote: On Fri, May 31, 2013 at 2:59 PM, Vinson Lee v...@freedesktop.org wrote: Fixes Uninitialized pointer read defect reported by Coverity. This looks like a false alarm, as shaders are not read when num_shaders is zero. Does

Re: [Mesa-dev] [PATCH] ilo: Ensure array 'shaders' in fully initialized.

On Fri, May 31, 2013 at 1:09 AM, Chia-I Wu olva...@gmail.com wrote: On Fri, May 31, 2013 at 3:48 PM, Vinson Lee v...@freedesktop.org wrote: On Fri, May 31, 2013 at 12:35 AM, Chia-I Wu olva...@gmail.com wrote: On Fri, May 31, 2013 at 2:59 PM, Vinson Lee v...@freedesktop.org wrote: Fixes

[Mesa-dev] [PATCH] mesa: Prevent possible out-of-bounds read by save_SamplerParameterfv.

Fixes Out-of-bounds access defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/dlist.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c index c37b15e..6c26313 100644 --- a/src/mesa/main

[Mesa-dev] [PATCH] radeonsi/compute: Fix memory leak in radeonsi_launch_grid.

Fixes Resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/radeonsi/radeonsi_compute.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/gallium/drivers/radeonsi/radeonsi_compute.c b/src/gallium/drivers/radeonsi

[Mesa-dev] [PATCH] mesa: Prevent possible out-of-bounds read by save_SamplerParameteriv.

Fixes Out-of-bounds access defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/dlist.c | 5 - 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/src/mesa/main/dlist.c b/src/mesa/main/dlist.c index abc8665..8900c89 100644 --- a/src/mesa/main

[Mesa-dev] [PATCH] glsl: Fix null check in read_dereference.

Fixes Logically dead code defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/ir_reader.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/ir_reader.cpp b/src/glsl/ir_reader.cpp index b366712..51534ca 100644 --- a/src/glsl

[Mesa-dev] [PATCH] glsl: Initialize member variable is_ubo_var in constructor.

Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/link_uniforms.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/glsl/link_uniforms.cpp b/src/glsl/link_uniforms.cpp index 010296b..d42b6cd 100644

Re: [Mesa-dev] [PATCH] dlist: fix save_SamplerParameteri

) GLint parray[4]; parray[0] = param; parray[1] = parray[2] = parray[3] = 0; - save_SamplerParameteriv(sampler, pname, param); + save_SamplerParameteriv(sampler, pname, parray); } static void GLAPIENTRY D'oh! Reviewed-by: Brian Paul bri...@vmware.com Reviewed-by: Vinson

[Mesa-dev] [PATCH] glsl: Fix gl_shader_program::UniformLocationBaseScale assert.

. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/uniforms.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mesa/main/uniforms.h b/src/mesa/main/uniforms.h index 14fe26d..9223917 100644 --- a/src/mesa/main/uniforms.h +++ b/src/mesa/main/uniforms.h @@ -272,7

[Mesa-dev] [PATCH] ilo: Remove max_threads dead code path.

max_threads cannot be greater than 28. It is either 21 or 28. Fixes Logically dead code defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/ilo/ilo_gpe_gen6.c |3 --- 1 file changed, 3 deletions(-) diff --git a/src/gallium/drivers/ilo

[Mesa-dev] [PATCH] i915, i965: Fix memory leak in intel_miptree_create_for_bo.

Fixes Resource leak defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 4 +++- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri

[Mesa-dev] [PATCH] i965: Initialize brw_blorp_const_color_program member variables.

Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp b/src/mesa

[Mesa-dev] [PATCH] glsl: Initialize ast_function member variables.

Fixes uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/glsl_parser_extras.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glsl/glsl_parser_extras.cpp b/src/glsl/glsl_parser_extras.cpp index f4087df

[Mesa-dev] [PATCH] glapi: Do not use backtrace on OpenBSD.

execinfo.h is not available on OpenBSD. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mapi/glapi/gen/gl_gentable.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/mapi/glapi/gen/gl_gentable.py b/src/mapi/glapi/gen/gl_gentable.py index a3c9898..5c35271 100644

[Mesa-dev] [PATCH] glsl: Initialize ast_aggregate_initializer::constructor_type.

Fixes Uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/ast.h | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/glsl/ast.h b/src/glsl/ast.h index 9b119ed..c0350e7 100644 --- a/src/glsl/ast.h +++ b/src/glsl

[Mesa-dev] [PATCH] r600g/sb: Initialize ra_constraint::cost.

Fixes Uninitialized scalar field reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/sb/sb_shader.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/sb/sb_shader.h b/src/gallium/drivers/r600/sb/sb_shader.h

[Mesa-dev] [PATCH] r600g/sb: Initialize ra_checker member variables.

Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/r600/sb/sb_pass.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/drivers/r600/sb/sb_pass.h b/src/gallium/drivers/r600/sb

[Mesa-dev] [PATCH] nvc0/ir: Initialize NVC0LegalizePostRA member variables.

Fixes Uninitialized pointer field defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/drivers/nvc0/codegen/nv50_ir_lowering_nvc0.cpp | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/gallium/drivers/nvc0/codegen

[Mesa-dev] [PATCH] glsl: Initialize ast_jump_statement::opt_return_value.

opt_return_value was not initialized if mode != ast_return. Fixes Uninitialized pointer field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/glsl/glsl_parser_extras.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/src/glsl/glsl_parser_extras.cpp b/src

Re: [Mesa-dev] [PATCH] configure.ac: add OpenBSD

On Wed, Jun 26, 2013 at 12:11 AM, Jonathan Gray j...@jsg.id.au wrote: Signed-off-by: Jonathan Gray j...@jsg.id.au --- configure.ac | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git configure.ac configure.ac index 6832b0d..6a817d0 100644 --- configure.ac +++

Re: [Mesa-dev] [PATCH] configure.ac: make grep tests more portable

On Thu, Jun 20, 2013 at 3:14 AM, Jonathan Gray j...@jsg.id.au wrote: Use grep -w instead of the empty string escape sequences which are less portable. Makes the grep tests function as intended on OpenBSD. Signed-off-by: Jonathan Gray j...@jsg.id.au --- configure.ac | 6 +++--- 1 file

Re: [Mesa-dev] [PATCH] configure.ac: On some systems, x86-64 is called amd64

On Wed, Jun 26, 2013 at 6:46 AM, Jean-Sébastien Pédron jean-sebastien.ped...@dumbbell.fr wrote: On 26.06.2013 15:35, Jonathan Gray wrote: On Wed, Jun 26, 2013 at 02:54:34PM +0200, Jean-Sébastien Pédron wrote: Hello, On FreeBSD, the x86-64 arch is named amd64. This patch fixes its detection

[Mesa-dev] [PATCH] gallivm: Remove dead code in lp_build_compare_ext.

There are earlier returns for PIPE_FUNC_NEVER and PIPE_FUNC_ALWAYS. The switch value of 'func' cannot be either of those values. Fixes Logically dead code defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/gallivm/lp_bld_logic.c | 6 -- 1

Re: [Mesa-dev] [PATCH] draw: always call and move util_cpu_detect() to draw context creation.

= draw_llvm_create(draw); -- 1.7.9.5 With this patch, glxgears runs again for me with softpipe. Tested-by: Vinson Lee v...@freedesktop.org ___ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH] gallivm: Remove NoFramePointerElimNonLeaf for LLVM = 3.4.

TargetOptions::NoFramePointerElimNonLeaf was removed in LLVM 3.4 r187093. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/gallium/auxiliary/gallivm/lp_bld_misc.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/gallium/auxiliary/gallivm/lp_bld_misc.cpp b/src/gallium/auxiliary

[Mesa-dev] [PATCH] Fix generate_mipmap_compressed memory leaks.

Fixes Resource leak defects reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/main/mipmap.c | 4 1 file changed, 4 insertions(+) diff --git a/src/mesa/main/mipmap.c b/src/mesa/main/mipmap.c index 5839632..0ef6536 100644 --- a/src/mesa/main/mipmap.c +++ b/src

[Mesa-dev] [PATCH] i965: Initialize brw_blorp_const_color_program::prog_data.

Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp b/src/mesa/drivers

Re: [Mesa-dev] [PATCH] i965: Initialize brw_blorp_const_color_program::prog_data.

On Thu, Jul 25, 2013 at 11:19 PM, Kenneth Graunke kenn...@whitecape.org wrote: On 07/25/2013 10:48 PM, Vinson Lee wrote: Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 3

[Mesa-dev] [PATCH v2] i965: Initialize brw_blorp_const_color_program::prog_data.

Fixes Uninitialized scalar field defect reported by Coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i965/brw_blorp_clear.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/mesa/drivers/dri/i965/brw_blorp_clear.cpp b/src/mesa/drivers/dri/i965

[Mesa-dev] [PATCH v2] i915, i965: Fix memory leak in intel_miptree_create_for_bo.

Fixes Resource leak defects reported by coverity. Signed-off-by: Vinson Lee v...@freedesktop.org --- src/mesa/drivers/dri/i915/intel_mipmap_tree.c | 4 +++- src/mesa/drivers/dri/i965/intel_mipmap_tree.c | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/mesa/drivers/dri

Re: [Mesa-dev] [PATCH] mesa: improve free() cleanup in generate_mipmap_compressed()

On Fri, Jul 26, 2013 at 8:54 AM, Brian Paul bri...@vmware.com wrote: Free all our temporary buffers in one place at the end of the function. Fixes memory leak detected by Coverity. Note: This is a candidate for the 9.x branches Cc: mesa-sta...@lists.freedesktop.org ---

  1   2   3   4   5   >