Hi,

On Wed, Jul 14, 2021 at 02:46:42PM +0100, David Korczynski wrote:
> Signed-off-by: David Korczynski <da...@adalogics.com>

This is good :-)

> ---
>  src/openvpn/route.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/openvpn/route.c b/src/openvpn/route.c
> index 3b2af8b4..e429e8c0 100644
> --- a/src/openvpn/route.c
> +++ b/src/openvpn/route.c
> @@ -1584,6 +1584,7 @@ add_route(struct route_ipv4 *r,
> 
>      if (!(r->flags & RT_DEFINED))
>      {
> +        argv_free(&argv);
>          return;
>      }

... as is this.  Can you send a single patch that contains both fixes?

We try to make patch granularity a good compromise - not too many 
individual patches, so "two one-line fixes for the same problem" would
go into one common patch, not two.

thanks,

gert

-- 
"If was one thing all people took for granted, was conviction that if you 
 feed honest figures into a computer, honest figures come out. Never doubted 
 it myself till I met a computer with a sense of humor."
                             Robert A. Heinlein, The Moon is a Harsh Mistress

Gert Doering - Munich, Germany                             g...@greenie.muc.de

Attachment: signature.asc
Description: PGP signature

_______________________________________________
Openvpn-devel mailing list
Openvpn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/openvpn-devel

Reply via email to