diff mbox

9pfs: xattr: fix memory leak in v9fs_list_xattr

Message ID 1491562132-69066-1-git-send-email-liqiang6-s@360.cn (mailing list archive)
State New, archived
Headers show

Commit Message

Li Qiang April 7, 2017, 10:48 a.m. UTC 
Free 'orig_value' in error path.

Signed-off-by: Li Qiang <liqiang6-s@360.cn>
---
 hw/9pfs/9p-xattr.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Greg Kurz April 10, 2017, 9:14 a.m. UTC | #1 
On Fri,  7 Apr 2017 03:48:52 -0700
Li Qiang <liq3ea@gmail.com> wrote:

> Free 'orig_value' in error path.
> 
> Signed-off-by: Li Qiang <liqiang6-s@360.cn>
> ---

Good catch. I'll send a pull req later today.

Cheers.

--
Greg

>  hw/9pfs/9p-xattr.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/9pfs/9p-xattr.c b/hw/9pfs/9p-xattr.c
> index eec160b..d05c1a1 100644
> --- a/hw/9pfs/9p-xattr.c
> +++ b/hw/9pfs/9p-xattr.c
> @@ -108,6 +108,7 @@ ssize_t v9fs_list_xattr(FsContext *ctx, const char *path,
>      g_free(name);
>      close_preserve_errno(dirfd);
>      if (xattr_len < 0) {
> +        g_free(orig_value);
>          return -1;
>      }
>
diff mbox

Patch

diff --git a/hw/9pfs/9p-xattr.c b/hw/9pfs/9p-xattr.c
index eec160b..d05c1a1 100644
--- a/hw/9pfs/9p-xattr.c
+++ b/hw/9pfs/9p-xattr.c
@@ -108,6 +108,7 @@  ssize_t v9fs_list_xattr(FsContext *ctx, const char *path,
     g_free(name);
     close_preserve_errno(dirfd);
     if (xattr_len < 0) {
+        g_free(orig_value);
         return -1;
     }