diff options
| author | Colin Ian King <colin.king@canonical.com> | 2021-01-15 11:11:26 +0100 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-05-02 07:41:21 +0200 |
| commit | 2b524801df52c50c244cf32e1037a68d391f6730 (patch) | |
| tree | ab4213b3a06de45a839a5ebb53109b96abdefdfd | |
| parent | c0c89fed99caadbf0213380547e0c8913baf08a6 (diff) | |
media: venus: Fix uninitialized variable count being checked for zero
commit c5b14df7a80acadbbd184808dc3f519fbb2ab96c upstream.
In the case where plat->codecs is NULL the variable count is uninitialized
but is being checked to see if it is 0. Fix this by initializing
count to 0.
Addresses-Coverity: ("Uninitialized scalar variable")
Fixes: e29929266be1 ("media: venus: Get codecs and capabilities from hfi platform")
Signed-off-by: Colin Ian King <colin.king@canonical.com>
Signed-off-by: Stanimir Varbanov <stanimir.varbanov@linaro.org>
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@kernel.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
| -rw-r--r-- | drivers/media/platform/qcom/venus/hfi_parser.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/qcom/venus/hfi_parser.c b/drivers/media/platform/qcom/venus/hfi_parser.c index c6be1564c876..cb7645f88a46 100644 --- a/drivers/media/platform/qcom/venus/hfi_parser.c +++ b/drivers/media/platform/qcom/venus/hfi_parser.c @@ -266,7 +266,7 @@ static int hfi_platform_parser(struct venus_core *core, struct venus_inst *inst) { const struct hfi_platform *plat; const struct hfi_plat_caps *caps = NULL; - u32 enc_codecs, dec_codecs, count; + u32 enc_codecs, dec_codecs, count = 0; unsigned int entries; if (inst) |