From 3cef9f6adbdcea67cf8c134381345d82909b75c5 Mon Sep 17 00:00:00 2001 From: Michal Privoznik Date: Fri, 10 May 2013 12:05:28 +0200 Subject: [PATCH] virGetStorageVol: Don't ignore NULL pool name The function takes pool name as argument. However, it is not acceptable for it to be NULL. Hence, we should check it and report error in case it is. --- src/datatypes.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/datatypes.c b/src/datatypes.c index f815b6c861..884eb3e954 100644 --- a/src/datatypes.c +++ b/src/datatypes.c @@ -520,6 +520,7 @@ virGetStorageVol(virConnectPtr conn, const char *pool, const char *name, virLibConnError(VIR_ERR_INVALID_CONN, "%s", _("no connection")); return NULL; } + virCheckNonNullArgReturn(pool, NULL); virCheckNonNullArgReturn(name, NULL); virCheckNonNullArgReturn(key, NULL);