diff options
| author | Gert Wollny <gert.wollny@collabora.com> | 2022-11-08 14:28:37 +0100 |
|---|---|---|
| committer | Marge Bot <emma+marge@anholt.net> | 2022-11-10 07:03:02 +0000 |
| commit | d576e123751fd57bcf5dad5c35ead6e4e0f9a47e (patch) | |
| tree | 3d64c89827a7b8af41739bfc0b35d6adb4612506 | |
| parent | 876b048d9234b278366ee3176e2ff24f70bbaec2 (diff) | |
| download | virglrenderer-d576e123751fd57bcf5dad5c35ead6e4e0f9a47e.tar.gz | |
vtest: don't double free wait in vtest_sync_wait_init
vtest_sync_wait already frees wait on failure.
Fixes Coverity ID: 1527210 "Double free"
Signed-off-by: Gert Wollny <gert.wollny@collabora.com>
Part-of: <https://gitlab.freedesktop.org/virgl/virglrenderer/-/merge_requests/975>
| -rw-r--r-- | vtest/vtest_renderer.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/vtest/vtest_renderer.c b/vtest/vtest_renderer.c index 913f579a..7041b985 100644 --- a/vtest/vtest_renderer.c +++ b/vtest/vtest_renderer.c @@ -1983,10 +1983,8 @@ int vtest_sync_wait(uint32_t length_dw) sync_wait_buf + 2, sync_count); free(sync_wait_buf); - if (ret) { - free(wait); + if (ret) return ret; - } is_ready = !wait->count; if ((wait->flags & VCMD_SYNC_WAIT_FLAG_ANY) && wait->count < sync_count) |