]> xenbits.xensource.com Git - libvirt.git/commitdiff
Do not check the return value of closedir
authorJán Tomko <jtomko@redhat.com>
Tue, 21 Jun 2016 11:20:48 +0000 (13:20 +0200)
committerJán Tomko <jtomko@redhat.com>
Thu, 23 Jun 2016 19:57:45 +0000 (21:57 +0200)
The only possible error is EBADFD.
Since we only use the directory stream returned by opendir,
this should never happen.

src/util/virhostcpu.c

index 4ed7b217d85fbc58687aae906410bb1f6301c891..0cdba0a0cd8791df1673a6e31c116831cc720fe1 100644 (file)
@@ -477,11 +477,8 @@ virHostCPUParseNode(const char *node,
     ret = processors;
 
  cleanup:
-    /* don't shadow a more serious error */
-    if (cpudir && closedir(cpudir) < 0 && ret >= 0) {
-        virReportSystemError(errno, _("problem closing %s"), node);
-        ret = -1;
-    }
+    if (cpudir)
+        closedir(cpudir);
     if (cores_maps)
         for (i = 0; i < sock_max; i++)
             virBitmapFree(cores_maps[i]);
@@ -777,12 +774,8 @@ virHostCPUGetInfoPopulateLinux(FILE *cpuinfo,
     ret = 0;
 
  cleanup:
-    /* don't shadow a more serious error */
-    if (nodedir && closedir(nodedir) < 0 && ret >= 0) {
-        virReportSystemError(errno, _("problem closing %s"), sysfs_nodedir);
-        ret = -1;
-    }
-
+    if (nodedir)
+        closedir(nodedir);
     virBitmapFree(present_cpus_map);
     virBitmapFree(online_cpus_map);
     VIR_FREE(sysfs_nodedir);