aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBastian Blank <bastian@waldi.eu.org>2013-08-11 22:10:20 +0200
committerIan Jackson <Ian.Jackson@eu.citrix.com>2013-09-13 14:39:51 +0100
commite36c0917dd54c932816e11a525f294101c77557d (patch)
tree418aa548e7d31b685c772b8beb8071e805266f19
parent797e29bee13b80fc162aab0e19722045a11e8f65 (diff)
downloadxen-e36c0917dd54c932816e11a525f294101c77557d.tar.gz
xen-e36c0917dd54c932816e11a525f294101c77557d.tar.bz2
xen-e36c0917dd54c932816e11a525f294101c77557d.zip
tools: xen-mceinj: Add missing return value checks
The return value of vasprintf must be checked. This check is enforced with the compiler options used in Debian by request and in Ubuntu by default. Check the return value and abort on error. Signed-off-by: Bastian Blank <waldi@debian.org> Acked-by: Ian Jackson <ian.jackson@eu.citrix.com> (cherry picked from commit 1efe90faa31be104a24fe75323429d227eae1d9f)
-rw-r--r--tools/tests/mce-test/tools/xen-mceinj.c6
1 files changed, 4 insertions, 2 deletions
diff --git a/tools/tests/mce-test/tools/xen-mceinj.c b/tools/tests/mce-test/tools/xen-mceinj.c
index e3e62f7855..21a488b48f 100644
--- a/tools/tests/mce-test/tools/xen-mceinj.c
+++ b/tools/tests/mce-test/tools/xen-mceinj.c
@@ -92,7 +92,8 @@ static void Lprintf(const char *fmt, ...)
va_list args;
va_start(args, fmt);
- vasprintf(&buf, fmt, args);
+ if (vasprintf(&buf, fmt, args) < 0)
+ abort();
fprintf(LOGFILE, "%s", buf);
va_end(args);
free(buf);
@@ -104,7 +105,8 @@ static void err(xc_interface *xc_handle, const char *fmt, ...)
va_list args;
va_start(args, fmt);
- vasprintf(&buf, fmt, args);
+ if (vasprintf(&buf, fmt, args) < 0)
+ abort();
perror(buf);
va_end(args);
free(buf);