This is an automatic generated email to let you know that the following patch 
were queued:

Subject: media: mtk-vcodec: vdec: fix incorrect pointer dereference
Author:  Yunfei Dong <yunfei.d...@mediatek.com>
Date:    Fri Sep 6 08:55:01 2019 -0300

mtk_q_data::fmt is actually a pointer and must be dereferenced as such.
This went under the radar because mtk_v4l2_debug() evaluates to nothing
unless DEBUG is defined.

[acourbot: split into its own commit]

Signed-off-by: Yunfei Dong <yunfei.d...@mediatek.com>
Signed-off-by: Alexandre Courbot <acour...@chromium.org>
Signed-off-by: Hans Verkuil <hverkuil-ci...@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab+sams...@kernel.org>

 drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

---

diff --git a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c 
b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
index 26a55c3e807e..653f111532ab 100644
--- a/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
+++ b/drivers/media/platform/mtk-vcodec/mtk_vcodec_dec.c
@@ -284,7 +284,7 @@ static void mtk_vdec_update_fmt(struct mtk_vcodec_ctx *ctx,
                fmt = &mtk_video_formats[k];
                if (fmt->fourcc == pixelformat) {
                        mtk_v4l2_debug(1, "Update cap fourcc(%d -> %d)",
-                               dst_q_data->fmt.fourcc, pixelformat);
+                               dst_q_data->fmt->fourcc, pixelformat);
                        dst_q_data->fmt = fmt;
                        return;
                }

_______________________________________________
linuxtv-commits mailing list
linuxtv-commits@linuxtv.org
https://www.linuxtv.org/cgi-bin/mailman/listinfo/linuxtv-commits

Reply via email to