RE: [PATCH ethtool] ethtool.c: fix memory leaks

2016-06-27 Thread David Laight
From: Ivan Vecera > Sent: 27 June 2016 10:41 > On 26.6.2016 10:59, Ben Hutchings wrote: > > On Fri, 2016-03-18 at 13:24 +0100, Ivan Vecera wrote: > >> Memory allocated at several places is not appropriately freed. > > > > Given that ethtool is not a library or a long-running application - why > > d

Re: [PATCH ethtool] ethtool.c: fix memory leaks

2016-06-27 Thread Ivan Vecera
On 26.6.2016 10:59, Ben Hutchings wrote: On Fri, 2016-03-18 at 13:24 +0100, Ivan Vecera wrote: Memory allocated at several places is not appropriately freed. Given that ethtool is not a library or a long-running application - why does that matter? Ben. Because each decently written program

Re: [PATCH ethtool] ethtool.c: fix memory leaks

2016-06-26 Thread Ben Hutchings
On Fri, 2016-03-18 at 13:24 +0100, Ivan Vecera wrote: > Memory allocated at several places is not appropriately freed. Given that ethtool is not a library or a long-running application - why does that matter? Ben. > Signed-off-by: Ivan Vecera > --- >  ethtool.c | 60

Re: [PATCH ethtool] ethtool.c: fix memory leaks

2016-04-08 Thread Ivan Vecera
On 18.3.2016 13:24, Ivan Vecera wrote: Memory allocated at several places is not appropriately freed. Signed-off-by: Ivan Vecera Ben, ping. I. --- ethtool.c | 60 +--- 1 file changed, 45 insertions(+), 15 deletions(-) diff --git a/e