[gegl] tiff: Do not pass warning strings directly to printf function



commit 79e220cb41957903c3d8c3106f389af925694842
Author: Martin Blanchard <tchaik gmx com>
Date:   Sun Mar 20 22:00:57 2016 +0100

    tiff: Do not pass warning strings directly to printf function

 operations/external/tiff-load.c |   16 ++++++++--------
 operations/external/tiff-save.c |   16 ++++++++--------
 2 files changed, 16 insertions(+), 16 deletions(-)
---
diff --git a/operations/external/tiff-load.c b/operations/external/tiff-load.c
index 4ca7e91..a190247 100644
--- a/operations/external/tiff-load.c
+++ b/operations/external/tiff-load.c
@@ -119,7 +119,7 @@ error_handler(const char *module,
   gchar *message;
 
   g_vasprintf(&message, format, arguments);
-  g_warning(message);
+  g_warning("%s", message);
 
   g_free(message);
 }
@@ -132,7 +132,7 @@ warning_handler(const char *module,
   gchar *message;
 
   g_vasprintf(&message, format, arguments);
-  g_message(message);
+  g_message("%s", message);
 
   g_free(message);
 }
@@ -158,7 +158,7 @@ read_from_stream(thandle_t handle,
                                  NULL, &error);
       if (read < 0)
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
     }
@@ -189,7 +189,7 @@ read_from_stream(thandle_t handle,
                                          NULL, &error);
               if (read < 0)
                 {
-                  g_warning(error->message);
+                  g_warning("%s", error->message);
                   g_error_free(error);
                   break;
                 }
@@ -242,7 +242,7 @@ seek_in_stream(thandle_t handle,
         position = g_seekable_tell(G_SEEKABLE(p->stream));
       else
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
     }
@@ -284,7 +284,7 @@ close_stream(thandle_t handle)
                                 NULL, &error);
   if (!closed)
     {
-      g_warning(error->message);
+      g_warning("%s", error->message);
       g_error_free(error);
     }
 
@@ -322,7 +322,7 @@ get_file_size(thandle_t handle)
                                NULL, &error);
       if (info == NULL)
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
       else
@@ -728,7 +728,7 @@ prepare(GeglOperation *operation)
         p->can_seek = g_seekable_can_seek(G_SEEKABLE(p->stream));
       if (p->stream == NULL)
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
           cleanup(operation);
           return;
diff --git a/operations/external/tiff-save.c b/operations/external/tiff-save.c
index 54ecff6..f0ce688 100644
--- a/operations/external/tiff-save.c
+++ b/operations/external/tiff-save.c
@@ -94,7 +94,7 @@ error_handler(const char *module,
   gchar *message;
 
   g_vasprintf(&message, format, arguments);
-  g_warning(message);
+  g_warning("%s", message);
 
   g_free(message);
 }
@@ -107,7 +107,7 @@ warning_handler(const char *module,
   gchar *message;
 
   g_vasprintf(&message, format, arguments);
-  g_message(message);
+  g_message("%s", message);
 
   g_free(message);
 }
@@ -144,7 +144,7 @@ write_to_stream(thandle_t handle,
                                       NULL, &error);
       if (written < 0)
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
     }
@@ -192,7 +192,7 @@ seek_in_stream(thandle_t handle,
         position = g_seekable_tell(G_SEEKABLE(p->stream));
       else
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
     }
@@ -243,7 +243,7 @@ close_stream(thandle_t handle)
                                           NULL, &error);
           if (written < 0)
             {
-                  g_warning(error->message);
+                  g_warning("%s", error->message);
                   g_error_free(error);
                   break;
             }
@@ -256,7 +256,7 @@ close_stream(thandle_t handle)
                                  NULL, &error);
   if (!closed)
     {
-      g_warning(error->message);
+      g_warning("%s", error->message);
       g_error_free(error);
     }
 
@@ -293,7 +293,7 @@ get_file_size(thandle_t handle)
                                NULL, &error);
       if (info == NULL)
         {
-          g_warning(error->message);
+          g_warning("%s", error->message);
           g_error_free(error);
         }
       else
@@ -567,7 +567,7 @@ process(GeglOperation *operation,
   if (p->stream == NULL)
     {
       status = FALSE;
-      g_warning(error->message);
+      g_warning("%s", error->message);
       goto cleanup;
     }
 


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]