[libvirt] [PATCH] S390: Buffer too small for large CPU numbers.

Viktor Mihajlovski mihajlov at linux.vnet.ibm.com
Fri Oct 5 13:59:08 UTC 2012


The output buffer for virFileReadAll was too small for systems with
more than 30 CPUs which leads to a log entry and incorrect behavior.
The new size will be sufficient for the current
architectural limits.

Signed-off-by: Viktor Mihajlovski <mihajlov at linux.vnet.ibm.com>
---
 src/util/sysinfo.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/util/sysinfo.c b/src/util/sysinfo.c
index d38b4ff..8add936 100644
--- a/src/util/sysinfo.c
+++ b/src/util/sysinfo.c
@@ -361,7 +361,7 @@ virSysinfoRead(void) {
         goto no_memory;
 
     /* Gather info from /proc/cpuinfo */
-    if (virFileReadAll(CPUINFO, 2048, &outbuf) < 0) {
+    if (virFileReadAll(CPUINFO, 8192, &outbuf) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Failed to open %s"), CPUINFO);
         return NULL;
@@ -376,7 +376,7 @@ virSysinfoRead(void) {
     VIR_FREE(outbuf);
 
     /* Gather info from /proc/sysinfo */
-    if (virFileReadAll(SYSINFO, 4096, &outbuf) < 0) {
+    if (virFileReadAll(SYSINFO, 8192, &outbuf) < 0) {
         virReportError(VIR_ERR_INTERNAL_ERROR,
                        _("Failed to open %s"), SYSINFO);
         return NULL;
-- 
1.7.0.4




More information about the libvir-list mailing list