Re: [libvirt] [cim][PATCH 1/2] Fix memory leak in set_other_id_info

2018-01-09 Thread Adam Majer
On 12/18/2017 02:31 PM, John Ferlan wrote: > Looks OK; however, there hasn't been a non build related source code > change to libvirt-cim since June 2014. So while this could be pushed, > I'm curious to understand why the "sudden interest" in CIM? The last > release was July 2013! I'm not sure th

Re: [libvirt] [cim][PATCH 1/2] Fix memory leak in set_other_id_info

2017-12-18 Thread John Ferlan
On 12/15/2017 09:05 AM, Adam Majer wrote: > model was allocated by asprintf but never freed after usage > and assignement. assignment > > Signed-off-by: Adam Majer > --- > src/Virt_ComputerSystem.c | 2 ++ > 1 file changed, 2 insertions(+) > Looks OK; however, there hasn't been a non build

[libvirt] [cim][PATCH 1/2] Fix memory leak in set_other_id_info

2017-12-15 Thread Adam Majer
model was allocated by asprintf but never freed after usage and assignement. Signed-off-by: Adam Majer --- src/Virt_ComputerSystem.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/Virt_ComputerSystem.c b/src/Virt_ComputerSystem.c index da07f93..b4930ac 100644 --- a/src/Virt_ComputerSy