Re: [RESEND PATCH] ceph: Fix memory leak in chain_listxattr

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Li,

Applied both of these.  THanks!

sage

On Wed, 30 Oct 2013, Li Wang wrote:

> Free allocated memory before return
> 
> Signed-off-by: Li Wang <liwang@xxxxxxxxxxxxxxx>
> ---
>  src/os/chain_xattr.cc |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/os/chain_xattr.cc b/src/os/chain_xattr.cc
> index 62733e3..52ae8db 100644
> --- a/src/os/chain_xattr.cc
> +++ b/src/os/chain_xattr.cc
> @@ -355,7 +355,7 @@ int chain_listxattr(const char *fn, char *names, size_t len) {
>  
>    r = sys_listxattr(fn, full_buf, total_len);
>    if (r < 0)
> -    return r;
> +    goto done;
>  
>    char *p = full_buf;
>    const char *end = full_buf + r;
> -- 
> 1.7.9.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 
> 
--
To unsubscribe from this list: send the line "unsubscribe ceph-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [CEPH Users]     [Ceph Large]     [Information on CEPH]     [Linux BTRFS]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux