mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-10-03 07:38:10 +00:00
media: coda: jpeg: Add check for kmalloc
[ Upstream commitf30ce3d376
] As kmalloc can return NULL pointer, it should be better to check the return value and return error, same as coda_jpeg_decode_header. Fixes:96f6f62c46
("media: coda: jpeg: add CODA960 JPEG encoder support") Signed-off-by: Jiasheng Jiang <jiasheng@iscas.ac.cn> Signed-off-by: Hans Verkuil <hverkuil-cisco@xs4all.nl> Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
parent
9a402adc9f
commit
aa81257dbf
1 changed files with 8 additions and 2 deletions
|
@ -1052,10 +1052,16 @@ static int coda9_jpeg_start_encoding(struct coda_ctx *ctx)
|
|||
v4l2_err(&dev->v4l2_dev, "error loading Huffman tables\n");
|
||||
return ret;
|
||||
}
|
||||
if (!ctx->params.jpeg_qmat_tab[0])
|
||||
if (!ctx->params.jpeg_qmat_tab[0]) {
|
||||
ctx->params.jpeg_qmat_tab[0] = kmalloc(64, GFP_KERNEL);
|
||||
if (!ctx->params.jpeg_qmat_tab[1])
|
||||
if (!ctx->params.jpeg_qmat_tab[0])
|
||||
return -ENOMEM;
|
||||
}
|
||||
if (!ctx->params.jpeg_qmat_tab[1]) {
|
||||
ctx->params.jpeg_qmat_tab[1] = kmalloc(64, GFP_KERNEL);
|
||||
if (!ctx->params.jpeg_qmat_tab[1])
|
||||
return -ENOMEM;
|
||||
}
|
||||
coda_set_jpeg_compression_quality(ctx, ctx->params.jpeg_quality);
|
||||
|
||||
return 0;
|
||||
|
|
Loading…
Reference in a new issue