media: cedrus: hevc: Add check for invalid timestamp
authorJernej Skrabec <jernej.skrabec@gmail.com>
Mon, 18 Jul 2022 16:56:49 +0000 (18:56 +0200)
committerMauro Carvalho Chehab <mchehab@kernel.org>
Wed, 27 Jul 2022 18:27:10 +0000 (20:27 +0200)
commit143201a6435bf65f0115435e9dc6d95c66b908e9
treed29bbcd24e48d0148c94eb6ef434363cb8dfcfdc
parente39cc4960207a67a26b39ebc8449f15c53da0a99
media: cedrus: hevc: Add check for invalid timestamp

Not all DPB entries will be used most of the time. Unused entries will
thus have invalid timestamps. They will produce negative buffer index
which is not specifically handled. This works just by chance in current
code. It will even produce bogus pointer, but since it's not used, it
won't do any harm.

Let's fix that brittle design by skipping writing DPB entry altogether
if timestamp is invalid.

Fixes: 86caab29da78 ("media: cedrus: Add HEVC/H.265 decoding support")
Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Reviewed-by: Ezequiel Garcia <ezequiel@vanguardiasur.com.ar>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: Mauro Carvalho Chehab <mchehab@kernel.org>
drivers/staging/media/sunxi/cedrus/cedrus_h265.c