diff mbox series

[1/4] plugins: Fixes a issue when dlsym failed, the handle not closed.

Message ID 20200926140415.1286-2-luoyonggang@gmail.com
State New
Headers show
Series Enable plugin support on msys2/mingw | expand

Commit Message

罗勇刚(Yonggang Luo) Sept. 26, 2020, 2:04 p.m. UTC 
Signed-off-by: Yonggang Luo <luoyonggang@gmail.com>
---
 plugins/loader.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Philippe Mathieu-Daudé Sept. 26, 2020, 8:30 p.m. UTC | #1 
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;

>
diff mbox series

Patch

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;