2010/12/7 Eric Blake <eblake@xxxxxxxxxx>: > popen must be matched with pclose (not fclose), or it will leak > resources. ÂFurthermore, it is a lousy interface when it comes to > signal handling. ÂWe're much better off using our decent command > wrapper. ÂNote that virCommand guarantees that VIR_FREE(outbuf) is > both required and safe to call, whether virCommandRun succeeded or > failed. > > * src/openvz/openvz_conf.c (openvzLoadDomains, openvzGetVEID): > Replace popen with virCommand usage. > --- > > v2: avoid memory leak on failure in openvzGetVEID > > This patch is awaiting review on these prerequisites: > https://www.redhat.com/archives/libvir-list/2010-December/msg00318.html > https://www.redhat.com/archives/libvir-list/2010-December/msg00321.html > > Âsrc/openvz/openvz_conf.c | Â 55 ++++++++++++++++++++++++--------------------- > Â1 files changed, 29 insertions(+), 26 deletions(-) > ACK. Matthias -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list