diff options
| author | Hans Verkuil <hverkuil+cisco@kernel.org> | 2026-05-12 08:46:15 +0200 |
|---|---|---|
| committer | Hans Verkuil <hverkuil+cisco@kernel.org> | 2026-05-12 09:39:03 +0200 |
| commit | 813dacdda8f2e05b1309a4942e91531333b296ed (patch) | |
| tree | 764a8efa76ec310f2cfd4c82c35cdf13f33473f3 | |
| parent | bbba3e260a62810a717b4442a3bb96d0ec0f6309 (diff) | |
media: visl: check if ctx->tpg_str_buf allocation failed
The result of ctx->tpg_str_buf = kzalloc(TPG_STR_BUF_SZ, GFP_KERNEL); was
never checked. Add this.
Signed-off-by: Hans Verkuil <hverkuil+cisco@kernel.org>
| -rw-r--r-- | drivers/media/test-drivers/visl/visl-core.c | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/drivers/media/test-drivers/visl/visl-core.c b/drivers/media/test-drivers/visl/visl-core.c index 127ab18bce99..5e9bdd425d4a 100644 --- a/drivers/media/test-drivers/visl/visl-core.c +++ b/drivers/media/test-drivers/visl/visl-core.c @@ -339,6 +339,10 @@ static int visl_open(struct file *file) } ctx->tpg_str_buf = kzalloc(TPG_STR_BUF_SZ, GFP_KERNEL); + if (!ctx->tpg_str_buf) { + rc = -ENOMEM; + goto free_ctx; + } v4l2_fh_init(&ctx->fh, video_devdata(file)); ctx->dev = dev; |