Re: [PATCH 1/2] Fix memory leak introduced in 5b96fecb97c752e08fdcebb331b983196f4b8935



On Wed, 2017-04-05 at 12:48 +0300, Timo Juhani Lindfors wrote:
Make sure that failure_reason is free'd by refactoring the code so
that the parent function handles allocation and parse_challenge only
parses the challenge, not the whole error message.
---
 src/nm-openvpn-service.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/src/nm-openvpn-service.c b/src/nm-openvpn-service.c
index 8451067..45e71c0 100644
--- a/src/nm-openvpn-service.c
+++ b/src/nm-openvpn-service.c
@@ -644,15 +644,14 @@ get_detail (const char *input, const char
*prefix)
 }

Hi Timo,


merged as 
https://git.gnome.org/browse/network-manager-openvpn/commit/?id=90db738220f69fcc01902759e97c9bb4ea8dd20a


Thanks!
Thomas

Attachment: signature.asc
Description: This is a digitally signed message part



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