Skip to content

L
libvirt

openeuler / libvirt

通知 3
Star 0
Fork 0
L
libvirt

体验新版 GitCode，发现更多精彩内容 >>

提交 9bcd47cd 编写于 作者: D Daniel P. Berrangé
浏览文件
操作

lxc: replace VIR_FREE with g_autofree / g_free 

Reviewed-by: NLaine Stump <laine@redhat.com>
Signed-off-by: NDaniel P. Berrangé <berrange@redhat.com>
上级 a5a297f3
展开全部 隐藏空白更改
内联 并排
Showing with 269 addition and 437 deletion
src/lxc/lxc_cgroup.c
浏览文件 @ 9bcd47cd
...@@ -387,16 +387,16 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def, ...@@ -387,16 +387,16 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def,
int *nicindexes) int *nicindexes)
{ {
virCgroupPtr cgroup = NULL; virCgroupPtr cgroup = NULL;
char *machineName = virLXCDomainGetMachineName(def, 0); g_autofree char *machineName = virLXCDomainGetMachineName(def, 0);
if (!machineName) if (!machineName)
goto cleanup; return NULL;
if (def->resource->partition[0] != '/') { if (def->resource->partition[0] != '/') {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
_("Resource partition '%s' must start with '/'"), _("Resource partition '%s' must start with '/'"),
def->resource->partition); def->resource->partition);
goto cleanup; return NULL;
} }
if (virCgroupNewMachine(machineName, if (virCgroupNewMachine(machineName,
...@@ -410,7 +410,7 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def, ...@@ -410,7 +410,7 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def,
-1, -1,
0, 0,
&cgroup) < 0) &cgroup) < 0)
goto cleanup; return NULL;
/* setup control group permissions for user namespace */ /* setup control group permissions for user namespace */
if (def->idmap.uidmap) { if (def->idmap.uidmap) {
...@@ -419,14 +419,10 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def, ...@@ -419,14 +419,10 @@ virCgroupPtr virLXCCgroupCreate(virDomainDefPtr def,
def->idmap.gidmap[0].target, def->idmap.gidmap[0].target,
(1 << VIR_CGROUP_CONTROLLER_SYSTEMD)) < 0) { (1 << VIR_CGROUP_CONTROLLER_SYSTEMD)) < 0) {
virCgroupFree(&cgroup); virCgroupFree(&cgroup);
cgroup = NULL; return NULL;
goto cleanup;
} }
} }
cleanup:
VIR_FREE(machineName);
return cgroup; return cgroup;
} }
......
src/lxc/lxc_conf.c
浏览文件 @ 9bcd47cd
...@@ -62,7 +62,7 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver) ...@@ -62,7 +62,7 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver)
virCapsPtr caps; virCapsPtr caps;
virCapsGuestPtr guest; virCapsGuestPtr guest;
virArch altArch; virArch altArch;
char *lxc_path = NULL; g_autofree char *lxc_path = NULL;
if ((caps = virCapabilitiesNew(virArchFromHost(), if ((caps = virCapabilitiesNew(virArchFromHost(),
false, false)) == NULL) false, false)) == NULL)
...@@ -135,8 +135,6 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver) ...@@ -135,8 +135,6 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver)
goto error; goto error;
} }
VIR_FREE(lxc_path);
if (driver) { if (driver) {
/* Security driver data */ /* Security driver data */
const char *doi, *model, *label, *type; const char *doi, *model, *label, *type;
...@@ -167,7 +165,6 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver) ...@@ -167,7 +165,6 @@ virCapsPtr virLXCDriverCapsInit(virLXCDriverPtr driver)
return caps; return caps;
error: error:
VIR_FREE(lxc_path);
virObjectUnref(caps); virObjectUnref(caps);
return NULL; return NULL;
} }
...@@ -290,9 +287,9 @@ virLXCDriverConfigDispose(void *obj) ...@@ -290,9 +287,9 @@ virLXCDriverConfigDispose(void *obj)
{ {
virLXCDriverConfigPtr cfg = obj; virLXCDriverConfigPtr cfg = obj;
VIR_FREE(cfg->configDir); g_free(cfg->configDir);
VIR_FREE(cfg->autostartDir); g_free(cfg->autostartDir);
VIR_FREE(cfg->stateDir); g_free(cfg->stateDir);
VIR_FREE(cfg->logDir); g_free(cfg->logDir);
VIR_FREE(cfg->securityDriverName); g_free(cfg->securityDriverName);
} }
src/lxc/lxc_container.c
浏览文件 @ 9bcd47cd
此差异已折叠。
src/lxc/lxc_controller.c
浏览文件 @ 9bcd47cd
...@@ -191,7 +191,7 @@ static virLXCControllerPtr virLXCControllerNew(const char *name) ...@@ -191,7 +191,7 @@ static virLXCControllerPtr virLXCControllerNew(const char *name)
{ {
virLXCControllerPtr ctrl = NULL; virLXCControllerPtr ctrl = NULL;
virLXCDriverPtr driver = NULL; virLXCDriverPtr driver = NULL;
char *configFile = NULL; g_autofree char *configFile = NULL;
if (VIR_ALLOC(ctrl) < 0) if (VIR_ALLOC(ctrl) < 0)
goto error; goto error;
...@@ -220,7 +220,6 @@ static virLXCControllerPtr virLXCControllerNew(const char *name) ...@@ -220,7 +220,6 @@ static virLXCControllerPtr virLXCControllerNew(const char *name)
goto error; goto error;
cleanup: cleanup:
VIR_FREE(configFile);
virLXCControllerDriverFree(driver); virLXCControllerDriverFree(driver);
return ctrl; return ctrl;
...@@ -288,22 +287,22 @@ static void virLXCControllerFree(virLXCControllerPtr ctrl) ...@@ -288,22 +287,22 @@ static void virLXCControllerFree(virLXCControllerPtr ctrl)
virObjectUnref(ctrl->securityManager); virObjectUnref(ctrl->securityManager);
for (i = 0; i < ctrl->nveths; i++) for (i = 0; i < ctrl->nveths; i++)
VIR_FREE(ctrl->veths[i]); g_free(ctrl->veths[i]);
VIR_FREE(ctrl->veths); g_free(ctrl->veths);
VIR_FREE(ctrl->nicindexes); g_free(ctrl->nicindexes);
for (i = 0; i < ctrl->npassFDs; i++) for (i = 0; i < ctrl->npassFDs; i++)
VIR_FORCE_CLOSE(ctrl->passFDs[i]); VIR_FORCE_CLOSE(ctrl->passFDs[i]);
VIR_FREE(ctrl->passFDs); g_free(ctrl->passFDs);
for (i = 0; i < ctrl->nconsoles; i++) for (i = 0; i < ctrl->nconsoles; i++)
virLXCControllerConsoleClose(&(ctrl->consoles[i])); virLXCControllerConsoleClose(&(ctrl->consoles[i]));
VIR_FREE(ctrl->consoles); g_free(ctrl->consoles);
VIR_FREE(ctrl->devptmx); g_free(ctrl->devptmx);
virDomainObjEndAPI(&ctrl->vm); virDomainObjEndAPI(&ctrl->vm);
VIR_FREE(ctrl->name); g_free(ctrl->name);
if (ctrl->timerShutdown != -1) if (ctrl->timerShutdown != -1)
virEventRemoveTimeout(ctrl->timerShutdown); virEventRemoveTimeout(ctrl->timerShutdown);
...@@ -311,14 +310,14 @@ static void virLXCControllerFree(virLXCControllerPtr ctrl) ...@@ -311,14 +310,14 @@ static void virLXCControllerFree(virLXCControllerPtr ctrl)
virObjectUnref(ctrl->daemon); virObjectUnref(ctrl->daemon);
virLXCControllerFreeFuse(ctrl); virLXCControllerFreeFuse(ctrl);
VIR_FREE(ctrl->nbdpids); g_free(ctrl->nbdpids);
VIR_FREE(ctrl->nsFDs); g_free(ctrl->nsFDs);
virCgroupFree(&ctrl->cgroup); virCgroupFree(&ctrl->cgroup);
/* This must always be the last thing to be closed */ /* This must always be the last thing to be closed */
VIR_FORCE_CLOSE(ctrl->handshakeFd); VIR_FORCE_CLOSE(ctrl->handshakeFd);
VIR_FREE(ctrl); g_free(ctrl);
} }
...@@ -469,7 +468,7 @@ static int virLXCControllerSetupLoopDeviceFS(virDomainFSDefPtr fs) ...@@ -469,7 +468,7 @@ static int virLXCControllerSetupLoopDeviceFS(virDomainFSDefPtr fs)
* the rest of container setup 'just works' * the rest of container setup 'just works'
*/ */
fs->type = VIR_DOMAIN_FS_TYPE_BLOCK; fs->type = VIR_DOMAIN_FS_TYPE_BLOCK;
VIR_FREE(fs->src->path); g_free(fs->src->path);
fs->src->path = loname; fs->src->path = loname;
loname = NULL; loname = NULL;
...@@ -480,7 +479,7 @@ static int virLXCControllerSetupLoopDeviceFS(virDomainFSDefPtr fs) ...@@ -480,7 +479,7 @@ static int virLXCControllerSetupLoopDeviceFS(virDomainFSDefPtr fs)
static int virLXCControllerSetupLoopDeviceDisk(virDomainDiskDefPtr disk) static int virLXCControllerSetupLoopDeviceDisk(virDomainDiskDefPtr disk)
{ {
int lofd; int lofd;
char *loname = NULL; g_autofree char *loname = NULL;
const char *src = virDomainDiskGetSource(disk); const char *src = virDomainDiskGetSource(disk);
int ret = -1; int ret = -1;
...@@ -501,7 +500,6 @@ static int virLXCControllerSetupLoopDeviceDisk(virDomainDiskDefPtr disk) ...@@ -501,7 +500,6 @@ static int virLXCControllerSetupLoopDeviceDisk(virDomainDiskDefPtr disk)
ret = 0; ret = 0;
cleanup: cleanup:
VIR_FREE(loname);
if (ret < 0) if (ret < 0)
VIR_FORCE_CLOSE(lofd); VIR_FORCE_CLOSE(lofd);
...@@ -533,7 +531,7 @@ static int virLXCControllerSetupNBDDeviceFS(virDomainFSDefPtr fs) ...@@ -533,7 +531,7 @@ static int virLXCControllerSetupNBDDeviceFS(virDomainFSDefPtr fs)
* the rest of container setup 'just works' * the rest of container setup 'just works'
*/ */
fs->type = VIR_DOMAIN_FS_TYPE_BLOCK; fs->type = VIR_DOMAIN_FS_TYPE_BLOCK;
VIR_FREE(fs->src->path); g_free(fs->src->path);
fs->src->path = dev; fs->src->path = dev;
return 0; return 0;
...@@ -542,7 +540,7 @@ static int virLXCControllerSetupNBDDeviceFS(virDomainFSDefPtr fs) ...@@ -542,7 +540,7 @@ static int virLXCControllerSetupNBDDeviceFS(virDomainFSDefPtr fs)
static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk) static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk)
{ {
char *dev; g_autofree char *dev = NULL;
const char *src = virDomainDiskGetSource(disk); const char *src = virDomainDiskGetSource(disk);
int format = virDomainDiskGetFormat(disk); int format = virDomainDiskGetFormat(disk);
...@@ -565,11 +563,8 @@ static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk) ...@@ -565,11 +563,8 @@ static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk)
* the rest of container setup 'just works' * the rest of container setup 'just works'
*/ */
virDomainDiskSetType(disk, VIR_STORAGE_TYPE_BLOCK); virDomainDiskSetType(disk, VIR_STORAGE_TYPE_BLOCK);
if (virDomainDiskSetSource(disk, dev) < 0) { if (virDomainDiskSetSource(disk, dev) < 0)
VIR_FREE(dev);
return -1; return -1;
}
VIR_FREE(dev);
return 0; return 0;
} }
...@@ -577,16 +572,15 @@ static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk) ...@@ -577,16 +572,15 @@ static int virLXCControllerSetupNBDDeviceDisk(virDomainDiskDefPtr disk)
static int virLXCControllerAppendNBDPids(virLXCControllerPtr ctrl, static int virLXCControllerAppendNBDPids(virLXCControllerPtr ctrl,
const char *dev) const char *dev)
{ {
char *pidpath = NULL; g_autofree char *pidpath = NULL;
pid_t *pids = NULL; g_autofree pid_t *pids = NULL;
size_t npids = 0; size_t npids = 0;
size_t i; size_t i;
int ret = -1;
size_t loops = 0; size_t loops = 0;
pid_t pid; pid_t pid;
if (!STRPREFIX(dev, "/dev/")) if (!STRPREFIX(dev, "/dev/"))
goto cleanup; return -1;
pidpath = g_strdup_printf("/sys/devices/virtual/block/%s/pid", dev + 5); pidpath = g_strdup_printf("/sys/devices/virtual/block/%s/pid", dev + 5);
...@@ -600,27 +594,22 @@ static int virLXCControllerAppendNBDPids(virLXCControllerPtr ctrl, ...@@ -600,27 +594,22 @@ static int virLXCControllerAppendNBDPids(virLXCControllerPtr ctrl,
virReportSystemError(errno, virReportSystemError(errno,
_("Cannot check NBD device %s pid"), _("Cannot check NBD device %s pid"),
dev + 5); dev + 5);
goto cleanup; return -1;
} }
} }
if (virPidFileReadPath(pidpath, &pid) < 0) if (virPidFileReadPath(pidpath, &pid) < 0)
goto cleanup; return -1;
if (virProcessGetPids(pid, &npids, &pids) < 0) if (virProcessGetPids(pid, &npids, &pids) < 0)
goto cleanup; return -1;
for (i = 0; i < npids; i++) { for (i = 0; i < npids; i++) {
if (VIR_APPEND_ELEMENT(ctrl->nbdpids, ctrl->nnbdpids, pids[i]) < 0) if (VIR_APPEND_ELEMENT(ctrl->nbdpids, ctrl->nnbdpids, pids[i]) < 0)
goto cleanup; return -1;
} }
ret = 0; return 0;
cleanup:
VIR_FREE(pids);
VIR_FREE(pidpath);
return ret;
} }
static int virLXCControllerSetupLoopDevices(virLXCControllerPtr ctrl) static int virLXCControllerSetupLoopDevices(virLXCControllerPtr ctrl)
...@@ -957,7 +946,7 @@ static int virLXCControllerSetupServer(virLXCControllerPtr ctrl) ...@@ -957,7 +946,7 @@ static int virLXCControllerSetupServer(virLXCControllerPtr ctrl)
{ {
virNetServerPtr srv = NULL; virNetServerPtr srv = NULL;
virNetServerServicePtr svc = NULL; virNetServerServicePtr svc = NULL;
char *sockpath; g_autofree char *sockpath = NULL;
sockpath = g_strdup_printf("%s/%s.sock", LXC_STATE_DIR, ctrl->name); sockpath = g_strdup_printf("%s/%s.sock", LXC_STATE_DIR, ctrl->name);
...@@ -1002,11 +991,9 @@ static int virLXCControllerSetupServer(virLXCControllerPtr ctrl) ...@@ -1002,11 +991,9 @@ static int virLXCControllerSetupServer(virLXCControllerPtr ctrl)
goto error; goto error;
virNetDaemonUpdateServices(ctrl->daemon, true); virNetDaemonUpdateServices(ctrl->daemon, true);
VIR_FREE(sockpath);
return 0; return 0;
error: error:
VIR_FREE(sockpath);
virObjectUnref(srv); virObjectUnref(srv);
virObjectUnref(ctrl->daemon); virObjectUnref(ctrl->daemon);
ctrl->daemon = NULL; ctrl->daemon = NULL;
...@@ -1606,11 +1593,10 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef, ...@@ -1606,11 +1593,10 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef,
virDomainHostdevDefPtr def, virDomainHostdevDefPtr def,
virSecurityManagerPtr securityDriver) virSecurityManagerPtr securityDriver)
{ {
int ret = -1; g_autofree char *src = NULL;
char *src = NULL; g_autofree char *dstdir = NULL;
char *dstdir = NULL; g_autofree char *dstfile = NULL;
char *dstfile = NULL; g_autofree char *vroot = NULL;
char *vroot = NULL;
struct stat sb; struct stat sb;
mode_t mode; mode_t mode;
virDomainHostdevSubsysUSBPtr usbsrc = &def->source.subsys.u.usb; virDomainHostdevSubsysUSBPtr usbsrc = &def->source.subsys.u.usb;
...@@ -1626,14 +1612,14 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef, ...@@ -1626,14 +1612,14 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef,
if (stat(src, &sb) < 0) { if (stat(src, &sb) < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("Unable to access %s"), src); _("Unable to access %s"), src);
goto cleanup; return -1;
} }
if (!S_ISCHR(sb.st_mode)) { if (!S_ISCHR(sb.st_mode)) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
_("USB source %s was not a character device"), _("USB source %s was not a character device"),
src); src);
goto cleanup; return -1;
} }
mode = 0700 | S_IFCHR; mode = 0700 | S_IFCHR;
...@@ -1641,7 +1627,7 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef, ...@@ -1641,7 +1627,7 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef,
if (virFileMakePath(dstdir) < 0) { if (virFileMakePath(dstdir) < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("Unable to create %s"), dstdir); _("Unable to create %s"), dstdir);
goto cleanup; return -1;
} }
VIR_DEBUG("Creating dev %s (%d,%d)", VIR_DEBUG("Creating dev %s (%d,%d)",
...@@ -1650,24 +1636,17 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef, ...@@ -1650,24 +1636,17 @@ virLXCControllerSetupHostdevSubsysUSB(virDomainDefPtr vmDef,
virReportSystemError(errno, virReportSystemError(errno,
_("Unable to create device %s"), _("Unable to create device %s"),
dstfile); dstfile);
goto cleanup; return -1;
} }
if (lxcContainerChown(vmDef, dstfile) < 0) if (lxcContainerChown(vmDef, dstfile) < 0)
goto cleanup; return -1;
if (virSecurityManagerSetHostdevLabel(securityDriver, if (virSecurityManagerSetHostdevLabel(securityDriver,
vmDef, def, vroot) < 0) vmDef, def, vroot) < 0)
goto cleanup; return -1;
ret = 0;
cleanup: return 0;
VIR_FREE(src);
VIR_FREE(dstfile);
VIR_FREE(dstdir);
VIR_FREE(vroot);
return ret;
} }
...@@ -1676,8 +1655,8 @@ virLXCControllerSetupHostdevCapsStorage(virDomainDefPtr vmDef, ...@@ -1676,8 +1655,8 @@ virLXCControllerSetupHostdevCapsStorage(virDomainDefPtr vmDef,
virDomainHostdevDefPtr def, virDomainHostdevDefPtr def,
virSecurityManagerPtr securityDriver) virSecurityManagerPtr securityDriver)
{ {
char *dst = NULL; g_autofree char *dst = NULL;
char *path = NULL; g_autofree char *path = NULL;
int len = 0; int len = 0;
int ret = -1; int ret = -1;
struct stat sb; struct stat sb;
...@@ -1741,8 +1720,6 @@ virLXCControllerSetupHostdevCapsStorage(virDomainDefPtr vmDef, ...@@ -1741,8 +1720,6 @@ virLXCControllerSetupHostdevCapsStorage(virDomainDefPtr vmDef,
cleanup: cleanup:
def->source.caps.u.storage.block = dev; def->source.caps.u.storage.block = dev;
VIR_FREE(dst);
VIR_FREE(path);
return ret; return ret;
} }
...@@ -1752,8 +1729,8 @@ virLXCControllerSetupHostdevCapsMisc(virDomainDefPtr vmDef, ...@@ -1752,8 +1729,8 @@ virLXCControllerSetupHostdevCapsMisc(virDomainDefPtr vmDef,
virDomainHostdevDefPtr def, virDomainHostdevDefPtr def,
virSecurityManagerPtr securityDriver) virSecurityManagerPtr securityDriver)
{ {
char *dst = NULL; g_autofree char *dst = NULL;
char *path = NULL; g_autofree char *path = NULL;
int len = 0; int len = 0;
int ret = -1; int ret = -1;
struct stat sb; struct stat sb;
...@@ -1817,8 +1794,6 @@ virLXCControllerSetupHostdevCapsMisc(virDomainDefPtr vmDef, ...@@ -1817,8 +1794,6 @@ virLXCControllerSetupHostdevCapsMisc(virDomainDefPtr vmDef,
cleanup: cleanup:
def->source.caps.u.misc.chardev = dev; def->source.caps.u.misc.chardev = dev;
VIR_FREE(dst);
VIR_FREE(path);
return ret; return ret;
} }
...@@ -1910,7 +1885,7 @@ static int virLXCControllerSetupDisk(virLXCControllerPtr ctrl, ...@@ -1910,7 +1885,7 @@ static int virLXCControllerSetupDisk(virLXCControllerPtr ctrl,
virDomainDiskDefPtr def, virDomainDiskDefPtr def,
virSecurityManagerPtr securityDriver) virSecurityManagerPtr securityDriver)
{ {
char *dst = NULL; g_autofree char *dst = NULL;
int ret = -1; int ret = -1;
struct stat sb; struct stat sb;
mode_t mode; mode_t mode;
...@@ -1979,7 +1954,6 @@ static int virLXCControllerSetupDisk(virLXCControllerPtr ctrl, ...@@ -1979,7 +1954,6 @@ static int virLXCControllerSetupDisk(virLXCControllerPtr ctrl,
cleanup: cleanup:
def->src->path = tmpsrc; def->src->path = tmpsrc;
VIR_FREE(dst);
return ret; return ret;
} }
...@@ -2125,7 +2099,8 @@ lxcCreateTty(virLXCControllerPtr ctrl, int *ttymaster, ...@@ -2125,7 +2099,8 @@ lxcCreateTty(virLXCControllerPtr ctrl, int *ttymaster,
cleanup: cleanup:
if (ret != 0) { if (ret != 0) {
VIR_FORCE_CLOSE(*ttymaster); VIR_FORCE_CLOSE(*ttymaster);
VIR_FREE(*ttyName); g_free(*ttyName);
*ttyName = NULL;
} }
return ret; return ret;
...@@ -2233,9 +2208,10 @@ virLXCControllerSetupConsoles(virLXCControllerPtr ctrl, ...@@ -2233,9 +2208,10 @@ virLXCControllerSetupConsoles(virLXCControllerPtr ctrl,
char **containerTTYPaths) char **containerTTYPaths)
{ {
size_t i; size_t i;
g_autofree char *ttyHostPath = NULL;
for (i = 0; i < ctrl->nconsoles; i++) { for (i = 0; i < ctrl->nconsoles; i++) {
g_autofree char *ttyHostPath = NULL;
VIR_DEBUG("Opening tty on private %s", ctrl->devptmx); VIR_DEBUG("Opening tty on private %s", ctrl->devptmx);
if (lxcCreateTty(ctrl, if (lxcCreateTty(ctrl,
&ctrl->consoles[i].contFd, &ctrl->consoles[i].contFd,
...@@ -2248,8 +2224,6 @@ virLXCControllerSetupConsoles(virLXCControllerPtr ctrl, ...@@ -2248,8 +2224,6 @@ virLXCControllerSetupConsoles(virLXCControllerPtr ctrl,
/* Change the owner of tty device to the root user of container */ /* Change the owner of tty device to the root user of container */
if (lxcContainerChown(ctrl->def, ttyHostPath) < 0) if (lxcContainerChown(ctrl->def, ttyHostPath) < 0)
return -1; return -1;
VIR_FREE(ttyHostPath);
} }
return 0; return 0;
...@@ -2484,8 +2458,8 @@ virLXCControllerRun(virLXCControllerPtr ctrl) ...@@ -2484,8 +2458,8 @@ virLXCControllerRun(virLXCControllerPtr ctrl)
VIR_FORCE_CLOSE(containerhandshake[1]); VIR_FORCE_CLOSE(containerhandshake[1]);
for (i = 0; i < ctrl->nconsoles; i++) for (i = 0; i < ctrl->nconsoles; i++)
VIR_FREE(containerTTYPaths[i]); g_free(containerTTYPaths[i]);
VIR_FREE(containerTTYPaths); g_free(containerTTYPaths);
virLXCControllerStopInit(ctrl); virLXCControllerStopInit(ctrl);
...@@ -2517,9 +2491,9 @@ int main(int argc, char *argv[]) ...@@ -2517,9 +2491,9 @@ int main(int argc, char *argv[])
{ "help", 0, NULL, 'h' }, { "help", 0, NULL, 'h' },
{ 0, 0, 0, 0 }, { 0, 0, 0, 0 },
}; };
int *ttyFDs = NULL; g_autofree int *ttyFDs = NULL;
size_t nttyFDs = 0; size_t nttyFDs = 0;
int *passFDs = NULL; g_autofree int *passFDs = NULL;
size_t npassFDs = 0; size_t npassFDs = 0;
virLXCControllerPtr ctrl = NULL; virLXCControllerPtr ctrl = NULL;
size_t i; size_t i;
...@@ -2760,10 +2734,8 @@ int main(int argc, char *argv[]) ...@@ -2760,10 +2734,8 @@ int main(int argc, char *argv[])
virLXCControllerDeleteInterfaces(ctrl); virLXCControllerDeleteInterfaces(ctrl);
for (i = 0; i < nttyFDs; i++) for (i = 0; i < nttyFDs; i++)
VIR_FORCE_CLOSE(ttyFDs[i]); VIR_FORCE_CLOSE(ttyFDs[i]);
VIR_FREE(ttyFDs);
for (i = 0; i < npassFDs; i++) for (i = 0; i < npassFDs; i++)
VIR_FORCE_CLOSE(passFDs[i]); VIR_FORCE_CLOSE(passFDs[i]);
VIR_FREE(passFDs);
virLXCControllerFree(ctrl); virLXCControllerFree(ctrl);
......
src/lxc/lxc_domain.c
浏览文件 @ 9bcd47cd
...@@ -159,7 +159,7 @@ virLXCDomainObjPrivateAlloc(void *opaque G_GNUC_UNUSED) ...@@ -159,7 +159,7 @@ virLXCDomainObjPrivateAlloc(void *opaque G_GNUC_UNUSED)
return NULL; return NULL;
if (virLXCDomainObjInitJob(priv) < 0) { if (virLXCDomainObjInitJob(priv) < 0) {
VIR_FREE(priv); g_free(priv);
return NULL; return NULL;
} }
...@@ -174,7 +174,7 @@ virLXCDomainObjPrivateFree(void *data) ...@@ -174,7 +174,7 @@ virLXCDomainObjPrivateFree(void *data)
virCgroupFree(&priv->cgroup); virCgroupFree(&priv->cgroup);
virLXCDomainObjFreeJob(priv); virLXCDomainObjFreeJob(priv);
VIR_FREE(priv); g_free(priv);
} }
...@@ -200,8 +200,8 @@ lxcDomainDefNamespaceFree(void *nsdata) ...@@ -200,8 +200,8 @@ lxcDomainDefNamespaceFree(void *nsdata)
size_t i; size_t i;
lxcDomainDefPtr lxcDef = nsdata; lxcDomainDefPtr lxcDef = nsdata;
for (i = 0; i < VIR_LXC_DOMAIN_NAMESPACE_LAST; i++) for (i = 0; i < VIR_LXC_DOMAIN_NAMESPACE_LAST; i++)
VIR_FREE(lxcDef->ns_val[i]); g_free(lxcDef->ns_val[i]);
VIR_FREE(nsdata); g_free(nsdata);
} }
static int static int
...@@ -209,12 +209,11 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt, ...@@ -209,12 +209,11 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt,
void **data) void **data)
{ {
lxcDomainDefPtr lxcDef = NULL; lxcDomainDefPtr lxcDef = NULL;
xmlNodePtr *nodes = NULL; g_autofree xmlNodePtr *nodes = NULL;
bool uses_lxc_ns = false; bool uses_lxc_ns = false;
xmlNodePtr node; xmlNodePtr node;
int feature; int feature;
int n; int n;
char *tmp = NULL;
size_t i; size_t i;
if (VIR_ALLOC(lxcDef) < 0) if (VIR_ALLOC(lxcDef) < 0)
...@@ -226,6 +225,7 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt, ...@@ -226,6 +225,7 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt,
uses_lxc_ns |= n > 0; uses_lxc_ns |= n > 0;
for (i = 0; i < n; i++) { for (i = 0; i < n; i++) {
g_autofree char *tmp = NULL;
if ((feature = virLXCDomainNamespaceTypeFromString( if ((feature = virLXCDomainNamespaceTypeFromString(
(const char *)nodes[i]->name)) < 0) { (const char *)nodes[i]->name)) < 0) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
...@@ -246,10 +246,8 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt, ...@@ -246,10 +246,8 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt,
virReportError(VIR_ERR_INTERNAL_ERROR, virReportError(VIR_ERR_INTERNAL_ERROR,
_("Unknown LXC namespace source '%s'"), _("Unknown LXC namespace source '%s'"),
tmp); tmp);
VIR_FREE(tmp);
goto error; goto error;
} }
VIR_FREE(tmp);
if (!(lxcDef->ns_val[feature] = if (!(lxcDef->ns_val[feature] =
virXMLPropString(nodes[i], "value"))) { virXMLPropString(nodes[i], "value"))) {
...@@ -258,15 +256,13 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt, ...@@ -258,15 +256,13 @@ lxcDomainDefNamespaceParse(xmlXPathContextPtr ctxt,
goto error; goto error;
} }
} }
VIR_FREE(nodes);
ctxt->node = node; ctxt->node = node;
if (uses_lxc_ns) if (uses_lxc_ns)
*data = lxcDef; *data = lxcDef;
else else
VIR_FREE(lxcDef); g_free(lxcDef);
return 0; return 0;
error: error:
VIR_FREE(nodes);
lxcDomainDefNamespaceFree(lxcDef); lxcDomainDefNamespaceFree(lxcDef);
return -1; return -1;
} }
...@@ -496,7 +492,9 @@ virLXCDomainSetRunlevel(virDomainObjPtr vm, ...@@ -496,7 +492,9 @@ virLXCDomainSetRunlevel(virDomainObjPtr vm,
lxcDomainInitctlCallback, lxcDomainInitctlCallback,
&data); &data);
cleanup: cleanup:
VIR_FREE(data.st); g_free(data.st);
VIR_FREE(data.st_valid); data.st = NULL;
g_free(data.st_valid);
data.st_valid = NULL;
return ret; return ret;
} }
src/lxc/lxc_driver.c
浏览文件 @ 9bcd47cd
...@@ -1496,7 +1496,8 @@ static int lxcStateInitialize(bool privileged, ...@@ -1496,7 +1496,8 @@ static int lxcStateInitialize(bool privileged,
return VIR_DRV_STATE_INIT_ERROR; return VIR_DRV_STATE_INIT_ERROR;
lxc_driver->lockFD = -1; lxc_driver->lockFD = -1;
if (virMutexInit(&lxc_driver->lock) < 0) { if (virMutexInit(&lxc_driver->lock) < 0) {
VIR_FREE(lxc_driver); g_free(lxc_driver);
lxc_driver = NULL;
return VIR_DRV_STATE_INIT_ERROR; return VIR_DRV_STATE_INIT_ERROR;
} }
...@@ -1633,7 +1634,8 @@ static int lxcStateCleanup(void) ...@@ -1633,7 +1634,8 @@ static int lxcStateCleanup(void)
virObjectUnref(lxc_driver->config); virObjectUnref(lxc_driver->config);
virMutexDestroy(&lxc_driver->lock); virMutexDestroy(&lxc_driver->lock);
VIR_FREE(lxc_driver); g_free(lxc_driver);
lxc_driver = NULL;
return 0; return 0;
} }
...@@ -2473,7 +2475,8 @@ static int lxcDomainSetAutostart(virDomainPtr dom, ...@@ -2473,7 +2475,8 @@ static int lxcDomainSetAutostart(virDomainPtr dom,
{ {
virLXCDriverPtr driver = dom->conn->privateData; virLXCDriverPtr driver = dom->conn->privateData;
virDomainObjPtr vm; virDomainObjPtr vm;
char *configFile = NULL, *autostartLink = NULL; g_autofree char *configFile = NULL;
g_autofree char *autostartLink = NULL;
int ret = -1; int ret = -1;
virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver); virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver);
...@@ -2538,8 +2541,6 @@ static int lxcDomainSetAutostart(virDomainPtr dom, ...@@ -2538,8 +2541,6 @@ static int lxcDomainSetAutostart(virDomainPtr dom,
virLXCDomainObjEndJob(driver, vm); virLXCDomainObjEndJob(driver, vm);
cleanup: cleanup:
VIR_FREE(configFile);
VIR_FREE(autostartLink);
virDomainObjEndAPI(&vm); virDomainObjEndAPI(&vm);
virObjectUnref(cfg); virObjectUnref(cfg);
return ret; return ret;
...@@ -2551,8 +2552,7 @@ static int lxcFreezeContainer(virDomainObjPtr vm) ...@@ -2551,8 +2552,7 @@ static int lxcFreezeContainer(virDomainObjPtr vm)
int check_interval = 1; /* In milliseconds */ int check_interval = 1; /* In milliseconds */
int exp = 10; int exp = 10;
int waited_time = 0; int waited_time = 0;
int ret = -1; g_autofree char *state = NULL;
char *state = NULL;
virLXCDomainObjPrivatePtr priv = vm->privateData; virLXCDomainObjPrivatePtr priv = vm->privateData;
while (waited_time < timeout) { while (waited_time < timeout) {
...@@ -2599,10 +2599,8 @@ static int lxcFreezeContainer(virDomainObjPtr vm) ...@@ -2599,10 +2599,8 @@ static int lxcFreezeContainer(virDomainObjPtr vm)
} }
VIR_DEBUG("Read freezer.state: %s", state); VIR_DEBUG("Read freezer.state: %s", state);
if (STREQ(state, "FROZEN")) { if (STREQ(state, "FROZEN"))
ret = 0; return 0;
goto cleanup;
}
waited_time += check_interval; waited_time += check_interval;
/* /*
...@@ -2614,7 +2612,6 @@ static int lxcFreezeContainer(virDomainObjPtr vm) ...@@ -2614,7 +2612,6 @@ static int lxcFreezeContainer(virDomainObjPtr vm)
* In that case, eager polling will just waste CPU time. * In that case, eager polling will just waste CPU time.
*/ */
check_interval *= exp; check_interval *= exp;
VIR_FREE(state);
} }
VIR_DEBUG("lxcFreezeContainer timeout"); VIR_DEBUG("lxcFreezeContainer timeout");
error: error:
...@@ -2624,11 +2621,7 @@ static int lxcFreezeContainer(virDomainObjPtr vm) ...@@ -2624,11 +2621,7 @@ static int lxcFreezeContainer(virDomainObjPtr vm)
* This is likely to fall the group back again gracefully. * This is likely to fall the group back again gracefully.
*/ */
virCgroupSetFreezerState(priv->cgroup, "THAWED"); virCgroupSetFreezerState(priv->cgroup, "THAWED");
ret = -1; return -1;
cleanup:
VIR_FREE(state);
return ret;
} }
static int lxcDomainSuspend(virDomainPtr dom) static int lxcDomainSuspend(virDomainPtr dom)
...@@ -3345,7 +3338,7 @@ lxcDomainAttachDeviceDiskLive(virLXCDriverPtr driver, ...@@ -3345,7 +3338,7 @@ lxcDomainAttachDeviceDiskLive(virLXCDriverPtr driver,
virDomainDiskDefPtr def = dev->data.disk; virDomainDiskDefPtr def = dev->data.disk;
int ret = -1; int ret = -1;
struct stat sb; struct stat sb;
char *file = NULL; g_autofree char *file = NULL;
int perms; int perms;
const char *src = NULL; const char *src = NULL;
...@@ -3433,7 +3426,6 @@ lxcDomainAttachDeviceDiskLive(virLXCDriverPtr driver, ...@@ -3433,7 +3426,6 @@ lxcDomainAttachDeviceDiskLive(virLXCDriverPtr driver,
cleanup: cleanup:
if (src) if (src)
virDomainAuditDisk(vm, NULL, def->src, "attach", ret == 0); virDomainAuditDisk(vm, NULL, def->src, "attach", ret == 0);
VIR_FREE(file);
return ret; return ret;
} }
...@@ -3584,7 +3576,7 @@ lxcDomainAttachDeviceHostdevSubsysUSBLive(virLXCDriverPtr driver, ...@@ -3584,7 +3576,7 @@ lxcDomainAttachDeviceHostdevSubsysUSBLive(virLXCDriverPtr driver,
virLXCDomainObjPrivatePtr priv = vm->privateData; virLXCDomainObjPrivatePtr priv = vm->privateData;
virDomainHostdevDefPtr def = dev->data.hostdev; virDomainHostdevDefPtr def = dev->data.hostdev;
int ret = -1; int ret = -1;
char *src = NULL; g_autofree char *src = NULL;
struct stat sb; struct stat sb;
virUSBDevicePtr usb = NULL; virUSBDevicePtr usb = NULL;
virDomainHostdevSubsysUSBPtr usbsrc; virDomainHostdevSubsysUSBPtr usbsrc;
...@@ -3643,7 +3635,6 @@ lxcDomainAttachDeviceHostdevSubsysUSBLive(virLXCDriverPtr driver, ...@@ -3643,7 +3635,6 @@ lxcDomainAttachDeviceHostdevSubsysUSBLive(virLXCDriverPtr driver,
cleanup: cleanup:
virDomainAuditHostdev(vm, def, "attach", ret == 0); virDomainAuditHostdev(vm, def, "attach", ret == 0);
virUSBDeviceFree(usb); virUSBDeviceFree(usb);
VIR_FREE(src);
return ret; return ret;
} }
...@@ -3910,14 +3901,14 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm, ...@@ -3910,14 +3901,14 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm,
{ {
virLXCDomainObjPrivatePtr priv = vm->privateData; virLXCDomainObjPrivatePtr priv = vm->privateData;
virDomainDiskDefPtr def = NULL; virDomainDiskDefPtr def = NULL;
int idx, ret = -1; int idx;
char *dst = NULL; g_autofree char *dst = NULL;
const char *src; const char *src;
if (!priv->initpid) { if (!priv->initpid) {
virReportError(VIR_ERR_OPERATION_INVALID, "%s", virReportError(VIR_ERR_OPERATION_INVALID, "%s",
_("Cannot attach disk until init PID is known")); _("Cannot attach disk until init PID is known"));
goto cleanup; return -1;
} }
if ((idx = virDomainDiskIndexByName(vm->def, if ((idx = virDomainDiskIndexByName(vm->def,
...@@ -3925,7 +3916,7 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm, ...@@ -3925,7 +3916,7 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm,
false)) < 0) { false)) < 0) {
virReportError(VIR_ERR_OPERATION_FAILED, virReportError(VIR_ERR_OPERATION_FAILED,
_("disk %s not found"), dev->data.disk->dst); _("disk %s not found"), dev->data.disk->dst);
goto cleanup; return -1;
} }
def = vm->def->disks[idx]; def = vm->def->disks[idx];
...@@ -3936,12 +3927,12 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm, ...@@ -3936,12 +3927,12 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm,
if (!virCgroupHasController(priv->cgroup, VIR_CGROUP_CONTROLLER_DEVICES)) { if (!virCgroupHasController(priv->cgroup, VIR_CGROUP_CONTROLLER_DEVICES)) {
virReportError(VIR_ERR_OPERATION_INVALID, "%s", virReportError(VIR_ERR_OPERATION_INVALID, "%s",
_("devices cgroup isn't mounted")); _("devices cgroup isn't mounted"));
goto cleanup; return -1;
} }
if (lxcDomainAttachDeviceUnlink(vm, dst) < 0) { if (lxcDomainAttachDeviceUnlink(vm, dst) < 0) {
virDomainAuditDisk(vm, def->src, NULL, "detach", false); virDomainAuditDisk(vm, def->src, NULL, "detach", false);
goto cleanup; return -1;
} }
virDomainAuditDisk(vm, def->src, NULL, "detach", true); virDomainAuditDisk(vm, def->src, NULL, "detach", true);
...@@ -3953,11 +3944,7 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm, ...@@ -3953,11 +3944,7 @@ lxcDomainDetachDeviceDiskLive(virDomainObjPtr vm,
virDomainDiskRemove(vm->def, idx); virDomainDiskRemove(vm->def, idx);
virDomainDiskDefFree(def); virDomainDiskDefFree(def);
ret = 0; return 0;
cleanup:
VIR_FREE(dst);
return ret;
} }
...@@ -4055,7 +4042,7 @@ lxcDomainDetachDeviceHostdevUSBLive(virLXCDriverPtr driver, ...@@ -4055,7 +4042,7 @@ lxcDomainDetachDeviceHostdevUSBLive(virLXCDriverPtr driver,
virLXCDomainObjPrivatePtr priv = vm->privateData; virLXCDomainObjPrivatePtr priv = vm->privateData;
virDomainHostdevDefPtr def = NULL; virDomainHostdevDefPtr def = NULL;
int idx, ret = -1; int idx, ret = -1;
char *dst = NULL; g_autofree char *dst = NULL;
virUSBDevicePtr usb = NULL; virUSBDevicePtr usb = NULL;
virHostdevManagerPtr hostdev_mgr = driver->hostdevMgr; virHostdevManagerPtr hostdev_mgr = driver->hostdevMgr;
virDomainHostdevSubsysUSBPtr usbsrc; virDomainHostdevSubsysUSBPtr usbsrc;
...@@ -4103,7 +4090,6 @@ lxcDomainDetachDeviceHostdevUSBLive(virLXCDriverPtr driver, ...@@ -4103,7 +4090,6 @@ lxcDomainDetachDeviceHostdevUSBLive(virLXCDriverPtr driver,
cleanup: cleanup:
virUSBDeviceFree(usb); virUSBDeviceFree(usb);
VIR_FREE(dst);
return ret; return ret;
} }
...@@ -4948,8 +4934,6 @@ lxcDomainGetHostname(virDomainPtr dom, ...@@ -4948,8 +4934,6 @@ lxcDomainGetHostname(virDomainPtr dom,
virDomainObjPtr vm = NULL; virDomainObjPtr vm = NULL;
char macaddr[VIR_MAC_STRING_BUFLEN]; char macaddr[VIR_MAC_STRING_BUFLEN];
g_autoptr(virConnect) conn = NULL; g_autoptr(virConnect) conn = NULL;
virNetworkDHCPLeasePtr *leases = NULL;
int n_leases;
size_t i, j; size_t i, j;
char *hostname = NULL; char *hostname = NULL;
...@@ -4973,6 +4957,8 @@ lxcDomainGetHostname(virDomainPtr dom, ...@@ -4973,6 +4957,8 @@ lxcDomainGetHostname(virDomainPtr dom,
for (i = 0; i < vm->def->nnets; i++) { for (i = 0; i < vm->def->nnets; i++) {
g_autoptr(virNetwork) network = NULL; g_autoptr(virNetwork) network = NULL;
virDomainNetDefPtr net = vm->def->nets[i]; virDomainNetDefPtr net = vm->def->nets[i];
g_autofree virNetworkDHCPLeasePtr *leases = NULL;
int n_leases;
if (net->type != VIR_DOMAIN_NET_TYPE_NETWORK) if (net->type != VIR_DOMAIN_NET_TYPE_NETWORK)
continue; continue;
...@@ -4996,8 +4982,6 @@ lxcDomainGetHostname(virDomainPtr dom, ...@@ -4996,8 +4982,6 @@ lxcDomainGetHostname(virDomainPtr dom,
virNetworkDHCPLeaseFree(lease); virNetworkDHCPLeaseFree(lease);
} }
VIR_FREE(leases);
if (hostname) if (hostname)
goto endjob; goto endjob;
} }
......
src/lxc/lxc_fuse.c
浏览文件 @ 9bcd47cd
...@@ -326,10 +326,10 @@ int lxcSetupFuse(virLXCFusePtr *f, virDomainDefPtr def) ...@@ -326,10 +326,10 @@ int lxcSetupFuse(virLXCFusePtr *f, virDomainDefPtr def)
*f = fuse; *f = fuse;
return ret; return ret;
cleanup1: cleanup1:
VIR_FREE(fuse->mountpoint); g_free(fuse->mountpoint);
virMutexDestroy(&fuse->lock); virMutexDestroy(&fuse->lock);
cleanup2: cleanup2:
VIR_FREE(fuse); g_free(fuse);
goto cleanup; goto cleanup;
} }
...@@ -356,8 +356,8 @@ void lxcFreeFuse(virLXCFusePtr *f) ...@@ -356,8 +356,8 @@ void lxcFreeFuse(virLXCFusePtr *f)
fuse_exit(fuse->fuse); fuse_exit(fuse->fuse);
virMutexUnlock(&fuse->lock); virMutexUnlock(&fuse->lock);
VIR_FREE(fuse->mountpoint); g_free(fuse->mountpoint);
VIR_FREE(*f); g_free(*f);
} }
} }
#else #else
......
src/lxc/lxc_monitor.c
浏览文件 @ 9bcd47cd
...@@ -143,7 +143,7 @@ virLXCMonitorPtr virLXCMonitorNew(virDomainObjPtr vm, ...@@ -143,7 +143,7 @@ virLXCMonitorPtr virLXCMonitorNew(virDomainObjPtr vm,
virLXCMonitorCallbacksPtr cb) virLXCMonitorCallbacksPtr cb)
{ {
virLXCMonitorPtr mon; virLXCMonitorPtr mon;
char *sockpath = NULL; g_autofree char *sockpath = NULL;
if (virLXCMonitorInitialize() < 0) if (virLXCMonitorInitialize() < 0)
return NULL; return NULL;
...@@ -180,14 +180,11 @@ virLXCMonitorPtr virLXCMonitorNew(virDomainObjPtr vm, ...@@ -180,14 +180,11 @@ virLXCMonitorPtr virLXCMonitorNew(virDomainObjPtr vm,
mon->vm = virObjectRef(vm); mon->vm = virObjectRef(vm);
memcpy(&mon->cb, cb, sizeof(mon->cb)); memcpy(&mon->cb, cb, sizeof(mon->cb));
cleanup:
VIR_FREE(sockpath);
return mon; return mon;
error: error:
virObjectUnref(mon); virObjectUnref(mon);
mon = NULL; return NULL;
goto cleanup;
} }
......
src/lxc/lxc_native.c
浏览文件 @ 9bcd47cd
...@@ -93,11 +93,11 @@ lxcFstabFree(lxcFstabPtr fstab) ...@@ -93,11 +93,11 @@ lxcFstabFree(lxcFstabPtr fstab)
lxcFstabPtr next = NULL; lxcFstabPtr next = NULL;
next = fstab->next; next = fstab->next;
VIR_FREE(fstab->src); g_free(fstab->src);
VIR_FREE(fstab->dst); g_free(fstab->dst);
VIR_FREE(fstab->type); g_free(fstab->type);
VIR_FREE(fstab->options); g_free(fstab->options);
VIR_FREE(fstab); g_free(fstab);
fstab = next; fstab = next;
} }
...@@ -105,7 +105,7 @@ lxcFstabFree(lxcFstabPtr fstab) ...@@ -105,7 +105,7 @@ lxcFstabFree(lxcFstabPtr fstab)
static char ** lxcStringSplit(const char *string) static char ** lxcStringSplit(const char *string)
{ {
char *tmp; g_autofree char *tmp = NULL;
size_t i; size_t i;
size_t ntokens = 0; size_t ntokens = 0;
char **parts; char **parts;
...@@ -136,12 +136,10 @@ static char ** lxcStringSplit(const char *string) ...@@ -136,12 +136,10 @@ static char ** lxcStringSplit(const char *string)
result[ntokens - 2] = g_strdup(parts[i]); result[ntokens - 2] = g_strdup(parts[i]);
} }
VIR_FREE(tmp);
virStringListFree(parts); virStringListFree(parts);
return result; return result;
error: error:
VIR_FREE(tmp);
virStringListFree(parts); virStringListFree(parts);
virStringListFree(result); virStringListFree(result);
return NULL; return NULL;
...@@ -257,7 +255,7 @@ static int ...@@ -257,7 +255,7 @@ static int
lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab) lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab)
{ {
const char *src = NULL; const char *src = NULL;
char *dst = NULL; g_autofree char *dst = NULL;
char **options = virStringSplit(fstab->options, ",", 0); char **options = virStringSplit(fstab->options, ",", 0);
bool readonly; bool readonly;
int type = VIR_DOMAIN_FS_TYPE_MOUNT; int type = VIR_DOMAIN_FS_TYPE_MOUNT;
...@@ -313,7 +311,6 @@ lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab) ...@@ -313,7 +311,6 @@ lxcAddFstabLine(virDomainDefPtr def, lxcFstabPtr fstab)
ret = 1; ret = 1;
cleanup: cleanup:
VIR_FREE(dst);
virStringListFree(options); virStringListFree(options);
return ret; return ret;
} }
...@@ -444,8 +441,8 @@ lxcAddNetworkRouteDefinition(const char *address, ...@@ -444,8 +441,8 @@ lxcAddNetworkRouteDefinition(const char *address,
size_t *nroutes) size_t *nroutes)
{ {
virNetDevIPRoutePtr route = NULL; virNetDevIPRoutePtr route = NULL;
char *familyStr = NULL; g_autofree char *familyStr = NULL;
char *zero = NULL; g_autofree char *zero = NULL;
zero = g_strdup(family == AF_INET ? VIR_SOCKET_ADDR_IPV4_ALL : VIR_SOCKET_ADDR_IPV6_ALL); zero = g_strdup(family == AF_INET ? VIR_SOCKET_ADDR_IPV4_ALL : VIR_SOCKET_ADDR_IPV6_ALL);
...@@ -459,14 +456,9 @@ lxcAddNetworkRouteDefinition(const char *address, ...@@ -459,14 +456,9 @@ lxcAddNetworkRouteDefinition(const char *address,
if (VIR_APPEND_ELEMENT(*routes, *nroutes, route) < 0) if (VIR_APPEND_ELEMENT(*routes, *nroutes, route) < 0)
goto error; goto error;
VIR_FREE(familyStr);
VIR_FREE(zero);
return 0; return 0;
error: error:
VIR_FREE(familyStr);
VIR_FREE(zero);
virNetDevIPRouteFree(route); virNetDevIPRouteFree(route);
return -1; return -1;
} }
...@@ -499,7 +491,7 @@ lxcAddNetworkDefinition(virDomainDefPtr def, lxcNetworkParseData *data) ...@@ -499,7 +491,7 @@ lxcAddNetworkDefinition(virDomainDefPtr def, lxcNetworkParseData *data)
/* This still requires the user to manually setup the vlan interface /* This still requires the user to manually setup the vlan interface
* on the host */ * on the host */
if (isVlan && data->vlanid) { if (isVlan && data->vlanid) {
VIR_FREE(hostdev->source.caps.u.net.ifname); g_free(hostdev->source.caps.u.net.ifname);
hostdev->source.caps.u.net.ifname = g_strdup_printf("%s.%s", hostdev->source.caps.u.net.ifname = g_strdup_printf("%s.%s",
data->link, data->link,
data->vlanid); data->vlanid);
...@@ -553,8 +545,9 @@ lxcAddNetworkDefinition(virDomainDefPtr def, lxcNetworkParseData *data) ...@@ -553,8 +545,9 @@ lxcAddNetworkDefinition(virDomainDefPtr def, lxcNetworkParseData *data)
error: error:
for (i = 0; i < data->nips; i++) for (i = 0; i < data->nips; i++)
VIR_FREE(data->ips[i]); g_free(data->ips[i]);
VIR_FREE(data->ips); g_free(data->ips);
data->ips = NULL;
virDomainNetDefFree(net); virDomainNetDefFree(net);
virDomainHostdevDefFree(hostdev); virDomainHostdevDefFree(hostdev);
return -1; return -1;
...@@ -568,7 +561,7 @@ lxcNetworkParseDataIPs(const char *name, ...@@ -568,7 +561,7 @@ lxcNetworkParseDataIPs(const char *name,
{ {
int family = AF_INET; int family = AF_INET;
char **ipparts = NULL; char **ipparts = NULL;
virNetDevIPAddrPtr ip = NULL; g_autofree virNetDevIPAddrPtr ip = NULL;
if (VIR_ALLOC(ip) < 0) if (VIR_ALLOC(ip) < 0)
return -1; return -1;
...@@ -585,16 +578,13 @@ lxcNetworkParseDataIPs(const char *name, ...@@ -585,16 +578,13 @@ lxcNetworkParseDataIPs(const char *name,
_("Invalid CIDR address: '%s'"), value->str); _("Invalid CIDR address: '%s'"), value->str);
virStringListFree(ipparts); virStringListFree(ipparts);
VIR_FREE(ip);
return -1; return -1;
} }
virStringListFree(ipparts); virStringListFree(ipparts);
if (VIR_APPEND_ELEMENT(parseData->ips, parseData->nips, ip) < 0) { if (VIR_APPEND_ELEMENT(parseData->ips, parseData->nips, ip) < 0)
VIR_FREE(ip);
return -1; return -1;
}
return 0; return 0;
} }
...@@ -792,16 +782,18 @@ lxcConvertNetworkSettings(virDomainDefPtr def, virConfPtr properties) ...@@ -792,16 +782,18 @@ lxcConvertNetworkSettings(virDomainDefPtr def, virConfPtr properties)
cleanup: cleanup:
for (i = 0; i < networks.ndata; i++) for (i = 0; i < networks.ndata; i++)
VIR_FREE(networks.parseData[i]); g_free(networks.parseData[i]);
VIR_FREE(networks.parseData); g_free(networks.parseData);
networks.parseData = NULL;
return ret; return ret;
error: error:
for (i = 0; i < networks.ndata; i++) { for (i = 0; i < networks.ndata; i++) {
lxcNetworkParseDataPtr data = networks.parseData[i]; lxcNetworkParseDataPtr data = networks.parseData[i];
for (j = 0; j < data->nips; j++) for (j = 0; j < data->nips; j++)
VIR_FREE(data->ips[j]); g_free(data->ips[j]);
VIR_FREE(data->ips); g_free(data->ips);
data->ips = NULL;
} }
goto cleanup; goto cleanup;
} }
...@@ -905,7 +897,8 @@ lxcSetMemTune(virDomainDefPtr def, virConfPtr properties) ...@@ -905,7 +897,8 @@ lxcSetMemTune(virDomainDefPtr def, virConfPtr properties)
size = size / 1024; size = size / 1024;
virDomainDefSetMemoryTotal(def, size); virDomainDefSetMemoryTotal(def, size);
def->mem.hard_limit = virMemoryLimitTruncate(size); def->mem.hard_limit = virMemoryLimitTruncate(size);
VIR_FREE(value); g_free(value);
value = NULL;
} }
if (virConfGetValueString(properties, if (virConfGetValueString(properties,
...@@ -914,7 +907,8 @@ lxcSetMemTune(virDomainDefPtr def, virConfPtr properties) ...@@ -914,7 +907,8 @@ lxcSetMemTune(virDomainDefPtr def, virConfPtr properties)
if (lxcConvertSize(value, &size) < 0) if (lxcConvertSize(value, &size) < 0)
return -1; return -1;
def->mem.soft_limit = virMemoryLimitTruncate(size / 1024); def->mem.soft_limit = virMemoryLimitTruncate(size / 1024);
VIR_FREE(value); g_free(value);
value = NULL;
} }
if (virConfGetValueString(properties, if (virConfGetValueString(properties,
...@@ -937,14 +931,16 @@ lxcSetCpuTune(virDomainDefPtr def, virConfPtr properties) ...@@ -937,14 +931,16 @@ lxcSetCpuTune(virDomainDefPtr def, virConfPtr properties)
if (virStrToLong_ull(value, NULL, 10, &def->cputune.shares) < 0) if (virStrToLong_ull(value, NULL, 10, &def->cputune.shares) < 0)
goto error; goto error;
def->cputune.sharesSpecified = true; def->cputune.sharesSpecified = true;
VIR_FREE(value); g_free(value);
value = NULL;
} }
if (virConfGetValueString(properties, "lxc.cgroup.cpu.cfs_quota_us", if (virConfGetValueString(properties, "lxc.cgroup.cpu.cfs_quota_us",
&value) > 0) { &value) > 0) {
if (virStrToLong_ll(value, NULL, 10, &def->cputune.quota) < 0) if (virStrToLong_ll(value, NULL, 10, &def->cputune.quota) < 0)
goto error; goto error;
VIR_FREE(value); g_free(value);
value = NULL;
} }
if (virConfGetValueString(properties, "lxc.cgroup.cpu.cfs_period_us", if (virConfGetValueString(properties, "lxc.cgroup.cpu.cfs_period_us",
...@@ -972,7 +968,8 @@ lxcSetCpusetTune(virDomainDefPtr def, virConfPtr properties) ...@@ -972,7 +968,8 @@ lxcSetCpusetTune(virDomainDefPtr def, virConfPtr properties)
if (virBitmapParse(value, &def->cpumask, VIR_DOMAIN_CPUMASK_LEN) < 0) if (virBitmapParse(value, &def->cpumask, VIR_DOMAIN_CPUMASK_LEN) < 0)
return -1; return -1;
def->placement_mode = VIR_DOMAIN_CPU_PLACEMENT_MODE_STATIC; def->placement_mode = VIR_DOMAIN_CPU_PLACEMENT_MODE_STATIC;
VIR_FREE(value); g_free(value);
value = NULL;
} }
if (virConfGetValueString(properties, "lxc.cgroup.cpuset.mems", if (virConfGetValueString(properties, "lxc.cgroup.cpuset.mems",
...@@ -1001,7 +998,7 @@ lxcBlkioDeviceWalkCallback(const char *name, virConfValuePtr value, void *data) ...@@ -1001,7 +998,7 @@ lxcBlkioDeviceWalkCallback(const char *name, virConfValuePtr value, void *data)
virBlkioDevicePtr device = NULL; virBlkioDevicePtr device = NULL;
virDomainDefPtr def = data; virDomainDefPtr def = data;
size_t i = 0; size_t i = 0;
char *path = NULL; g_autofree char *path = NULL;
int ret = -1; int ret = -1;
if (!STRPREFIX(name, "lxc.cgroup.blkio.") || if (!STRPREFIX(name, "lxc.cgroup.blkio.") ||
...@@ -1077,8 +1074,6 @@ lxcBlkioDeviceWalkCallback(const char *name, virConfValuePtr value, void *data) ...@@ -1077,8 +1074,6 @@ lxcBlkioDeviceWalkCallback(const char *name, virConfValuePtr value, void *data)
cleanup: cleanup:
virStringListFree(parts); virStringListFree(parts);
VIR_FREE(path);
return ret; return ret;
} }
...@@ -1172,7 +1167,8 @@ lxcParseConfigString(const char *config, ...@@ -1172,7 +1167,8 @@ lxcParseConfigString(const char *config,
else if (arch == VIR_ARCH_NONE && STREQ(value, "amd64")) else if (arch == VIR_ARCH_NONE && STREQ(value, "amd64"))
arch = VIR_ARCH_X86_64; arch = VIR_ARCH_X86_64;
vmdef->os.arch = arch; vmdef->os.arch = arch;
VIR_FREE(value); g_free(value);
value = NULL;
} }
vmdef->os.init = g_strdup("/sbin/init"); vmdef->os.init = g_strdup("/sbin/init");
......
src/lxc/lxc_process.c
浏览文件 @ 9bcd47cd
...@@ -135,14 +135,13 @@ static void ...@@ -135,14 +135,13 @@ static void
lxcProcessRemoveDomainStatus(virLXCDriverConfigPtr cfg, lxcProcessRemoveDomainStatus(virLXCDriverConfigPtr cfg,
virDomainObjPtr vm) virDomainObjPtr vm)
{ {
char *file = NULL; g_autofree char *file = g_strdup_printf("%s/%s.xml",
cfg->stateDir,
file = g_strdup_printf("%s/%s.xml", cfg->stateDir, vm->def->name); vm->def->name);
if (unlink(file) < 0 && errno != ENOENT && errno != ENOTDIR) if (unlink(file) < 0 && errno != ENOENT && errno != ENOTDIR)
VIR_WARN("Failed to remove domain XML for %s: %s", VIR_WARN("Failed to remove domain XML for %s: %s",
vm->def->name, g_strerror(errno)); vm->def->name, g_strerror(errno));
VIR_FREE(file);
} }
...@@ -170,13 +169,12 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver, ...@@ -170,13 +169,12 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver,
/* now that we know it's stopped call the hook if present */ /* now that we know it's stopped call the hook if present */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
/* we can't stop the operation even if the script raised an error */ /* we can't stop the operation even if the script raised an error */
virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name, virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_STOPPED, VIR_HOOK_SUBOP_END, VIR_HOOK_LXC_OP_STOPPED, VIR_HOOK_SUBOP_END,
NULL, xml, NULL); NULL, xml, NULL);
VIR_FREE(xml);
} }
virSecurityManagerRestoreAllLabel(driver->securityManager, virSecurityManagerRestoreAllLabel(driver->securityManager,
...@@ -185,9 +183,12 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver, ...@@ -185,9 +183,12 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver,
/* Clear out dynamically assigned labels */ /* Clear out dynamically assigned labels */
if (vm->def->nseclabels && if (vm->def->nseclabels &&
vm->def->seclabels[0]->type == VIR_DOMAIN_SECLABEL_DYNAMIC) { vm->def->seclabels[0]->type == VIR_DOMAIN_SECLABEL_DYNAMIC) {
VIR_FREE(vm->def->seclabels[0]->model); g_free(vm->def->seclabels[0]->model);
VIR_FREE(vm->def->seclabels[0]->label); g_free(vm->def->seclabels[0]->label);
VIR_FREE(vm->def->seclabels[0]->imagelabel); g_free(vm->def->seclabels[0]->imagelabel);
vm->def->seclabels[0]->model = NULL;
vm->def->seclabels[0]->label = NULL;
vm->def->seclabels[0]->imagelabel = NULL;
} }
/* Stop autodestroy in case guest is restarted */ /* Stop autodestroy in case guest is restarted */
...@@ -243,17 +244,17 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver, ...@@ -243,17 +244,17 @@ static void virLXCProcessCleanup(virLXCDriverPtr driver,
* the bug we are working around here. * the bug we are working around here.
*/ */
virCgroupTerminateMachine(priv->machineName); virCgroupTerminateMachine(priv->machineName);
VIR_FREE(priv->machineName); g_free(priv->machineName);
priv->machineName = NULL;
/* The "release" hook cleans up additional resources */ /* The "release" hook cleans up additional resources */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
/* we can't stop the operation even if the script raised an error */ /* we can't stop the operation even if the script raised an error */
virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name, virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_RELEASE, VIR_HOOK_SUBOP_END, VIR_HOOK_LXC_OP_RELEASE, VIR_HOOK_SUBOP_END,
NULL, xml, NULL); NULL, xml, NULL);
VIR_FREE(xml);
} }
virDomainObjRemoveTransientDef(vm); virDomainObjRemoveTransientDef(vm);
...@@ -442,12 +443,9 @@ static int virLXCProcessSetupNamespaceName(virLXCDriverPtr driver, ...@@ -442,12 +443,9 @@ static int virLXCProcessSetupNamespaceName(virLXCDriverPtr driver,
static int virLXCProcessSetupNamespacePID(int ns_type, const char *name) static int virLXCProcessSetupNamespacePID(int ns_type, const char *name)
{ {
int fd; g_autofree char *path = g_strdup_printf("/proc/%s/ns/%s",
char *path; name, nsInfoLocal[ns_type]);
int fd = open(path, O_RDONLY);
path = g_strdup_printf("/proc/%s/ns/%s", name, nsInfoLocal[ns_type]);
fd = open(path, O_RDONLY);
VIR_FREE(path);
if (fd < 0) { if (fd < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("failed to open ns %s"), _("failed to open ns %s"),
...@@ -460,7 +458,7 @@ static int virLXCProcessSetupNamespacePID(int ns_type, const char *name) ...@@ -460,7 +458,7 @@ static int virLXCProcessSetupNamespacePID(int ns_type, const char *name)
static int virLXCProcessSetupNamespaceNet(int ns_type, const char *name) static int virLXCProcessSetupNamespaceNet(int ns_type, const char *name)
{ {
char *path; g_autofree char *path = NULL;
int fd; int fd;
if (ns_type != VIR_LXC_DOMAIN_NAMESPACE_SHARENET) { if (ns_type != VIR_LXC_DOMAIN_NAMESPACE_SHARENET) {
virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s", virReportError(VIR_ERR_CONFIG_UNSUPPORTED, "%s",
...@@ -471,7 +469,6 @@ static int virLXCProcessSetupNamespaceNet(int ns_type, const char *name) ...@@ -471,7 +469,6 @@ static int virLXCProcessSetupNamespaceNet(int ns_type, const char *name)
path = g_strdup_printf("%s/netns/%s", RUNSTATEDIR, name); path = g_strdup_printf("%s/netns/%s", RUNSTATEDIR, name);
fd = open(path, O_RDONLY); fd = open(path, O_RDONLY);
VIR_FREE(path);
if (fd < 0) { if (fd < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("failed to open netns %s"), name); _("failed to open netns %s"), name);
...@@ -669,7 +666,8 @@ virLXCProcessCleanInterfaces(virDomainDefPtr def) ...@@ -669,7 +666,8 @@ virLXCProcessCleanInterfaces(virDomainDefPtr def)
size_t i; size_t i;
for (i = 0; i < def->nnets; i++) { for (i = 0; i < def->nnets; i++) {
VIR_FREE(def->nets[i]->ifname_guest_actual); g_free(def->nets[i]->ifname_guest_actual);
def->nets[i]->ifname_guest_actual = NULL;
VIR_DEBUG("Cleared net names: %s", def->nets[i]->ifname_guest); VIR_DEBUG("Cleared net names: %s", def->nets[i]->ifname_guest);
} }
} }
...@@ -757,24 +755,20 @@ virLXCProcessGetNsInode(pid_t pid, ...@@ -757,24 +755,20 @@ virLXCProcessGetNsInode(pid_t pid,
const char *nsname, const char *nsname,
ino_t *inode) ino_t *inode)
{ {
char *path = NULL; g_autofree char *path = NULL;
struct stat sb; struct stat sb;
int ret = -1;
path = g_strdup_printf("/proc/%lld/ns/%s", (long long)pid, nsname); path = g_strdup_printf("/proc/%lld/ns/%s", (long long)pid, nsname);
if (stat(path, &sb) < 0) { if (stat(path, &sb) < 0) {
virReportSystemError(errno, virReportSystemError(errno,
_("Unable to stat %s"), path); _("Unable to stat %s"), path);
goto cleanup; return -1;
} }
*inode = sb.st_ino; *inode = sb.st_ino;
ret = 0;
cleanup: return 0;
VIR_FREE(path);
return ret;
} }
...@@ -929,8 +923,8 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver, ...@@ -929,8 +923,8 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver,
const char *pidfile) const char *pidfile)
{ {
size_t i; size_t i;
char *filterstr; g_autofree char *filterstr = NULL;
char *outputstr; g_autofree char *outputstr = NULL;
virCommandPtr cmd; virCommandPtr cmd;
virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver); virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver);
...@@ -950,7 +944,6 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver, ...@@ -950,7 +944,6 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver,
} }
virCommandAddEnvPair(cmd, "LIBVIRT_LOG_FILTERS", filterstr); virCommandAddEnvPair(cmd, "LIBVIRT_LOG_FILTERS", filterstr);
VIR_FREE(filterstr);
} }
if (cfg->log_libvirtd) { if (cfg->log_libvirtd) {
...@@ -962,7 +955,6 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver, ...@@ -962,7 +955,6 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver,
} }
virCommandAddEnvPair(cmd, "LIBVIRT_LOG_OUTPUTS", outputstr); virCommandAddEnvPair(cmd, "LIBVIRT_LOG_OUTPUTS", outputstr);
VIR_FREE(outputstr);
} }
} else { } else {
virCommandAddEnvFormat(cmd, virCommandAddEnvFormat(cmd,
...@@ -985,12 +977,11 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver, ...@@ -985,12 +977,11 @@ virLXCProcessBuildControllerCmd(virLXCDriverPtr driver,
for (i = 0; i < VIR_LXC_DOMAIN_NAMESPACE_LAST; i++) { for (i = 0; i < VIR_LXC_DOMAIN_NAMESPACE_LAST; i++) {
if (nsInheritFDs[i] > 0) { if (nsInheritFDs[i] > 0) {
char *tmp = NULL; g_autofree char *tmp = g_strdup_printf("--share-%s",
tmp = g_strdup_printf("--share-%s", nsInfoLocal[i]); nsInfoLocal[i]);
virCommandAddArg(cmd, tmp); virCommandAddArg(cmd, tmp);
virCommandAddArgFormat(cmd, "%d", nsInheritFDs[i]); virCommandAddArgFormat(cmd, "%d", nsInheritFDs[i]);
virCommandPassFD(cmd, nsInheritFDs[i], 0); virCommandPassFD(cmd, nsInheritFDs[i], 0);
VIR_FREE(tmp);
} }
} }
...@@ -1188,15 +1179,15 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1188,15 +1179,15 @@ int virLXCProcessStart(virConnectPtr conn,
{ {
int rc = -1, r; int rc = -1, r;
size_t nttyFDs = 0; size_t nttyFDs = 0;
int *ttyFDs = NULL; g_autofree int *ttyFDs = NULL;
size_t i; size_t i;
char *logfile = NULL; g_autofree char *logfile = NULL;
int logfd = -1; int logfd = -1;
VIR_AUTOSTRINGLIST veths = NULL; VIR_AUTOSTRINGLIST veths = NULL;
int handshakefds[2] = { -1, -1 }; int handshakefds[2] = { -1, -1 };
off_t pos = -1; off_t pos = -1;
char ebuf[1024]; char ebuf[1024];
char *timestamp; g_autofree char *timestamp = NULL;
int nsInheritFDs[VIR_LXC_DOMAIN_NAMESPACE_LAST]; int nsInheritFDs[VIR_LXC_DOMAIN_NAMESPACE_LAST];
virCommandPtr cmd = NULL; virCommandPtr cmd = NULL;
virLXCDomainObjPrivatePtr priv = vm->privateData; virLXCDomainObjPrivatePtr priv = vm->privateData;
...@@ -1205,7 +1196,7 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1205,7 +1196,7 @@ int virLXCProcessStart(virConnectPtr conn,
virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver); virLXCDriverConfigPtr cfg = virLXCDriverGetConfig(driver);
virCgroupPtr selfcgroup; virCgroupPtr selfcgroup;
int status; int status;
char *pidfile = NULL; g_autofree char *pidfile = NULL;
if (virCgroupNewSelf(&selfcgroup) < 0) if (virCgroupNewSelf(&selfcgroup) < 0)
return -1; return -1;
...@@ -1283,18 +1274,14 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1283,18 +1274,14 @@ int virLXCProcessStart(virConnectPtr conn,
/* Run an early hook to set-up missing devices */ /* Run an early hook to set-up missing devices */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
int hookret;
hookret = virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_PREPARE, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL);
VIR_FREE(xml);
/* /*
* If the script raised an error abort the launch * If the script raised an error abort the launch
*/ */
if (hookret < 0) if (virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_PREPARE, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL) < 0)
goto cleanup; goto cleanup;
} }
...@@ -1348,10 +1335,10 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1348,10 +1335,10 @@ int virLXCProcessStart(virConnectPtr conn,
goto cleanup; goto cleanup;
} }
VIR_FREE(vm->def->consoles[i]->source->data.file.path); g_free(vm->def->consoles[i]->source->data.file.path);
vm->def->consoles[i]->source->data.file.path = ttyPath; vm->def->consoles[i]->source->data.file.path = ttyPath;
VIR_FREE(vm->def->consoles[i]->info.alias); g_free(vm->def->consoles[i]->info.alias);
vm->def->consoles[i]->info.alias = g_strdup_printf("console%zu", i); vm->def->consoles[i]->info.alias = g_strdup_printf("console%zu", i);
} }
...@@ -1388,18 +1375,14 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1388,18 +1375,14 @@ int virLXCProcessStart(virConnectPtr conn,
/* now that we know it is about to start call the hook if present */ /* now that we know it is about to start call the hook if present */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
int hookret;
hookret = virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_START, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL);
VIR_FREE(xml);
/* /*
* If the script raised an error abort the launch * If the script raised an error abort the launch
*/ */
if (hookret < 0) if (virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_START, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL) < 0)
goto cleanup; goto cleanup;
} }
...@@ -1411,7 +1394,6 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1411,7 +1394,6 @@ int virLXCProcessStart(virConnectPtr conn,
VIR_WARN("Unable to write timestamp to logfile: %s", VIR_WARN("Unable to write timestamp to logfile: %s",
g_strerror(errno)); g_strerror(errno));
} }
VIR_FREE(timestamp);
/* Log generated command line */ /* Log generated command line */
virCommandWriteArgLog(cmd, logfd); virCommandWriteArgLog(cmd, logfd);
...@@ -1530,18 +1512,14 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1530,18 +1512,14 @@ int virLXCProcessStart(virConnectPtr conn,
/* finally we can call the 'started' hook script if any */ /* finally we can call the 'started' hook script if any */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
int hookret;
hookret = virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_STARTED, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL);
VIR_FREE(xml);
/* /*
* If the script raised an error abort the launch * If the script raised an error abort the launch
*/ */
if (hookret < 0) if (virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_STARTED, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL) < 0)
goto cleanup; goto cleanup;
} }
...@@ -1559,11 +1537,8 @@ int virLXCProcessStart(virConnectPtr conn, ...@@ -1559,11 +1537,8 @@ int virLXCProcessStart(virConnectPtr conn,
virCommandFree(cmd); virCommandFree(cmd);
for (i = 0; i < nttyFDs; i++) for (i = 0; i < nttyFDs; i++)
VIR_FORCE_CLOSE(ttyFDs[i]); VIR_FORCE_CLOSE(ttyFDs[i]);
VIR_FREE(ttyFDs);
VIR_FORCE_CLOSE(handshakefds[0]); VIR_FORCE_CLOSE(handshakefds[0]);
VIR_FORCE_CLOSE(handshakefds[1]); VIR_FORCE_CLOSE(handshakefds[1]);
VIR_FREE(pidfile);
VIR_FREE(logfile);
virObjectUnref(cfg); virObjectUnref(cfg);
virObjectUnref(caps); virObjectUnref(caps);
...@@ -1710,15 +1685,12 @@ virLXCProcessReconnectDomain(virDomainObjPtr vm, ...@@ -1710,15 +1685,12 @@ virLXCProcessReconnectDomain(virDomainObjPtr vm,
/* now that we know it's reconnected call the hook if present */ /* now that we know it's reconnected call the hook if present */
if (virHookPresent(VIR_HOOK_DRIVER_LXC)) { if (virHookPresent(VIR_HOOK_DRIVER_LXC)) {
char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0); g_autofree char *xml = virDomainDefFormat(vm->def, driver->xmlopt, 0);
int hookret;
/* we can't stop the operation even if the script raised an error */ /* we can't stop the operation even if the script raised an error */
hookret = virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name, if (virHookCall(VIR_HOOK_DRIVER_LXC, vm->def->name,
VIR_HOOK_LXC_OP_RECONNECT, VIR_HOOK_SUBOP_BEGIN, VIR_HOOK_LXC_OP_RECONNECT, VIR_HOOK_SUBOP_BEGIN,
NULL, xml, NULL); NULL, xml, NULL) < 0)
VIR_FREE(xml);
if (hookret < 0)
goto error; goto error;
} }
......
Markdown is supported
0% 或 .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑！
想要评论请 注册 或