[Xen-changelog] [xen stable-4.2] tools: xen-mceinj: Add missing return value checks

[patchbot@xxxxxxx]

commit 18ac5a8fe17bf3bf78c3bdbc0f22fea8c388d70a
Author:     Bastian Blank <bastian@xxxxxxxxxxxx>
AuthorDate: Sun Aug 11 22:10:20 2013 +0200
Commit:     Ian Jackson <Ian.Jackson@xxxxxxxxxxxxx>
CommitDate: Fri Sep 13 14:46:05 2013 +0100

    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@xxxxxxxxxx>
    Acked-by: Ian Jackson <ian.jackson@xxxxxxxxxxxxx>
    (cherry picked from commit 1efe90faa31be104a24fe75323429d227eae1d9f)
    (cherry picked from commit e36c0917dd54c932816e11a525f294101c77557d)
---
 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 b69c65d..2519d58 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);
--
generated by git-patchbot for /home/xen/git/xen.git#stable-4.2

_______________________________________________
Xen-changelog mailing list
Xen-changelog@xxxxxxxxxxxxx
http://lists.xensource.com/xen-changelog