changeset 2268:4fed9f38e501

Don't generate a random pointer from a pthread_t in the debug output. 2010-06-14 Andrew John Hughes <ahughes@redhat.com> Don't print out the return value of pthread_self which is a pthread_t. pthread_t is an opaque type and we don't know what it actually is (it varies from system to system; recent versions of Linux use an unsigned long int). * plugin/icedteanp/IcedTeaPluginUtils.h: (PLUGIN_DEBUG_0ARG(str)): Don't print the thread_t. (PLUGIN_DEBUG_1ARG(str,arg1)): Likewise. (PLUGIN_DEBUG_2ARG(str,arg1,arg2)): Likewise. (PLUGIN_DEBUG_3ARG(str,arg1,arg2,arg3)): Likewise. (PLUGIN_DEBUG_4ARG(str,arg1,arg2,arg3,arg4)): Likewise. (PLUGIN_DEBUG_5ARG(str,arg1,arg2,arg3,arg4,arg5)): Likewise.
author Andrew John Hughes <ahughes@redhat.com>
date Tue, 15 Jun 2010 18:34:45 +0100
parents c4a1546cbcbd
children c829a9d16fa0
files ChangeLog plugin/icedteanp/IcedTeaPluginUtils.h
diffstat 2 files changed, 21 insertions(+), 6 deletions(-) [+]
line wrap: on
line diff
--- a/ChangeLog	Mon Jun 14 15:03:55 2010 -0400
+++ b/ChangeLog	Tue Jun 15 18:34:45 2010 +0100
@@ -1,3 +1,18 @@
+2010-06-14 Andrew John Hughes  <ahughes@redhat.com>
+
+	Don't print out the return value of pthread_self
+	which is a pthread_t.  pthread_t is an opaque type
+	and we don't know what it actually is (it varies
+	from system to system; recent versions of Linux
+	use an unsigned long int).
+	* plugin/icedteanp/IcedTeaPluginUtils.h:
+	(PLUGIN_DEBUG_0ARG(str)): Don't print the thread_t.
+	(PLUGIN_DEBUG_1ARG(str,arg1)): Likewise.
+	(PLUGIN_DEBUG_2ARG(str,arg1,arg2)): Likewise.
+	(PLUGIN_DEBUG_3ARG(str,arg1,arg2,arg3)): Likewise.
+	(PLUGIN_DEBUG_4ARG(str,arg1,arg2,arg3,arg4)): Likewise.
+	(PLUGIN_DEBUG_5ARG(str,arg1,arg2,arg3,arg4,arg5)): Likewise.
+
 2010-06-14  Deepak Bhole <dbhole@redhat.com>
 
 	* plugin/icedteanp/java/sun/applet/PluginMessageConsumer.java: Fix bug
--- a/plugin/icedteanp/IcedTeaPluginUtils.h	Mon Jun 14 15:03:55 2010 -0400
+++ b/plugin/icedteanp/IcedTeaPluginUtils.h	Tue Jun 15 18:34:45 2010 +0100
@@ -71,7 +71,7 @@
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str);                \
     }                                       \
   } while (0)
@@ -81,7 +81,7 @@
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1);          \
     }                                       \
   } while (0)
@@ -91,7 +91,7 @@
   {                                         \
     if (plugin_debug)                       \
     {                                       \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2);    \
     }                                       \
   } while (0)
@@ -101,7 +101,7 @@
   {                                            \
     if (plugin_debug)                          \
     {                                          \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3); \
     }                                          \
   } while (0)
@@ -111,7 +111,7 @@
   {                                                  \
     if (plugin_debug)                                \
     {                                                \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3, arg4); \
     }                                                \
   } while (0)
@@ -121,7 +121,7 @@
   {                                                  \
     if (plugin_debug)                                \
     {                                                \
-      fprintf(stderr, "ICEDTEA NP PLUGIN: thread %p: ", pthread_self()); \
+      fprintf(stderr, "ICEDTEA NP PLUGIN: "); \
       fprintf(stderr, str, arg1, arg2, arg3, arg4, arg5); \
     }                                                \
   } while (0)