OSDN Git Service

media: coda: don't overwrite h.264 profile_idc on decoder instance
authorLucas Stach <l.stach@pengutronix.de>
Wed, 1 Aug 2018 14:18:04 +0000 (10:18 -0400)
committerMauro Carvalho Chehab <mchehab+samsung@kernel.org>
Mon, 24 Sep 2018 13:30:54 +0000 (09:30 -0400)
commit1f32061e843205f6fe8404d5100d5adcec334e75
treee322f7cf4fa87d94670a6afc2968ff6deb08b1fe
parent8ee92410e5bf29b7a2fcaf53ca241554c4bae7a9
media: coda: don't overwrite h.264 profile_idc on decoder instance

On a decoder instance, after the profile has been parsed from the stream
__v4l2_ctrl_s_ctrl() is called to notify userspace about changes in the
read-only profile control. This ends up calling back into the CODA driver
where a missing check on the s_ctrl caused the profile information that has
just been parsed from the stream to be overwritten with the default
baseline profile.

Later on the driver fails to enable frame reordering, based on the wrong
profile information.

Fixes: 347de126d1da (media: coda: add read-only h.264 decoder
                     profile/level controls)

Signed-off-by: Lucas Stach <l.stach@pengutronix.de>
Reviewed-by: Philipp Zabel <p.zabel@pengutronix.de>
Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab+samsung@kernel.org>
drivers/media/platform/coda/coda-common.c