Avoid using stderr to be as robust as possible in out-of-memory.
authorOwen Taylor <otaylor@redhat.com>
Mon, 14 May 2001 14:53:59 +0000 (14:53 +0000)
committerOwen Taylor <otaylor@src.gnome.org>
Mon, 14 May 2001 14:53:59 +0000 (14:53 +0000)
Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>

* gmessages.c (g_log_write_prefix): Avoid using stderr
to be as robust as possible in out-of-memory.

* gmessages.c (g_log_default_handler): Remove some dead
code.

* gutils.c (g_parse_debug_string): Fix to avoid mallocs.

12 files changed:
ChangeLog
ChangeLog.pre-2-0
ChangeLog.pre-2-10
ChangeLog.pre-2-12
ChangeLog.pre-2-2
ChangeLog.pre-2-4
ChangeLog.pre-2-6
ChangeLog.pre-2-8
glib/gmessages.c
glib/gutils.c
gmessages.c
gutils.c

index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 0db0474ca8d4d4e1a49ffd4f7c1912e6ba6986aa..dd54851e7df2cb88aaa6586d837c95b9e91e2458 100644 (file)
@@ -1,3 +1,13 @@
+Thu May 10 23:21:30 2001  Owen Taylor  <otaylor@redhat.com>
+
+       * gmessages.c (g_log_write_prefix): Avoid using stderr
+       to be as robust as possible in out-of-memory.
+
+       * gmessages.c (g_log_default_handler): Remove some dead
+       code.
+
+       * gutils.c (g_parse_debug_string): Fix to avoid mallocs.
+
 Fri May 11 18:25:23 2001  Tim Janik  <timj@gtk.org>
 
        * gdataset.c:
index 3a4aa33f805d07239c16499064c9e204ce93e9a9..6db19332a024796a8b37198ec8d22a066b9847d9 100644 (file)
@@ -150,6 +150,59 @@ ensure_stdout_valid (void)
 #define ensure_stdout_valid()  /* Define as empty */
 #endif
 
+static void
+write_unsigned (gint   fd,
+               gulong num,
+               guint  radix)
+{
+  char buffer[64];
+  gulong tmp;
+  char c;
+  int i, n;
+
+  g_return_if_fail (radix >= 2 && radix <= 36);
+  
+  if (!num)
+    {
+      write (fd, "0", 1);
+      return;
+    } 
+  
+  if (radix == 16)
+    write (fd, "0x", 2);
+  else if (radix == 8)
+    write (fd, "0", 1);
+       
+  n = 0;
+  tmp = num;
+  while (tmp)
+    {
+      tmp /= radix;
+      n++;
+    }
+
+  i = n;
+  while (num)
+    {
+      i--;
+      c = (num % radix);
+      if (c < 10)
+       buffer[i] = c + '0';
+      else
+       buffer[i] = c + 'a' - 10;
+      num /= radix;
+    }
+  
+  write (fd, buffer, n);
+}
+
+static void
+write_string (gint   fd,
+             gchar *string)
+{
+  write (fd, string, strlen (string));
+}
+
 static void
 g_log_write_prefix (gint           fd,
                     GLogLevelFlags mask)
@@ -185,20 +238,20 @@ g_log_write_prefix (gint           fd,
   
   if ((g_log_msg_prefix & mask) == mask)
     {
-      gchar prg_pid[64], *prg_name;
+      gchar *prg_name;
 
       prg_name = g_get_prgname ();
-      
+
       if (!prg_name)
-        {
-          prg_name = "(process";
-          sprintf (prg_pid, ":%u): ", getpid ());
-        }
+       write_string (fd, "(process:");
       else
-        sprintf (prg_pid, " (pid:%u): ", getpid ());
-      
-      write (fd, prg_name, strlen (prg_name));
-      write (fd, prg_pid, strlen (prg_pid));
+       {
+         write_string (fd, prg_name);
+         write_string (fd, " (pid:");
+       }
+
+      write_unsigned (fd, getpid (), 10);
+      write_string (fd, "): ");
     }
 }
 
