From f238a86c711e77e53bd169dc63b1fed4e784d98f Mon Sep 17 00:00:00 2001 From: Vitezslav Crhonek Date: Tue, 4 Feb 2014 13:03:44 +0100 Subject: [PATCH] Fix sblim-gather FTBFS if -Werror=format-security flag is used --- sblim-gather-2.2.8-format-security.patch | 59 ++++++++++++++++++++++++ sblim-gather.spec | 9 +++- 2 files changed, 67 insertions(+), 1 deletion(-) create mode 100644 sblim-gather-2.2.8-format-security.patch diff --git a/sblim-gather-2.2.8-format-security.patch b/sblim-gather-2.2.8-format-security.patch new file mode 100644 index 0000000..6cec3e2 --- /dev/null +++ b/sblim-gather-2.2.8-format-security.patch @@ -0,0 +1,59 @@ +diff -up sblim-gather-2.2.8/comms/rcctest.c.orig sblim-gather-2.2.8/comms/rcctest.c +--- sblim-gather-2.2.8/comms/rcctest.c.orig 2014-02-04 12:53:25.036297751 +0100 ++++ sblim-gather-2.2.8/comms/rcctest.c 2014-02-04 12:53:43.738362560 +0100 +@@ -42,7 +42,7 @@ int main(int argc, char *argv[]) + if (argc == 1) { + gethostname((char*)&hostname,sizeof(hostname)); + } else { +- sprintf(hostname,argv[1]); ++ sprintf(hostname,"%s",argv[1]); + } + printf("Contacting %s\n",hostname); + if (rcc_init(hostname,&port) < 0 ) { +diff -up sblim-gather-2.2.8/gatherctl.c.orig sblim-gather-2.2.8/gatherctl.c +--- sblim-gather-2.2.8/gatherctl.c.orig 2013-02-12 02:08:25.000000000 +0100 ++++ sblim-gather-2.2.8/gatherctl.c 2014-02-04 12:53:06.888233912 +0100 +@@ -146,5 +146,5 @@ static void printhelp() + { + int i; + for (i=0;commands[i];i++) +- printf(commands[i]); ++ printf("%s",commands[i]); + } +diff -up sblim-gather-2.2.8/reposctl.c.orig sblim-gather-2.2.8/reposctl.c +--- sblim-gather-2.2.8/reposctl.c.orig 2013-02-12 02:08:25.000000000 +0100 ++++ sblim-gather-2.2.8/reposctl.c 2014-02-04 12:53:06.888233912 +0100 +@@ -271,7 +271,7 @@ static void printhelp() + { + int i; + for (i=0;commands[i];i++) +- printf(commands[i]); ++ printf("%s",commands[i]); + } + + static void printvalue(ValueRequest *vr) +@@ -317,7 +317,7 @@ static void printvalue(ValueRequest *vr) + printf("%f",*(double*)vr->vsValues[i].viValue); + break; + case MD_STRING: +- printf(vr->vsValues[i].viValue); ++ printf("%s",vr->vsValues[i].viValue); + break; + default: + printf("datatype %0x not supported",vr->vsDataType); +diff -up sblim-gather-2.2.8/util/mlog.c.orig sblim-gather-2.2.8/util/mlog.c +--- sblim-gather-2.2.8/util/mlog.c.orig 2013-02-12 02:08:25.000000000 +0100 ++++ sblim-gather-2.2.8/util/mlog.c 2014-02-04 12:53:06.888233912 +0100 +@@ -52,10 +52,10 @@ void m_log(int priority, int errout, con + va_start(ap,fmt); + + vsnprintf(buf,4096,fmt,ap); +- syslog(priosysl,buf); ++ syslog(priosysl,"%s",buf); + + if (errout) { +- fprintf(stderr,buf); ++ fprintf(stderr,"%s",buf); + } + va_end(ap); + } diff --git a/sblim-gather.spec b/sblim-gather.spec index 5e7130a..ecec73f 100644 --- a/sblim-gather.spec +++ b/sblim-gather.spec @@ -4,7 +4,7 @@ Name: sblim-gather Version: 2.2.8 -Release: 4%{?dist} +Release: 5%{?dist} Summary: SBLIM Gatherer Group: Applications/System @@ -31,6 +31,8 @@ Patch2: sblim-gather-2.2.7-typos.patch # Patch3: removes version from docdir Patch3: sblim-gather-2.2.8-docdir.patch +# Patch4: fixes build fail if "-Werror=format-security" flag is used +Patch4: sblim-gather-2.2.8-format-security.patch Requires: tog-pegasus >= %{tog_pegasus_version} Requires(post): systemd @@ -88,6 +90,7 @@ tar xfvz %{SOURCE4} %patch1 -p1 -b .missing_providers %patch2 -p1 -b .typos %patch3 -p1 -b .docdir +%patch4 -p1 -b .format-security %build %ifarch s390 s390x ppc ppc64 @@ -240,6 +243,10 @@ fi %postun provider -p /sbin/ldconfig %changelog +* Tue Feb 04 2014 Vitezslav Crhonek - 2.2.8-5 +- Fix sblim-gather FTBFS if "-Werror=format-security" flag is used + Resolves: #1037318 + * Wed Aug 28 2013 Vitezslav Crhonek - 2.2.8-4 - Fix for unversioned docdir change Resolves: #994086