Re: [PATCH v6] btrfs: Fix memory leakage in the tree-log.c

2013-10-14 Thread Josef Bacik
On Sun, Oct 13, 2013 at 07:03:12PM -0300, Geyslan Gregório Bem wrote: 2013/10/11 Stefan Behrens sbehr...@giantdisaster.de: On 10/11/2013 20:35, Geyslan G. Bem wrote: In add_inode_ref() function: Initializes local pointers. Reduces the logical condition with the __add_inode_ref()

Re: [PATCH v6] btrfs: Fix memory leakage in the tree-log.c

2013-10-14 Thread Geyslan Gregório Bem
2013/10/14 Josef Bacik jba...@fusionio.com: On Sun, Oct 13, 2013 at 07:03:12PM -0300, Geyslan Gregório Bem wrote: 2013/10/11 Stefan Behrens sbehr...@giantdisaster.de: On 10/11/2013 20:35, Geyslan G. Bem wrote: In add_inode_ref() function: Initializes local pointers. Reduces the

Re: [PATCH v6] btrfs: Fix memory leakage in the tree-log.c

2013-10-13 Thread Geyslan Gregório Bem
2013/10/11 Stefan Behrens sbehr...@giantdisaster.de: On 10/11/2013 20:35, Geyslan G. Bem wrote: In add_inode_ref() function: Initializes local pointers. Reduces the logical condition with the __add_inode_ref() return value by using only one 'goto out'. Centralizes the exiting, ensuring

[PATCH v6] btrfs: Fix memory leakage in the tree-log.c

2013-10-11 Thread Geyslan G. Bem
In add_inode_ref() function: Initializes local pointers. Reduces the logical condition with the __add_inode_ref() return value by using only one 'goto out'. Centralizes the exiting, ensuring the freeing of all used memory. Signed-off-by: Geyslan G. Bem geys...@gmail.com ---

Re: [PATCH v6] btrfs: Fix memory leakage in the tree-log.c

2013-10-11 Thread Stefan Behrens
On 10/11/2013 20:35, Geyslan G. Bem wrote: In add_inode_ref() function: Initializes local pointers. Reduces the logical condition with the __add_inode_ref() return value by using only one 'goto out'. Centralizes the exiting, ensuring the freeing of all used memory. Signed-off-by: Geyslan G.