Message ID | 152231458624.69730.1752893648612848392.stgit@bahia.lan |
---|---|
State | New |
Headers | show |
Series | fix memory leaks when using object_property_get_str() | expand |
On Thu, 29 Mar 2018 11:09:46 +0200 Greg Kurz <groug@kaod.org> wrote: > The string returned by object_property_get_str() is dynamically allocated. > > Signed-off-by: Greg Kurz <groug@kaod.org> > --- > exec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/exec.c b/exec.c > index c09bd93df31e..02b1efebb7c3 100644 > --- a/exec.c > +++ b/exec.c > @@ -1495,6 +1495,7 @@ static int find_max_supported_pagesize(Object *obj, void *opaque) > mem_path = object_property_get_str(obj, "mem-path", NULL); > if (mem_path) { > long hpsize = qemu_mempath_getpagesize(mem_path); > + g_free(mem_path); > if (hpsize < *hpsize_min) { > *hpsize_min = hpsize; > } > Personally, I'd probably do the g_free() at the end of the if (mem_path) branch, but this works as well. Reviewed-by: Cornelia Huck <cohuck@redhat.com>
On Thu, 29 Mar 2018 11:18:31 +0200 Cornelia Huck <cohuck@redhat.com> wrote: > On Thu, 29 Mar 2018 11:09:46 +0200 > Greg Kurz <groug@kaod.org> wrote: > > > The string returned by object_property_get_str() is dynamically allocated. > > > > Signed-off-by: Greg Kurz <groug@kaod.org> > > --- > > exec.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/exec.c b/exec.c > > index c09bd93df31e..02b1efebb7c3 100644 > > --- a/exec.c > > +++ b/exec.c > > @@ -1495,6 +1495,7 @@ static int find_max_supported_pagesize(Object *obj, void *opaque) > > mem_path = object_property_get_str(obj, "mem-path", NULL); > > if (mem_path) { > > long hpsize = qemu_mempath_getpagesize(mem_path); > > + g_free(mem_path); > > if (hpsize < *hpsize_min) { > > *hpsize_min = hpsize; > > } > > > > Personally, I'd probably do the g_free() at the end of the if > (mem_path) branch, but this works as well. > I usually prefer to free the string as soon as it isn't needed anymore... matter of taste :) > Reviewed-by: Cornelia Huck <cohuck@redhat.com>
On 29.03.2018 11:09, Greg Kurz wrote: > The string returned by object_property_get_str() is dynamically allocated. > > Signed-off-by: Greg Kurz <groug@kaod.org> > --- > exec.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/exec.c b/exec.c > index c09bd93df31e..02b1efebb7c3 100644 > --- a/exec.c > +++ b/exec.c > @@ -1495,6 +1495,7 @@ static int find_max_supported_pagesize(Object *obj, void *opaque) > mem_path = object_property_get_str(obj, "mem-path", NULL); > if (mem_path) { > long hpsize = qemu_mempath_getpagesize(mem_path); > + g_free(mem_path); > if (hpsize < *hpsize_min) { > *hpsize_min = hpsize; > } > > Reviewed-by: Thomas Huth <thuth@redhat.com>
On Thu, Mar 29, 2018 at 11:09:46AM +0200, Greg Kurz wrote: > The string returned by object_property_get_str() is dynamically allocated. > > Signed-off-by: Greg Kurz <groug@kaod.org> Queued, thanks.
diff --git a/exec.c b/exec.c index c09bd93df31e..02b1efebb7c3 100644 --- a/exec.c +++ b/exec.c @@ -1495,6 +1495,7 @@ static int find_max_supported_pagesize(Object *obj, void *opaque) mem_path = object_property_get_str(obj, "mem-path", NULL); if (mem_path) { long hpsize = qemu_mempath_getpagesize(mem_path); + g_free(mem_path); if (hpsize < *hpsize_min) { *hpsize_min = hpsize; }
The string returned by object_property_get_str() is dynamically allocated. Signed-off-by: Greg Kurz <groug@kaod.org> --- exec.c | 1 + 1 file changed, 1 insertion(+)