diff options
| author | Bastian Blank <bastian@waldi.eu.org> | 2013-08-11 22:10:20 +0200 |
|---|---|---|
| committer | Ian Jackson <Ian.Jackson@eu.citrix.com> | 2013-08-19 15:31:13 +0100 |
| commit | 1efe90faa31be104a24fe75323429d227eae1d9f (patch) | |
| tree | 2f86017a91ddd3e29c472015fdbf80ba150e5c38 | |
| parent | 3da32f9dafe6016decb9ebbad008cca6fab818ca (diff) | |
| download | xen-1efe90faa31be104a24fe75323429d227eae1d9f.tar.gz xen-1efe90faa31be104a24fe75323429d227eae1d9f.tar.bz2 xen-1efe90faa31be104a24fe75323429d227eae1d9f.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>
| -rw-r--r-- | tools/tests/mce-test/tools/xen-mceinj.c | 6 |
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); |