]> git.ipfire.org Git - thirdparty/libvirt.git/commitdiff
domain_conf.c: Initialize arrVar and cntVar
authorMichal Privoznik <mprivozn@redhat.com>
Tue, 15 Oct 2013 17:32:28 +0000 (19:32 +0200)
committerMichal Privoznik <mprivozn@redhat.com>
Tue, 15 Oct 2013 17:34:10 +0000 (19:34 +0200)
Some ancient gcc fails to see the variables are initialized in a
separate function and a false positive is produced:

cc1: warnings being treated as errors
conf/domain_conf.c: In function 'virDomainChrGetDomainPtrs':
conf/domain_conf.c:10342: error: 'arrVar' may be used uninitialized in this function [-Wuninitialized]
conf/domain_conf.c:10343: error: 'cntVar' may be used uninitialized in this function [-Wuninitialized]
conf/domain_conf.c: In function 'virDomainChrInsert':
conf/domain_conf.c:10362: error: 'arrPtr' may be used uninitialized in this function [-Wuninitialized]
conf/domain_conf.c:10363: error: 'cntPtr' may be used uninitialized in this function [-Wuninitialized]
conf/domain_conf.c: In function 'virDomainChrRemove':
conf/domain_conf.c:10374: error: 'arrPtr' may be used uninitialized in this function [-Wuninitialized]
conf/domain_conf.c:10375: error: 'cntPtr' may be used uninitialized in this function [-Wuninitialized]

Signed-off-by: Michal Privoznik <mprivozn@redhat.com>
src/conf/domain_conf.c

index 365de7744cff16e3183afdfc1e234ad61bed49c8..562d98b032c9ba85642a57419d514cd9544115ad 100644 (file)
@@ -10339,8 +10339,8 @@ virDomainChrGetDomainPtrs(const virDomainDef *vmdef,
                           const virDomainChrDef ***arrPtr,
                           size_t *cntPtr)
 {
-    virDomainChrDef ***arrVar;
-    size_t *cntVar;
+    virDomainChrDef ***arrVar = NULL;
+    size_t *cntVar = NULL;
 
     /* Cast away const; we add it back in the final assignment.  */
     virDomainChrGetDomainPtrsInternal((virDomainDefPtr) vmdef, type,
@@ -10359,8 +10359,8 @@ int
 virDomainChrInsert(virDomainDefPtr vmdef,
                    virDomainChrDefPtr chr)
 {
-    virDomainChrDefPtr **arrPtr;
-    size_t *cntPtr;
+    virDomainChrDefPtr **arrPtr = NULL;
+    size_t *cntPtr = NULL;
 
     virDomainChrGetDomainPtrsInternal(vmdef, chr->deviceType, &arrPtr, &cntPtr);
 
@@ -10371,8 +10371,8 @@ virDomainChrDefPtr
 virDomainChrRemove(virDomainDefPtr vmdef,
                    virDomainChrDefPtr chr)
 {
-    virDomainChrDefPtr ret, **arrPtr;
-    size_t i, *cntPtr;
+    virDomainChrDefPtr ret, **arrPtr = NULL;
+    size_t i, *cntPtr = NULL;
 
     virDomainChrGetDomainPtrsInternal(vmdef, chr->deviceType, &arrPtr, &cntPtr);