From dfd358be63ba30c401185d53f59d70fc0da0db58 Mon Sep 17 00:00:00 2001 From: Hauke Mehrtens Date: Sun, 22 Nov 2015 14:13:47 +0000 Subject: [PATCH] xfsprogs: activate format-security checks This patch was taken from upstream. Signed-off-by: Hauke Mehrtens SVN-Revision: 47581 --- package/utils/xfsprogs/Makefile | 1 - .../patches/001-automake-compat.patch | 6 ---- .../utils/xfsprogs/patches/100-no_aio.patch | 2 -- .../patches/110-uclibc_no_ustat.patch | 6 ++-- .../xfsprogs/patches/120-portability.patch | 4 +-- .../patches/130-uclibc_no_xattr.patch | 6 ++-- .../utils/xfsprogs/patches/140-no_po.patch | 9 +++--- .../patches/160-format-security.patch | 29 +++++++++++++++++++ 8 files changed, 38 insertions(+), 25 deletions(-) create mode 100644 package/utils/xfsprogs/patches/160-format-security.patch diff --git a/package/utils/xfsprogs/Makefile b/package/utils/xfsprogs/Makefile index 6333077dbd..638ec4c67b 100644 --- a/package/utils/xfsprogs/Makefile +++ b/package/utils/xfsprogs/Makefile @@ -16,7 +16,6 @@ PKG_MD5SUM:=049cf9873794ea49d0bb3f12d45748a4 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz PKG_INSTALL:=1 PKG_FIXUP:=autoreconf -PKG_CHECK_FORMAT_SECURITY:=0 include $(INCLUDE_DIR)/package.mk include $(INCLUDE_DIR)/nls.mk diff --git a/package/utils/xfsprogs/patches/001-automake-compat.patch b/package/utils/xfsprogs/patches/001-automake-compat.patch index 3003d6d210..ce9289a4af 100644 --- a/package/utils/xfsprogs/patches/001-automake-compat.patch +++ b/package/utils/xfsprogs/patches/001-automake-compat.patch @@ -4,8 +4,6 @@ Date: Thu Jan 26 00:34:15 2012 +0100 001-automake-compat -diff --git a/configure.in b/configure.in -index 664c0e9..d91b6ec 100644 --- a/configure.in +++ b/configure.in @@ -2,7 +2,8 @@ AC_INIT(include/libxfs.h) @@ -18,8 +16,6 @@ index 664c0e9..d91b6ec 100644 AC_PREFIX_DEFAULT(/usr) AC_PROG_LIBTOOL -diff --git a/include/builddefs.in b/include/builddefs.in -index 81ebfcd..5a4a0e8 100644 --- a/include/builddefs.in +++ b/include/builddefs.in @@ -20,6 +20,8 @@ @@ -31,8 +27,6 @@ index 81ebfcd..5a4a0e8 100644 DEBUG = @debug_build@ OPTIMIZER = @opt_build@ MALLOCLIB = @malloc_lib@ -diff --git a/m4/package_types.m4 b/m4/package_types.m4 -index 0a0e087..66a136a 100644 --- a/m4/package_types.m4 +++ b/m4/package_types.m4 @@ -9,7 +9,7 @@ AC_DEFUN([AC_TYPE_PSINT], diff --git a/package/utils/xfsprogs/patches/100-no_aio.patch b/package/utils/xfsprogs/patches/100-no_aio.patch index 30d36b4da8..4cc85d7deb 100644 --- a/package/utils/xfsprogs/patches/100-no_aio.patch +++ b/package/utils/xfsprogs/patches/100-no_aio.patch @@ -4,8 +4,6 @@ Date: Thu Jan 26 00:35:43 2012 +0100 100-no_aio -diff --git a/configure.in b/configure.in -index d91b6ec..8dc8b4a 100644 --- a/configure.in +++ b/configure.in @@ -92,8 +92,8 @@ AC_PACKAGE_GLOBALS(xfsprogs) diff --git a/package/utils/xfsprogs/patches/110-uclibc_no_ustat.patch b/package/utils/xfsprogs/patches/110-uclibc_no_ustat.patch index 071dfc1aa8..5d5d08c8d6 100644 --- a/package/utils/xfsprogs/patches/110-uclibc_no_ustat.patch +++ b/package/utils/xfsprogs/patches/110-uclibc_no_ustat.patch @@ -4,8 +4,6 @@ Date: Thu Jan 26 00:36:42 2012 +0100 110-uclibc_no_ustat -diff --git a/libxfs/linux.c b/libxfs/linux.c -index 2e07d54..6a6c905 100644 --- a/libxfs/linux.c +++ b/libxfs/linux.c @@ -21,7 +21,6 @@ @@ -16,7 +14,7 @@ index 2e07d54..6a6c905 100644 #include #include #include -@@ -49,6 +51,7 @@ static int max_block_alignment; +@@ -49,6 +48,7 @@ static int max_block_alignment; int platform_check_ismounted(char *name, char *block, struct stat64 *s, int verbose) { @@ -24,7 +22,7 @@ index 2e07d54..6a6c905 100644 /* Pad ust; pre-2.6.28 linux copies out too much in 32bit compat mode */ struct ustat ust[2]; struct stat64 st; -@@ -68,6 +71,7 @@ platform_check_ismounted(char *name, char *block, struct stat64 *s, int verbose) +@@ -68,6 +68,7 @@ platform_check_ismounted(char *name, cha progname, name); return 1; } diff --git a/package/utils/xfsprogs/patches/120-portability.patch b/package/utils/xfsprogs/patches/120-portability.patch index 1e9b533654..1d5905a67d 100644 --- a/package/utils/xfsprogs/patches/120-portability.patch +++ b/package/utils/xfsprogs/patches/120-portability.patch @@ -4,11 +4,9 @@ Date: Thu Jan 26 00:37:52 2012 +0100 120-portability -diff --git a/copy/xfs_copy.c b/copy/xfs_copy.c -index c01e0b9..2e2f320 100644 --- a/copy/xfs_copy.c +++ b/copy/xfs_copy.c -@@ -463,6 +463,15 @@ read_ag_header(int fd, xfs_agnumber_t agno, wbuf *buf, ag_header_t *ag, +@@ -463,6 +463,15 @@ read_ag_header(int fd, xfs_agnumber_t ag } diff --git a/package/utils/xfsprogs/patches/130-uclibc_no_xattr.patch b/package/utils/xfsprogs/patches/130-uclibc_no_xattr.patch index a9b5a6fe94..c5c4a2a2ad 100644 --- a/package/utils/xfsprogs/patches/130-uclibc_no_xattr.patch +++ b/package/utils/xfsprogs/patches/130-uclibc_no_xattr.patch @@ -4,8 +4,6 @@ Date: Thu Jan 26 00:38:27 2012 +0100 130-uclibc_no_xattr -diff --git a/fsr/xfs_fsr.c b/fsr/xfs_fsr.c -index 40c2e6f..4f54059 100644 --- a/fsr/xfs_fsr.c +++ b/fsr/xfs_fsr.c @@ -35,7 +35,6 @@ @@ -16,7 +14,7 @@ index 40c2e6f..4f54059 100644 #ifndef XFS_XFLAG_NODEFRAG -@@ -990,6 +992,7 @@ fsr_setup_attr_fork( +@@ -990,6 +989,7 @@ fsr_setup_attr_fork( int tfd, xfs_bstat_t *bstatp) { @@ -24,7 +22,7 @@ index 40c2e6f..4f54059 100644 struct stat64 tstatbuf; int i; int last_forkoff = 0; -@@ -1108,6 +1111,7 @@ fsr_setup_attr_fork( +@@ -1108,6 +1108,7 @@ fsr_setup_attr_fork( out: if (dflag) fsrprintf(_("set temp attr\n")); diff --git a/package/utils/xfsprogs/patches/140-no_po.patch b/package/utils/xfsprogs/patches/140-no_po.patch index f915a9cf27..6407744536 100644 --- a/package/utils/xfsprogs/patches/140-no_po.patch +++ b/package/utils/xfsprogs/patches/140-no_po.patch @@ -1,7 +1,6 @@ -diff -urN xfsprogs-3.1.7/Makefile xfsprogs-3.1.7.new/Makefile ---- xfsprogs-3.1.7/Makefile 2011-11-18 00:30:24.000000000 +0100 -+++ xfsprogs-3.1.7.new/Makefile 2012-04-20 14:15:48.641722955 +0200 -@@ -41,7 +41,7 @@ +--- a/Makefile ++++ b/Makefile +@@ -41,7 +41,7 @@ endif LIB_SUBDIRS = libxfs libxlog libxcmd libhandle libdisk TOOL_SUBDIRS = copy db estimate fsck fsr growfs io logprint mkfs quota \ @@ -10,7 +9,7 @@ diff -urN xfsprogs-3.1.7/Makefile xfsprogs-3.1.7.new/Makefile SUBDIRS = include $(LIB_SUBDIRS) $(TOOL_SUBDIRS) -@@ -135,7 +135,6 @@ +@@ -135,7 +135,6 @@ ifeq ($(HAVE_BUILDDEFS), no) $(Q)$(MAKE) $(MAKEOPTS) -C . $@ else $(Q)$(MAKE) $(MAKEOPTS) $(SRCDIR) diff --git a/package/utils/xfsprogs/patches/160-format-security.patch b/package/utils/xfsprogs/patches/160-format-security.patch new file mode 100644 index 0000000000..5ae627d4a1 --- /dev/null +++ b/package/utils/xfsprogs/patches/160-format-security.patch @@ -0,0 +1,29 @@ +From: Ben Myers +Date: Fri, 1 Feb 2013 21:50:22 +0000 (-0600) +Subject: xfsprogs: fix warning in libxcmd/input.c +X-Git-Tag: v3.1.11~25 +X-Git-Url: http://oss.sgi.com/cgi-bin/gitweb.cgi?p=xfs%2Fcmds%2Fxfsprogs.git;a=commitdiff_plain;h=50a3aa8977821ad072f3aa5b63645827f9b8ca1d + +xfsprogs: fix warning in libxcmd/input.c + +Fix an error when building with -Werror=format-security. + +input.c: In function 'fetchline': +input.c:91:2: error: format not a string literal and no format arguments [-Werror=format-security] + +Signed-off-by: Ben Myers +Reported by: Arkadiusz Miƛkiewicz +Reviewed-by: Mark Tinguely +--- + +--- a/libxcmd/input.c ++++ b/libxcmd/input.c +@@ -88,7 +88,7 @@ fetchline(void) + + if (!line) + return NULL; +- printf(get_prompt()); ++ printf("%s", get_prompt()); + fflush(stdout); + if (!fgets(line, MAXREADLINESZ, stdin)) { + free(line);