[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/4] plugins: Fixes a issue when dlsym failed, the handle not
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 1/4] plugins: Fixes a issue when dlsym failed, the handle not closed. |
Date: |
Sat, 26 Sep 2020 22:30:04 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.11.0 |
On 9/26/20 4:04 PM, Yonggang Luo wrote:
> Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
Superfluous trailing '.' in patch subject.
Reviewed-by: Philippe Mathieu-Daudé <f4bug@amsat.org>
> ---
> plugins/loader.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/plugins/loader.c b/plugins/loader.c
> index 685d334e1a..8ac5dbc20f 100644
> --- a/plugins/loader.c
> +++ b/plugins/loader.c
> @@ -235,6 +235,7 @@ static int plugin_load(struct qemu_plugin_desc *desc,
> const qemu_info_t *info)
> return rc;
>
> err_symbol:
> + g_module_close(ctx->handle);
> err_dlopen:
> qemu_vfree(ctx);
> return 1;
>
- [PATCH 0/4] Enable plugin support on msys2/mingw, Yonggang Luo, 2020/09/26
- [PATCH 1/4] plugins: Fixes a issue when dlsym failed, the handle not closed., Yonggang Luo, 2020/09/26
- Re: [PATCH 1/4] plugins: Fixes a issue when dlsym failed, the handle not closed.,
Philippe Mathieu-Daudé <=
- [PATCH 2/4] plugin: Fixes compiling errors on msys2/mingw, Yonggang Luo, 2020/09/26
- [PATCH 3/4] Getting qemu-plugin works under win32., Yonggang Luo, 2020/09/26
- [PATCH 4/4] cirrus: Enable plugin in cirrus for windows, Yonggang Luo, 2020/09/26