Re: [libvirt] [PATCH] nodeinfo: skip offline CPUs

On 08/10/2010 04:01 PM, Daniel Veillard wrote: Thanks for the review. I'm squashing this in before applying, based on an IRC comment by Dave Allan that 1024 is an awfully big stack allocation when we know that the value will fit in an int. diff --git i/src/nodeinfo.c w/src/nodeinfo.c index 2a20679..59e0163 100644 --- i/src/nodeinfo.c +++ w/src/nodeinfo.c @@ -48,6 +48,7 @@ #include "logging.h" #include "virterror_internal.h" #include "count-one-bits.h" +#include "intprops.h" #define VIR_FROM_THIS VIR_FROM_NONE @@ -72,7 +73,7 @@ static int get_cpu_value(unsigned int cpu, const char *file, bool missing_ok) { char *path; FILE *pathfp; - char value_str[1024]; + char value_str[INT_BUFSIZE_BOUND(int)]; char *tmp; int value = -1; -- Eric Blake eblake(a)redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org