On Wed, 2015-08-05 at 14:19 +0200, Jiri Denemark wrote:
On Tue, Aug 04, 2015 at 11:37:54 +0200, Andrea Bolognani wrote:
--- src/cpu/cpu_ppc64.c | 3 +++ 1 file changed, 3 insertions(+)
diff --git a/src/cpu/cpu_ppc64.c b/src/cpu/cpu_ppc64.c index 05ff8f2..dd02a3f 100644 --- a/src/cpu/cpu_ppc64.c +++ b/src/cpu/cpu_ppc64.c @@ -115,6 +115,9 @@ ppc64ModelCopy(const struct ppc64_model *model) { struct ppc64_model *copy;
+ if (!model) + return NULL; + if (VIR_ALLOC(copy) < 0 || VIR_STRDUP(copy->name, model->name) < 0) { ppc64ModelFree(copy);
This doesn't seem to be really necessary since the function is not called with model == NULL and I don't think that should change. If any caller wants to pass NULL for a model, it should rather report an error and return instead of trying to copy this NULL.
I've dropped this commit and removed a similar check from ppc64DataCopy(), which will be introduced later in the series. Cheers. -- Andrea Bolognani Software Engineer - Virtualization Team