@@ -530,7 +583,6 @@ g_log_default_handler (const gchar    *log_domain,
   GErrorFunc local_glib_error_func;
   GWarningFunc local_glib_warning_func;
   GPrintFunc local_glib_message_func;
-  gchar prg_pid[64], *prg_name = g_get_prgname ();
   
   in_recursion = (log_level & G_LOG_FLAG_RECURSION) != 0;
   is_fatal = (log_level & G_LOG_FLAG_FATAL) != 0;
@@ -538,13 +590,6 @@ g_log_default_handler (const gchar    *log_domain,
   
   if (!message)
     message = "g_log_default_handler(): (NULL) message";
-  if (!prg_name)
-    {
-      prg_name = "(process";
-      sprintf (prg_pid, ":%u): ", getpid ());
-    }
-  else
-    sprintf (prg_pid, " (pid:%u): ", getpid ());
   
 #ifdef G_OS_WIN32
   /* Use just stdout as stderr is hard to get redirected from the
index f00262c8a9e3e53d58fa13c498a5c27132d2733f..ac53d2b2e0bc7a8ecfa1ab0a6a1a0a56c135625b 100644 (file)
@@ -437,9 +437,8 @@ g_parse_debug_string  (const gchar     *string,
     }
   else
     {
-      gchar *str = g_strdup (string);
-      gchar *p = str;
-      gchar *q;
+      const gchar *p = string;
+      const gchar *q;
       gboolean done = FALSE;
       
       while (*p && !done)
@@ -451,16 +450,13 @@ g_parse_debug_string  (const gchar     *string,
              done = TRUE;
            }
          
-         *q = 0;
-         
          for (i=0; i<nkeys; i++)
-           if (!g_strcasecmp(keys[i].key, p))
+           if (g_strncasecmp(keys[i].key, p, q - p) == 0 &&
+               keys[i].key[q - p] == '\0')
              result |= keys[i].value;
          
-         p = q+1;
+         p = q + 1;
        }
-      
-      g_free (str);
     }
   
   return result;
index 3a4aa33f805d07239c16499064c9e204ce93e9a9..6db19332a024796a8b37198ec8d22a066b9847d9 100644 (file)
@@ -150,6 +150,59 @@ ensure_stdout_valid (void)
 #define ensure_stdout_valid()  /* Define as empty */
 #endif
 
+static void
+write_unsigned (gint   fd,
+               gulong num,
+               guint  radix)
+{
+  char buffer[64];
+  gulong tmp;
+  char c;
+  int i, n;
+
+  g_return_if_fail (radix >= 2 && radix <= 36);
+  
+  if (!num)
+    {
+      write (fd, "0", 1);
+      return;
+    } 
+  
+  if (radix == 16)
+    write (fd, "0x", 2);
+  else if (radix == 8)
+    write (fd, "0", 1);
+       
+  n = 0;
+  tmp = num;
+  while (tmp)
+    {
+      tmp /= radix;
+      n++;
+    }
+
+  i = n;
+  while (num)
+    {
+      i--;
+      c = (num % radix);
+      if (c < 10)
+       buffer[i] = c + '0';
+      else
+       buffer[i] = c + 'a' - 10;
+      num /= radix;
+    }
+  
+  write (fd, buffer, n);
+}
+
+static void
+write_string (gint   fd,
+             gchar *string)
+{
+  write (fd, string, strlen (string));
+}
+
 static void
 g_log_write_prefix (gint           fd,
                     GLogLevelFlags mask)
@@ -185,20 +238,20 @@ g_log_write_prefix (gint           fd,
   
   if ((g_log_msg_prefix & mask) == mask)
     {
-      gchar prg_pid[64], *prg_name;
+      gchar *prg_name;
 
       prg_name = g_get_prgname ();
-      
+
       if (!prg_name)
-        {
-          prg_name = "(process";
-          sprintf (prg_pid, ":%u): ", getpid ());
-        }
+       write_string (fd, "(process:");
       else
-        sprintf (prg_pid, " (pid:%u): ", getpid ());
-      
-      write (fd, prg_name, strlen (prg_name));
-      write (fd, prg_pid, strlen (prg_pid));
+       {
+         write_string (fd, prg_name);
+         write_string (fd, " (pid:");
+       }
+
+      write_unsigned (fd, getpid (), 10);
+      write_string (fd, "): ");
     }
 }
 
@@ -530,7 +583,6 @@ g_log_default_handler (const gchar    *log_domain,
   GErrorFunc local_glib_error_func;
   GWarningFunc local_glib_warning_func;
   GPrintFunc local_glib_message_func;
-  gchar prg_pid[64], *prg_name = g_get_prgname ();
   
   in_recursion = (log_level & G_LOG_FLAG_RECURSION) != 0;
   is_fatal = (log_level & G_LOG_FLAG_FATAL) != 0;
@@ -538,13 +590,6 @@ g_log_default_handler (const gchar    *log_domain,
   
   if (!message)
     message = "g_log_default_handler(): (NULL) message";
-  if (!prg_name)
-    {
-      prg_name = "(process";
-      sprintf (prg_pid, ":%u): ", getpid ());
-    }
-  else
-    sprintf (prg_pid, " (pid:%u): ", getpid ());
   
 #ifdef G_OS_WIN32
   /* Use just stdout as stderr is hard to get redirected from the
index f00262c8a9e3e53d58fa13c498a5c27132d2733f..ac53d2b2e0bc7a8ecfa1ab0a6a1a0a56c135625b 100644 (file)
--- a/gutils.c
+++ b/gutils.c
@@ -437,9 +437,8 @@ g_parse_debug_string  (const gchar     *string,
     }
   else
     {
-      gchar *str = g_strdup (string);
-      gchar *p = str;
-      gchar *q;
+      const gchar *p = string;
+      const gchar *q;
       gboolean done = FALSE;
       
       while (*p && !done)
@@ -451,16 +450,13 @@ g_parse_debug_string  (const gchar     *string,
              done = TRUE;
            }
          
-         *q = 0;
-         
          for (i=0; i<nkeys; i++)
-           if (!g_strcasecmp(keys[i].key, p))
+           if (g_strncasecmp(keys[i].key, p, q - p) == 0 &&
+               keys[i].key[q - p] == '\0')
              result |= keys[i].value;
          
-         p = q+1;
+         p = q + 1;
        }
-      
-      g_free (str);
     }
   
   return result;