diff options
| author | Vaibhav Agarwal <[email protected]> | 2020-08-14 18:03:15 +0530 |
|---|---|---|
| committer | Greg Kroah-Hartman <[email protected]> | 2020-08-18 16:12:54 +0200 |
| commit | 1dffeb8b8b4c261c45416d53c75ea51e6ece1770 (patch) | |
| tree | 5ef7ae4e25d228945b541f8059c1eec7b349333d /lib/pm-notifier-error-inject.c | |
| parent | fea22e159d51c766ba70473f473a0ec914cc7e92 (diff) | |
staging: greybus: audio: fix uninitialized value issue
The current implementation for gbcodec_mixer_dapm_ctl_put() uses
uninitialized gbvalue for comparison with updated value. This was found
using static analysis with coverity.
Uninitialized scalar variable (UNINIT)
11. uninit_use: Using uninitialized value
gbvalue.value.integer_value[0].
460 if (gbvalue.value.integer_value[0] != val) {
This patch fixes the issue with fetching the gbvalue before using it for
comparision.
Fixes: 6339d2322c47 ("greybus: audio: Add topology parser for GB codec")
Reported-by: Colin Ian King <[email protected]>
Signed-off-by: Vaibhav Agarwal <[email protected]>
Cc: stable <[email protected]>
Link: https://lore.kernel.org/r/bc4f29eb502ccf93cd2ffd98db0e319fa7d0f247.1597408126.git.vaibhav.sr@gmail.com
Signed-off-by: Greg Kroah-Hartman <[email protected]>
Diffstat (limited to 'lib/pm-notifier-error-inject.c')
0 files changed, 0 insertions, 0 deletions