aboutsummaryrefslogtreecommitdiff
path: root/render/gles2/util.c
diff options
context:
space:
mode:
authorrandom human <random.bored.human@gmail.com>2018-08-31 16:11:46 +0530
committeremersion <contact@emersion.fr>2018-08-31 19:41:18 +0200
commit8589ae19de1d5699a586f96fe34a35c586e1e29c (patch)
tree25cd281dba819188489689df6957e42416f834cb /render/gles2/util.c
parent2f484537181c515082b632f9d575613abda72a48 (diff)
Fix bugs listed by clang's static analyzer
A few pedantic changes and unused variables (1-4), and genuine bugs (5, 6). The reports with the corresponding files and lines numbers are as follows. 1. backend/libinput/tablet_pad.c@31,44,57 "Allocator sizeof operand mismatch" "Result of 'calloc' is converted to a pointer of type 'unsigned int', which is incompatible with sizeof operand type 'int'" 2. types/tablet_v2/wlr_tablet_v2_pad.c@371 "Allocator sizeof operand mismatch" "Result of 'calloc' is converted to a pointer of type 'uint32_t', which is incompatible with sizeof operand type 'int'" 3. types/wlr_cursor.c@335 "Dead initialization" "Value stored to 'dx'/'dy' during its initialization is never read" 4. rootston/xdg_shell.c@510 "Dead initialization" "Value stored to 'desktop' during its initialization is never read" 5. types/tablet_v2/wlr_tablet_v2_pad.c@475 "Dereference of null pointer" "Access to field 'strips' results in a dereference of a null pointer (loaded from field 'current_client')" The boolean logic was incorrect (c.f. the check in the following function). 6. examples/idle.c@163,174,182 "Uninitialized argument value" "1st function call argument is an uninitialized value" If close_timeout != 0, but simulate_activity_timeout >= close_timeout, the program would segfault at pthread_cancel(t1).
Diffstat (limited to 'render/gles2/util.c')
0 files changed, 0 insertions, 0 deletions