media: cedrus: h265: Associate mv col buffers with buffer

Browse source

Currently mv col aux buffers are allocated as a pool. This is not
optimal because pool size equals number of buffers before stream is
started. Buffers can easily be allocated afterwards. In such cases,
invalid pointer is assigned to the decoding frame and Cedrus might
overwrite memory location which is allocated to different task.

Solve this issue with allocating mv col buffer once capture buffer is
actually used.

Signed-off-by: Jernej Skrabec <jernej.skrabec@gmail.com>
Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl>

...

This commit is contained in:

Jernej Skrabec 2022-10-19 19:45:50 +02:00 • committed by Hans Verkuil

parent d555409dd1

commit 0ee952c2f4

2 changed files with 38 additions and 34 deletions

Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL

Download patch file Download diff file Expand all files Collapse all files

9

drivers/staging/media/sunxi/cedrus/cedrus.h

View file

63

drivers/staging/media/sunxi/cedrus/cedrus_h265.c

View file