[ovs-dev] [PATCH ovn 13/14] ovn-nbctl: Fix IP leak on failure of lr policy addition.
Ilya Maximets
i.maximets at ovn.org
Fri Nov 20 17:52:50 UTC 2020
On 11/20/20 6:12 PM, Dumitru Ceara wrote:
> On 11/20/20 1:17 AM, Ilya Maximets wrote:
>> Fixes: 742474bad730 ("ovn-nbctl: Enhance lr-policy-add to set the options.")
>> Signed-off-by: Ilya Maximets <i.maximets at ovn.org>
>> ---
>> utilities/ovn-nbctl.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/utilities/ovn-nbctl.c b/utilities/ovn-nbctl.c
>> index 526dbf86c..448c86f42 100644
>> --- a/utilities/ovn-nbctl.c
>> +++ b/utilities/ovn-nbctl.c
>> @@ -3689,6 +3689,7 @@ nbctl_lr_policy_add(struct ctl_context *ctx)
>> } else {
>> ctl_error(ctx, "No value specified for the option : %s", key);
>> free(key);
>> + free(next_hop);
>
> Looks like we also need to:
>
> smap_destroy(&options);
Good catch. We also need to free 'value', I guess, because an empty
string still consumes some memory (at least one byte).
This might be fixed in a separate patch, though.
Dumitru, Numan, what do you think?
Best regards, Ilya Maximets.
More information about the dev
mailing list