commit 05b07158841423adc250f04e034bf11e6f892b23
author sewardj <sewardj@a5019735-40e9-0310-863c-91ae7b9d1cf9> Mon Jan 04 01:01:02 2010 +0000
committer sewardj <sewardj@a5019735-40e9-0310-863c-91ae7b9d1cf9> Mon Jan 04 01:01:02 2010 +0000
tree c4e128dcfc57a5c5a61cc25b819ddacba78cc6e5
parent 41f12420a2081699294c2bffbefb5f16e2220c46

Don't assume that va_list can be casted to an unsigned word.  Fixes
the build on arm-linux with gcc-4.4 or later.  Patch from Dmitry
Zhurikhin.



git-svn-id: svn://svn.valgrind.org/valgrind/trunk@11006 a5019735-40e9-0310-863c-91ae7b9d1cf9

coregrind/m_scheduler/scheduler.c

diff --git a/coregrind/m_scheduler/scheduler.c b/coregrind/m_scheduler/scheduler.c
index 63d38a1..67fb7bf 100644
--- a/coregrind/m_scheduler/scheduler.c
+++ b/coregrind/m_scheduler/scheduler.c
@@ -1408,15 +1408,27 @@
          break;
 
       case VG_USERREQ__PRINTF: {
-         Int count = 
-            VG_(vmessage)( Vg_ClientMsg, (char *)arg[1], (void*)arg[2] );
+         union {
+            va_list vargs;
+            unsigned long ul;
+         } args;
+         Int count;
+         args.ul = (unsigned long)arg[2];
+         count =
+            VG_(vmessage)( Vg_ClientMsg, (char *)arg[1], args.vargs );
             VG_(message_flush)();
             SET_CLREQ_RETVAL( tid, count );
          break; }
 
       case VG_USERREQ__INTERNAL_PRINTF: {
-         Int count = 
-            VG_(vmessage)( Vg_DebugMsg, (char *)arg[1], (void*)arg[2] );
+         union {
+            va_list vargs;
+            unsigned long ul;
+         } args;
+         Int count;
+         args.ul = (unsigned long)arg[2];
+         count =
+            VG_(vmessage)( Vg_DebugMsg, (char *)arg[1], args.vargs );
             VG_(message_flush)();
             SET_CLREQ_RETVAL( tid, count );
          break; }
@@ -1427,8 +1439,14 @@
          break; }
 
       case VG_USERREQ__PRINTF_BACKTRACE: {
-         Int count =
-            VG_(vmessage)( Vg_ClientMsg, (char *)arg[1], (void*)arg[2] );
+         union {
+            va_list vargs;
+            unsigned long ul;
+         } args;
+         Int count;
+         args.ul = (unsigned long)arg[2];
+         count =
+            VG_(vmessage)( Vg_ClientMsg, (char *)arg[1], args.vargs );
             VG_(message_flush)();
             VG_(get_and_pp_StackTrace)( tid, VG_(clo_backtrace_size) );
             SET_CLREQ_RETVAL( tid, count );

coregrind/pub_core_clreq.h

diff --git a/coregrind/pub_core_clreq.h b/coregrind/pub_core_clreq.h
index 412f89e..d998021 100644
--- a/coregrind/pub_core_clreq.h
+++ b/coregrind/pub_core_clreq.h
@@ -64,13 +64,16 @@
 static int VALGRIND_INTERNAL_PRINTF(const char *format, ...)
 {
    unsigned long _qzz_res = 0;
-   va_list vargs;
-   va_start(vargs, format);
+   union {
+      va_list vargs;
+      unsigned long ul;
+   } args;
+   va_start(args.vargs, format);
    VALGRIND_DO_CLIENT_REQUEST(
       _qzz_res, 0, VG_USERREQ__INTERNAL_PRINTF,
-      (unsigned long)format, (unsigned long)vargs, 0, 0, 0
+      (unsigned long)format, (unsigned long)(args.ul), 0, 0, 0
    );
-   va_end(vargs);
+   va_end(args.vargs);
    return _qzz_res;
 }
 

include/valgrind.h

diff --git a/include/valgrind.h b/include/valgrind.h
index fc46465..e311f66 100644
--- a/include/valgrind.h
+++ b/include/valgrind.h
@@ -4183,12 +4183,16 @@
 VALGRIND_PRINTF(const char *format, ...)
 {
    unsigned long _qzz_res;
-   va_list vargs;
-   va_start(vargs, format);
+   union {
+      va_list vargs;
+      unsigned long ul;
+   } args;
+   va_start(args.vargs, format);
    VALGRIND_DO_CLIENT_REQUEST(_qzz_res, 0, VG_USERREQ__PRINTF,
-                              (unsigned long)format, (unsigned long)vargs, 
+                              (unsigned long)format,
+                              (unsigned long)(args.ul),
                               0, 0, 0);
-   va_end(vargs);
+   va_end(args.vargs);
    return (int)_qzz_res;
 }
 
@@ -4198,12 +4202,16 @@
 VALGRIND_PRINTF_BACKTRACE(const char *format, ...)
 {
    unsigned long _qzz_res;
-   va_list vargs;
-   va_start(vargs, format);
+   union {
+      va_list vargs;
+      unsigned long ul;
+   } args;
+   va_start(args.vargs, format);
    VALGRIND_DO_CLIENT_REQUEST(_qzz_res, 0, VG_USERREQ__PRINTF_BACKTRACE,
-                              (unsigned long)format, (unsigned long)vargs, 
+                              (unsigned long)format,
+                              (unsigned long)(args.ul),
                               0, 0, 0);
-   va_end(vargs);
+   va_end(args.vargs);
    return (int)_qzz_res;
 }