[packages/cpqarrayd] - fix format string errors - rel 6

baggins baggins at pld-linux.org
Mon Sep 3 10:40:30 CEST 2018


commit 2e2e74a836f7572eb4cb8fde42bc7b7715dd3b91
Author: Jan Rękorajski <baggins at pld-linux.org>
Date:   Mon Sep 3 10:40:13 2018 +0200

    - fix format string errors
    - rel 6

 cpqarrayd.spec        |  4 +++-
 format-security.patch | 64 +++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 67 insertions(+), 1 deletion(-)
---
diff --git a/cpqarrayd.spec b/cpqarrayd.spec
index cdcb508..5eaa294 100644
--- a/cpqarrayd.spec
+++ b/cpqarrayd.spec
@@ -2,7 +2,7 @@ Summary:	Cpqarrayd - SmartArray controllers monitoring
 Summary(pl.UTF-8):	Cpqarrayd - monitorowanie kontrolerów SmartArray
 Name:		cpqarrayd
 Version:	2.3
-Release:	5
+Release:	6
 License:	GPL v2+
 Group:		Applications/System
 Source0:	http://www.strocamp.net/opensource/compaq/downloads/%{name}-%{version}.tar.gz
@@ -10,6 +10,7 @@ Source0:	http://www.strocamp.net/opensource/compaq/downloads/%{name}-%{version}.
 Source1:	%{name}.init
 Source2:	%{name}.sysconfig
 Patch0:		%{name}-headers.patch
+Patch1:		format-security.patch
 URL:		http://www.strocamp.net/opensource/cpqarrayd.php
 BuildRequires:	autoconf >= 2.50
 BuildRequires:	automake
@@ -29,6 +30,7 @@ wysyłanie pułapek SNMP oraz sysloga.
 %prep
 %setup -q
 %patch0 -p1
+%patch1 -p1
 
 %build
 %{__aclocal}
diff --git a/format-security.patch b/format-security.patch
new file mode 100644
index 0000000..eb84681
--- /dev/null
+++ b/format-security.patch
@@ -0,0 +1,64 @@
+--- cpqarrayd-2.3/status.c.orig	2007-12-03 18:06:22.000000000 +0100
++++ cpqarrayd-2.3/status.c	2018-09-03 10:37:05.375891671 +0200
+@@ -140,8 +140,7 @@
+ 		   "problem sending snmp trap (sendtrap() returned %d)\n",
+ 		   trap_stat);
+ 	    if (opts.debug) {
+-	      printf("DEBUG: Problem sending snmp trap",
+-		     "(sendtrap() returned %d)\n", 
++	      printf("DEBUG: Problem sending snmp trap (sendtrap() returned %d)\n", 
+ 		     trap_stat);
+ 	    }
+ 	  }
+@@ -167,8 +168,7 @@
+ 		   "problem sending snmp trap (sendtrap() returned %d)\n",
+ 		   trap_stat);
+ 	    if (opts.debug) {
+-	      printf("DEBUG: Problem sending snmp trap ",
+-		     "(sendtrap() returned %d)\n",
++	      printf("DEBUG: Problem sending snmp trap (sendtrap() returned %d)\n",
+ 		     trap_stat);
+ 	    }
+ 	  }
+@@ -215,17 +217,16 @@
+ 		 event.detail.logstatchange.logicaldrivenumber,
+ 		 logicaldrivestatusstr[event.detail.logstatchange.newlogicaldrivestate]);
+ 	status = event.detail.logstatchange.newlogicaldrivestate;
+-	syslog(LOG_WARNING, statusmsg);
++	syslog(LOG_WARNING, "%s", statusmsg);
+ 	if (opts.debug) {
+-	  printf (statusmsg);
++	  puts (statusmsg);
+ 	}
+ 	if (trap_stat = sendtrap(opts, "public", status, statusmsg)) {
+ 	  syslog(LOG_WARNING, 
+ 		 "problem sending snmp trap (sendtrap() returned %d)\n",
+ 		 trap_stat);
+ 	  if (opts.debug) {
+-	    printf("DEBUG: Problem sending snmp trap",
+-		   "(sendtrap() returned %d)\n", 
++	    printf("DEBUG: Problem sending snmp trap (sendtrap() returned %d)\n", 
+ 		   trap_stat);
+ 	  }
+ 	}
+@@ -234,17 +237,16 @@
+ 		 ctrls_found[ctrl_cntr].devicefile,
+ 		 event.mesgstring);
+ 	status = 255;
+-	syslog(LOG_WARNING, statusmsg);
++	syslog(LOG_WARNING, "%s", statusmsg);
+ 	if (opts.debug) {
+-	  printf (statusmsg);
++	  puts (statusmsg);
+ 	}
+ 	if (trap_stat = sendtrap(opts, "public", status, statusmsg)) {
+ 	  syslog(LOG_WARNING, 
+ 		 "problem sending snmp trap (sendtrap() returned %d)\n",
+ 		 trap_stat);
+ 	  if (opts.debug) {
+-	    printf("DEBUG: Problem sending snmp trap",
+-		   "(sendtrap() returned %d)\n", 
++	    printf("DEBUG: Problem sending snmp trap (sendtrap() returned %d)\n", 
+ 		   trap_stat);
+ 	  }
+ 	}
================================================================

---- gitweb:

http://git.pld-linux.org/gitweb.cgi/packages/cpqarrayd.git/commitdiff/2e2e74a836f7572eb4cb8fde42bc7b7715dd3b91



More information about the pld-cvs-commit mailing list