[packages/libldm] - add -Werror=format-security fixes to warnings patch

qboosh qboosh at pld-linux.org
Mon Jan 7 19:52:17 CET 2013


commit c39a493f702a12884db04988135f39c794907454
Author: Jakub Bogusz <qboosh at pld-linux.org>
Date:   Mon Jan 7 19:52:18 2013 +0100

    - add -Werror=format-security fixes to warnings patch

 libldm-warnings.patch | 38 ++++++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)
---
diff --git a/libldm-warnings.patch b/libldm-warnings.patch
index 92fea1a..92b7b16 100644
--- a/libldm-warnings.patch
+++ b/libldm-warnings.patch
@@ -169,3 +169,41 @@
      }
  
      if (memcmp(privhead->magic, "PRIVHEAD", 8) != 0) {
+--- libldm-0.2.3/src/ldm.c.orig	2013-01-07 19:44:31.767064210 +0100
++++ libldm-0.2.3/src/ldm.c	2013-01-07 19:50:07.909184339 +0100
+@@ -2759,7 +2759,7 @@
+         GString * chunk = _dm_create_part(part, cookie, err);
+         if (chunk == NULL) {
+             if (err && (*err)->code == LDM_ERROR_MISSING_DISK) {
+-                g_warning((*err)->message);
++                g_warning("%s", (*err)->message);
+                 g_error_free(*err); *err = NULL;
+                 g_string_append(target.params, " - -");
+                 continue;
+@@ -2804,7 +2804,7 @@
+         for (int i = devices->len; i > 0; i--) {
+             GString *device = g_array_index(devices, GString *, i - 1);
+             if (!_dm_remove(device->str, 0, &cleanup_err)) {
+-                g_warning(cleanup_err->message);
++                g_warning("%s", cleanup_err->message);
+                 g_error_free(cleanup_err); cleanup_err = NULL;
+             }
+         }
+@@ -2847,7 +2847,7 @@
+         GString * chunk = _dm_create_part(part, cookie, err);
+         if (chunk == NULL) {
+             if (err && (*err)->code == LDM_ERROR_MISSING_DISK) {
+-                g_warning((*err)->message);
++                g_warning("%s", (*err)->message);
+                 g_error_free(*err); *err = NULL;
+                 g_string_append(target.params, " - -");
+                 continue;
+@@ -2892,7 +2892,7 @@
+         for (int i = devices->len; i > 0; i--) {
+             GString *device = g_array_index(devices, GString *, i - 1);
+             if (!_dm_remove(device->str, 0, &cleanup_err)) {
+-                g_warning(cleanup_err->message);
++                g_warning("%s", cleanup_err->message);
+                 g_error_free(cleanup_err); cleanup_err = NULL;
+             }
+         }
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/libldm.git/commitdiff/c39a493f702a12884db04988135f39c794907454



More information about the pld-cvs-commit mailing list