From a001a5e28b920a6a89f1e4c47ef311a988f7f341 Mon Sep 17 00:00:00 2001 From: Alex Jia Date: Wed, 30 Nov 2011 15:50:44 +0800 Subject: [PATCH] util: avoid null deref on qcowXGetBackingStore Detected by Coverity. the only case is caller passes a NULL to 'format' variable, then taking 'if (format)' false branch, the function qcow2GetBackingStoreFormat will directly dereferences the NULL 'format' pointer variable. Signed-off-by: Alex Jia --- src/util/storage_file.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/util/storage_file.c b/src/util/storage_file.c index f33ea74a92..ba9cfc51c9 100644 --- a/src/util/storage_file.c +++ b/src/util/storage_file.c @@ -333,7 +333,7 @@ qcowXGetBackingStore(char **res, * between the end of the header (QCOW2_HDR_TOTAL_SIZE) * and the start of the backingStoreName (offset) */ - if (isQCow2) + if (isQCow2 && format) qcow2GetBackingStoreFormat(format, buf, buf_size, QCOW2_HDR_TOTAL_SIZE, offset); return BACKING_STORE_OK; -- GitLab