Hi yanlei,
Thanks for your patch.
On 2022/3/15 20:18, yan wrote:
1> From: Yan Lei yan_lei@dahuatech.com
mainline-inclusion
该补丁是你自己开发的，不是主线 backport 的，不要标成 mainline inclusion, 你可以写成 driver inclusioin, 或者 dahua inclusion 都可以。
from mainline-openEuler-22.03-LTS
自己开发的补丁，不需要这一行。
category: bugfix Subject: [PATCH openEuler-22.03-LTS] fix CWE-457 bug#1
Subject 是补丁标题，不是标在这里的。使用 git format-patch 生成的补丁， 应该格式就 OK 了。
Declaring variable "tuner_status" without initializer. Using uninitialized value "tuner_status" when calling "*fe_tuner_ops->get_status". (The function pointer resolves to "cx24113_get_status".)
该补丁，确实 fix 了这特场景的初始化问题。 我看社区主线也有这个问。
建议也同时往社区发一下。
这个补丁格式修改一下，再往openEuler发一版本吧。
Signed-off-by: Yan Lei yan_lei@dahuatech.com
drivers/media/v4l2-core/tuner-core.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) mode change 100644 => 100755 drivers/media/v4l2-core/tuner-core.c
diff --git a/drivers/media/v4l2-core/tuner-core.c b/drivers/media/v4l2-core/tuner-core.c index 12d1e0c..15bcde4 --- a/drivers/media/v4l2-core/tuner-core.c +++ b/drivers/media/v4l2-core/tuner-core.c @@ -1118,7 +1118,7 @@ static void tuner_status(struct dvb_frontend *fe) if (t->mode != V4L2_TUNER_RADIO) return; if (fe_tuner_ops->get_status) {
u32 tuner_status;
u32 tuner_status = 0;fe_tuner_ops->get_status(&t->fe, &tuner_status); if (tuner_status & TUNER_STATUS_LOCKED)
@@ -1258,7 +1258,7 @@ static int tuner_g_tuner(struct v4l2_subdev *sd, struct v4l2_tuner *vt) if (vt->type == t->mode) { vt->rxsubchans = V4L2_TUNER_SUB_MONO | V4L2_TUNER_SUB_STEREO; if (fe_tuner_ops->get_status) {
u32 tuner_status;
u32 tuner_status = 0; fe_tuner_ops->get_status(&t->fe, &tuner_status); vt->rxsubchans =-- 2.7.4