[Ocfs2-tools-devel] [PATCH] o2cb_ctl: free 'cf' at the end of j_config_free()

[Joseph Qi]

Looks good to me, thanks.

Reviewed-by: Joseph Qi <jiangqi903 at gmail.com>


On 16/11/18 19:15, piaojun wrote:
> Hi,
>
> Could anyone help reviewing my patch?
>
> thanks,
> Jun
>
> On 2016-10-17 10:43, piaojun wrote:
>> 'cf' need to be freed at the end of j_config_free().
>>
>> Signed-off-by: Jun Piao <piaojun at huawei.com>
>> ---
>>   o2cb_ctl/o2cb_config.c | 1 +
>>   1 file changed, 1 insertion(+)
>>
>> diff --git a/o2cb_ctl/o2cb_config.c b/o2cb_ctl/o2cb_config.c
>> index 17749f8..2bb1094 100644
>> --- a/o2cb_ctl/o2cb_config.c
>> +++ b/o2cb_ctl/o2cb_config.c
>> @@ -549,6 +549,7 @@ gint o2cb_config_store(O2CBConfig *config, const gchar *filename)
>>       g_free(text);
>>   
>>   out:
>> +    j_config_free(cf);
>>       return rc;
>>   }  /* o2cb_config_store() */
>>   
>>
>
> _______________________________________________
> Ocfs2-tools-devel mailing list
> Ocfs2-tools-devel at oss.oracle.com
> https://oss.oracle.com/mailman/listinfo/ocfs2-tools-devel