From c4f9edf1a1efda21271f6cf5d08e574548488a3f Mon Sep 17 00:00:00 2001 From: "Daniel P. Berrange" Date: Wed, 3 Apr 2013 14:14:23 +0100 Subject: [PATCH] Use VIR_ALLOC_VAR instead of VIR_ALLOC_N for creating virObject The current way virObject instances are allocated using VIR_ALLOC_N causes alignment warnings util/virobject.c: In function 'virObjectNew': util/virobject.c:195:11: error: cast increases required alignment of target type [-Werror=cast-align] Changing to use VIR_ALLOC_VAR will avoid the need todo the casts entirely. Signed-off-by: Daniel P. Berrange --- src/util/virobject.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/util/virobject.c b/src/util/virobject.c index 808edc4837..93e37e4a5d 100644 --- a/src/util/virobject.c +++ b/src/util/virobject.c @@ -186,13 +186,13 @@ bool virClassIsDerivedFrom(virClassPtr klass, void *virObjectNew(virClassPtr klass) { virObjectPtr obj = NULL; - char *somebytes; - if (VIR_ALLOC_N(somebytes, klass->objectSize) < 0) { + if (VIR_ALLOC_VAR(obj, + char, + klass->objectSize - sizeof(virObject)) < 0) { virReportOOMError(); return NULL; } - obj = (virObjectPtr)somebytes; obj->magic = klass->magic; obj->klass = klass;