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:46:05 +0100
commit18ac5a8fe17bf3bf78c3bdbc0f22fea8c388d70a (patch)
tree3cab50fcbf073d62bd3bfced7f1ec22c28b3b633
parent7071c972b4dfaadbc2e6bae6d9876315be4d1d5d (diff)
downloadxen-18ac5a8fe17bf3bf78c3bdbc0f22fea8c388d70a.tar.gz
xen-18ac5a8fe17bf3bf78c3bdbc0f22fea8c388d70a.tar.bz2
xen-18ac5a8fe17bf3bf78c3bdbc0f22fea8c388d70a.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) (cherry picked from commit e36c0917dd54c932816e11a525f294101c77557d)
-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 b69c65dd8a..2519d58f38 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);