aboutsummaryrefslogtreecommitdiffstats
path: root/libavutil/hwcontext_vulkan.c
diff options
context:
space:
mode:
authorLynne <dev@lynne.ee>2023-06-13 04:36:54 +0200
committerLynne <dev@lynne.ee>2023-06-15 22:00:41 +0200
commitd0f1d937fea514abb6e62ff9949eb0b304f99231 (patch)
tree7ed40ab28128044b1b1694e47dacfdbe75527a8d /libavutil/hwcontext_vulkan.c
parentb4d5baa8b05974a406615925f379b56fa1c850a6 (diff)
downloadffmpeg-d0f1d937fea514abb6e62ff9949eb0b304f99231.tar.gz
hwcontext_vulkan: free temporary array once unneeded
Fixes a small memory leak. This also prevents leaks on malloc/mutex init errors.
Diffstat (limited to 'libavutil/hwcontext_vulkan.c')
-rw-r--r--libavutil/hwcontext_vulkan.c11
1 files changed, 9 insertions, 2 deletions
diff --git a/libavutil/hwcontext_vulkan.c b/libavutil/hwcontext_vulkan.c
index c86229ba65..ca802cc86e 100644
--- a/libavutil/hwcontext_vulkan.c
+++ b/libavutil/hwcontext_vulkan.c
@@ -1427,24 +1427,31 @@ static int vulkan_device_init(AVHWDeviceContext *ctx)
vk->GetPhysicalDeviceQueueFamilyProperties(hwctx->phys_dev, &qf_num, qf);
p->qf_mutex = av_calloc(qf_num, sizeof(*p->qf_mutex));
- if (!p->qf_mutex)
+ if (!p->qf_mutex) {
+ av_free(qf);
return AVERROR(ENOMEM);
+ }
p->nb_tot_qfs = qf_num;
for (uint32_t i = 0; i < qf_num; i++) {
p->qf_mutex[i] = av_calloc(qf[i].queueCount, sizeof(**p->qf_mutex));
- if (!p->qf_mutex[i])
+ if (!p->qf_mutex[i]) {
+ av_free(qf);
return AVERROR(ENOMEM);
+ }
for (uint32_t j = 0; j < qf[i].queueCount; j++) {
err = pthread_mutex_init(&p->qf_mutex[i][j], NULL);
if (err != 0) {
av_log(ctx, AV_LOG_ERROR, "pthread_mutex_init failed : %s\n",
av_err2str(err));
+ av_free(qf);
return AVERROR(err);
}
}
}
+ av_free(qf);
+
graph_index = hwctx->queue_family_index;
comp_index = hwctx->queue_family_comp_index;
tx_index = hwctx->queue_family_tx_index;