From c5ec45a3a4ae3a268d390f0ec88b9b56cd6ec189 Mon Sep 17 00:00:00 2001 From: Jiri Denemark Date: Thu, 17 Jun 2010 16:45:27 +0200 Subject: [PATCH] Remove unnecessary check for non-NULL uuid The first thing we do in all these functions is to check uuid != NULL and fail if it isn't. --- src/datatypes.c | 12 ++++-------- 1 file changed, 4 insertions(+), 8 deletions(-) diff --git a/src/datatypes.c b/src/datatypes.c index 20b2d74cc8..8750406b08 100644 --- a/src/datatypes.c +++ b/src/datatypes.c @@ -369,8 +369,7 @@ virGetDomain(virConnectPtr conn, const char *name, const unsigned char *uuid) { ret->magic = VIR_DOMAIN_MAGIC; ret->conn = conn; ret->id = -1; - if (uuid != NULL) - memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); + memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); ret->snapshots = virHashCreate(20); if (virHashAddEntry(conn->domains, uuidstr, ret) < 0) { @@ -515,8 +514,7 @@ virGetNetwork(virConnectPtr conn, const char *name, const unsigned char *uuid) { } ret->magic = VIR_NETWORK_MAGIC; ret->conn = conn; - if (uuid != NULL) - memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); + memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); if (virHashAddEntry(conn->networks, uuidstr, ret) < 0) { virMutexUnlock(&conn->lock); @@ -837,8 +835,7 @@ virGetStoragePool(virConnectPtr conn, const char *name, const unsigned char *uui } ret->magic = VIR_STORAGE_POOL_MAGIC; ret->conn = conn; - if (uuid != NULL) - memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); + memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); if (virHashAddEntry(conn->storagePools, uuidstr, ret) < 0) { virMutexUnlock(&conn->lock); @@ -1452,8 +1449,7 @@ virGetNWFilter(virConnectPtr conn, const char *name, const unsigned char *uuid) } ret->magic = VIR_NWFILTER_MAGIC; ret->conn = conn; - if (uuid != NULL) - memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); + memcpy(&(ret->uuid[0]), uuid, VIR_UUID_BUFLEN); if (virHashAddEntry(conn->nwfilterPools, uuidstr, ret) < 0) { virMutexUnlock(&conn->lock);