External/libvirt
1
0
Fork 0
mirror of https://gitlab.com/libvirt/libvirt.git synced 2024-07-11 04:15:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

src: use g_new0 instead of VIR_ALLOC

Browse Source 
Signed-off-by: Ján Tomko <jtomko@redhat.com>
Reviewed-by: Pavel Hrdina <phrdina@redhat.com>
This commit is contained in:
Ján Tomko 2020-09-23 20:44:29 +02:00
parent 3109f6d86c
commit a80a81be65
6 changed files with 32 additions and 60 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/access/viraccessmanager.c
View File

@ -91,8 +91,7 @@ static virAccessManagerPtr virAccessManagerNewDriver(virAccessDriverPtr drv)
if (virAccessManagerInitialize() < 0)
return NULL;
if (VIR_ALLOC_N(privateData, drv->privateDataLen) < 0)
return NULL;
privateData = g_new0(char, drv->privateDataLen);
if (!(mgr = virObjectLockableNew(virAccessManagerClass))) {
VIR_FREE(privateData);

6
src/admin/admin_server_dispatch.c
View File

@ -88,8 +88,7 @@ remoteAdmClientNew(virNetServerClientPtr client G_GNUC_UNUSED,
return NULL;
}
if (VIR_ALLOC(priv) < 0)
return NULL;
priv = g_new0(struct daemonAdmClientPrivate, 1);
if (virMutexInit(&priv->lock) < 0) {
VIR_FREE(priv);
@ -511,8 +510,7 @@ adminDispatchConnectGetLoggingFilters(virNetServerPtr server G_GNUC_UNUSED,
ret->filters = NULL;
} else {
char **ret_filters = NULL;
if (VIR_ALLOC(ret_filters) < 0)
return -1;
ret_filters = g_new0(char *, 1);
*ret_filters = filters;
ret->filters = ret_filters;

3
src/hypervisor/domain_driver.c
View File

@ -219,8 +219,7 @@ virDomainDriverParseBlkioDeviceStr(char *blkioDeviceStr, const char *type,
ndevices = (nsep + 1) / 2;
if (VIR_ALLOC_N(result, ndevices) < 0)
return -1;
result = g_new0(virBlkioDevice, ndevices);
i = 0;
temp = blkioDeviceStr;

7
src/hypervisor/virclosecallbacks.c
View File

@ -120,9 +120,7 @@ virCloseCallbacksSet(virCloseCallbacksPtr closeCallbacks,
closeDef->cb = cb;
} else {
if (VIR_ALLOC(closeDef) < 0)
goto cleanup;
closeDef = g_new0(virDriverCloseDef, 1);
closeDef->conn = conn;
closeDef->cb = cb;
if (virHashAddEntry(closeCallbacks->list, uuidstr, closeDef) < 0) {
@ -284,8 +282,7 @@ virCloseCallbacksGetForConn(virCloseCallbacksPtr closeCallbacks,
virCloseCallbacksListPtr list = NULL;
struct virCloseCallbacksData data;
if (VIR_ALLOC(list) < 0)
return NULL;
list = g_new0(virCloseCallbacksList, 1);
data.conn = conn;
data.list = list;

7
src/libxl/xen_xm.c
View File

@ -42,8 +42,7 @@ xenParseXMOS(virConfPtr conf, virDomainDefPtr def)
if (def->os.type == VIR_DOMAIN_OSTYPE_HVM) {
g_autofree char *boot = NULL;
if (VIR_ALLOC(def->os.loader) < 0)
return -1;
def->os.loader = g_new0(virDomainLoaderDef, 1);
if (xenConfigCopyString(conf, "kernel", &def->os.loader->path) < 0)
return -1;
@ -346,9 +345,7 @@ xenFormatXMDisk(virConfValuePtr list,
return -1;
}
if (VIR_ALLOC(val) < 0)
return -1;
val = g_new0(virConfValue, 1);
val->type = VIR_CONF_STRING;
val->str = virBufferContentAndReset(&buf);
tmp = list->list;

66
src/test/test_driver.c
View File

@ -212,15 +212,14 @@ testDomainDefNamespaceParse(xmlXPathContextPtr ctxt,
unsigned int tmpuint;
g_autofree xmlNodePtr *nodes = NULL;
if (VIR_ALLOC(nsdata) < 0)
return -1;
nsdata = g_new0(testDomainNamespaceDef, 1);
n = virXPathNodeSet("./test:domainsnapshot", ctxt, &nodes);
if (n < 0)
goto error;
if (n && VIR_ALLOC_N(nsdata->snap_nodes, n) < 0)
goto error;
if (n)
nsdata->snap_nodes = g_new0(xmlNodePtr, n);
for (i = 0; i < n; i++) {
xmlNodePtr newnode = xmlCopyNode(nodes[i], 1);
@ -302,8 +301,7 @@ testBuildCapabilities(virConnectPtr conn)
virCapabilitiesHostInitIOMMU(caps);
if (VIR_ALLOC_N(caps->host.pagesSize, 4) < 0)
goto error;
caps->host.pagesSize = g_new0(unsigned int, 4);
caps->host.pagesSize[caps->host.nPagesSize++] = 4;
caps->host.pagesSize[caps->host.nPagesSize++] = 8;
@ -316,11 +314,8 @@ testBuildCapabilities(virConnectPtr conn)
virCapsHostNUMACellPageInfoPtr pages;
size_t nPages = caps->host.nPagesSize - 1;
if (VIR_ALLOC_N(cpu_cells, privconn->cells[i].numCpus) < 0 ||
VIR_ALLOC_N(pages, nPages) < 0) {
VIR_FREE(cpu_cells);
goto error;
}
cpu_cells = g_new0(virCapsHostNUMACellCPU, privconn->cells[i].numCpus);
pages = g_new0(virCapsHostNUMACellPageInfo, nPages);
memcpy(cpu_cells, privconn->cells[i].cpus,
sizeof(*cpu_cells) * privconn->cells[i].numCpus);
@ -364,8 +359,7 @@ testBuildCapabilities(virConnectPtr conn)
}
caps->host.nsecModels = 1;
if (VIR_ALLOC_N(caps->host.secModels, caps->host.nsecModels) < 0)
goto error;
caps->host.secModels = g_new0(virCapsHostSecModel, caps->host.nsecModels);
caps->host.secModels[0].model = g_strdup("testSecurity");
caps->host.secModels[0].doi = g_strdup("");
@ -396,8 +390,7 @@ testDomainObjPrivateAlloc(void *opaque)
{
testDomainObjPrivatePtr priv;
if (VIR_ALLOC(priv) < 0)
return NULL;
priv = g_new0(testDomainObjPrivate, 1);
priv->driver = opaque;
priv->frozen[0] = priv->frozen[1] = false;
@ -760,8 +753,7 @@ static char *testBuildFilename(const char *relativeTo,
if ((baseLen = (offset-relativeTo+1))) {
char *absFile;
int totalLen = baseLen + strlen(filename) + 1;
if (VIR_ALLOC_N(absFile, totalLen) < 0)
return NULL;
absFile = g_new0(char, totalLen);
if (virStrncpy(absFile, relativeTo, baseLen, totalLen) < 0) {
VIR_FREE(absFile);
return NULL;
@ -1233,8 +1225,8 @@ testParseAuthUsers(testDriverPtr privconn,
return -1;
privconn->numAuths = num;
if (num && VIR_ALLOC_N(privconn->auths, num) < 0)
return -1;
if (num)
privconn->auths = g_new0(testAuth, num);
for (i = 0; i < num; i++) {
g_autofree char *username = NULL;
@ -2298,8 +2290,7 @@ testDomainSaveImageOpen(testDriverPtr driver,
goto error;
}
if (VIR_ALLOC_N(xml, len+1) < 0)
goto error;
xml = g_new0(char, len + 1);
if (saferead(fd, xml, len) != len) {
virReportSystemError(errno, _("incomplete metadata in '%s'"), path);
@ -3491,10 +3482,9 @@ testDomainSetInterfaceParameters(virDomainPtr dom,
if (!(net = virDomainNetFind(def, device)))
goto cleanup;
if ((VIR_ALLOC(bandwidth) < 0) ||
(VIR_ALLOC(bandwidth->in) < 0) ||
(VIR_ALLOC(bandwidth->out) < 0))
goto cleanup;
bandwidth = g_new0(virNetDevBandwidth, 1);
bandwidth->in = g_new0(virNetDevBandwidthRate, 1);
bandwidth->out = g_new0(virNetDevBandwidthRate, 1);
for (i = 0; i < nparams; i++) {
virTypedParameterPtr param = &params[i];
@ -4726,21 +4716,17 @@ testDomainGetFSInfo(virDomainPtr dom,
if (vm->def->disks[i]->device == VIR_DOMAIN_DISK_DEVICE_DISK) {
char *name = vm->def->disks[i]->dst;
if (VIR_ALLOC_N(info_ret, 2) < 0)
goto cleanup;
if (VIR_ALLOC(info_ret[0]) < 0 ||
VIR_ALLOC(info_ret[0]->devAlias) < 0)
goto cleanup;
info_ret = g_new0(virDomainFSInfo *, 2);
info_ret[0] = g_new0(virDomainFSInfo, 1);
info_ret[0]->devAlias = g_new0(char *, 1);
info_ret[0]->mountpoint = g_strdup("/");
info_ret[0]->fstype = g_strdup("ext4");
info_ret[0]->devAlias[0] = g_strdup(name);
info_ret[0]->name = g_strdup_printf("%s1", name);
if (VIR_ALLOC(info_ret[1]) < 0 ||
VIR_ALLOC(info_ret[1]->devAlias) < 0)
goto cleanup;
info_ret[1] = g_new0(virDomainFSInfo, 1);
info_ret[1]->devAlias = g_new0(char *, 1);
info_ret[1]->mountpoint = g_strdup("/boot");
info_ret[1]->fstype = g_strdup("ext4");
@ -5076,22 +5062,19 @@ testDomainInterfaceAddresses(virDomainPtr dom,
if (virDomainObjCheckActive(vm) < 0)
goto cleanup;
if (VIR_ALLOC_N(ifaces_ret, vm->def->nnets) < 0)
goto cleanup;
ifaces_ret = g_new0(virDomainInterfacePtr, vm->def->nnets);
for (i = 0; i < vm->def->nnets; i++) {
const virDomainNetDef *net = vm->def->nets[i];
if (VIR_ALLOC(iface) < 0)
goto cleanup;
iface = g_new0(virDomainInterface, 1);
iface->name = g_strdup(net->ifname);
virMacAddrFormat(&net->mac, macaddr);
iface->hwaddr = g_strdup(macaddr);
if (VIR_ALLOC(iface->addrs) < 0)
goto cleanup;
iface->addrs = g_new0(virDomainIPAddress, 1);
iface->naddrs = 1;
if (net->type == VIR_DOMAIN_NET_TYPE_NETWORK) {
@ -7741,8 +7724,7 @@ testNodeGetCPUMap(virConnectPtr conn G_GNUC_UNUSED,
virCheckFlags(0, -1);
if (cpumap) {
if (VIR_ALLOC_N(*cpumap, 1) < 0)
return -1;
*cpumap = g_new0(unsigned char, 1);
*cpumap[0] = 0x15;
}