On Tue, Apr 26, 2011 at 09:20:31AM +0800, Hu Tao wrote: > virsh schedinfo inactive-domain will trigger the problem. > --- > daemon/remote.c | 7 +++++-- > 1 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/daemon/remote.c b/daemon/remote.c > index 1c98bba..eedbc77 100644 > --- a/daemon/remote.c > +++ b/daemon/remote.c > @@ -945,8 +945,11 @@ remoteDispatchDomainGetSchedulerParameters(struct qemud_server *server ATTRIBUTE > cleanup: > if (rv < 0) { > remoteDispatchError(rerr); > - for (i = 0 ; i < nparams ; i++) > - VIR_FREE(ret->params.params_val[i].field); > + if (ret->params.params_val) { > + for (i = 0 ; i < nparams ; i++) > + VIR_FREE(ret->params.params_val[i].field); > + VIR_FREE(ret->params.params_val); > + } > } > if (dom) > virDomainFree(dom); ACK, a daemon leak, good fix ! Daniel -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@xxxxxxxxxxxx | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/ -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list