To: vim-dev@vim.org Subject: Patch 7.2.251 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 7.2.251 (after 7.2.044) Problem: Compiler adds invalid memory bounds check. Solution: Remove _FORTIFY_SOURCE=2 from CFLAGS. (Dominique Pelle) Files: src/auto/configure, src/configure.in *** ../vim-7.2.250/src/auto/configure 2009-07-22 11:16:54.000000000 +0200 --- src/auto/configure 2009-08-07 14:49:52.000000000 +0200 *************** *** 17135,17141 **** { $as_echo "$as_me:$LINENO: checking whether we need -D_FORTIFY_SOURCE=1" >&5 $as_echo_n "checking whether we need -D_FORTIFY_SOURCE=1... " >&6; } if test "$gccmajor" -gt "3"; then ! CFLAGS="$CFLAGS -D_FORTIFY_SOURCE=1" { $as_echo "$as_me:$LINENO: result: yes" >&5 $as_echo "yes" >&6; } else --- 17135,17141 ---- { $as_echo "$as_me:$LINENO: checking whether we need -D_FORTIFY_SOURCE=1" >&5 $as_echo_n "checking whether we need -D_FORTIFY_SOURCE=1... " >&6; } if test "$gccmajor" -gt "3"; then ! CFLAGS=`echo "$CFLAGS -D_FORTIFY_SOURCE=1" | sed -e 's/-Wp,-D_FORTIFY_SOURCE=2//g' -e 's/-D_FORTIFY_SOURCE=2//g'` { $as_echo "$as_me:$LINENO: result: yes" >&5 $as_echo "yes" >&6; } else *** ../vim-7.2.250/src/configure.in 2009-07-22 11:16:54.000000000 +0200 --- src/configure.in 2009-08-07 14:49:47.000000000 +0200 *************** *** 3233,3239 **** dnl declared as char x[1] but actually longer. Introduced in gcc 4.0. AC_MSG_CHECKING(whether we need -D_FORTIFY_SOURCE=1) if test "$gccmajor" -gt "3"; then ! CFLAGS="$CFLAGS -D_FORTIFY_SOURCE=1" AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) --- 3233,3239 ---- dnl declared as char x[1] but actually longer. Introduced in gcc 4.0. AC_MSG_CHECKING(whether we need -D_FORTIFY_SOURCE=1) if test "$gccmajor" -gt "3"; then ! CFLAGS=`echo "$CFLAGS -D_FORTIFY_SOURCE=1" | sed -e 's/-Wp,-D_FORTIFY_SOURCE=2//g' -e 's/-D_FORTIFY_SOURCE=2//g'` AC_MSG_RESULT(yes) else AC_MSG_RESULT(no) *** ../vim-7.2.250/src/version.c 2009-09-11 13:26:38.000000000 +0200 --- src/version.c 2009-09-11 13:43:46.000000000 +0200 *************** *** 678,679 **** --- 678,681 ---- { /* Add new patch number below this line */ + /**/ + 251, /**/ -- hundred-and-one symptoms of being an internet addict: 225. You sign up for free subscriptions for all the computer magazines /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ download, build and distribute -- http://www.A-A-P.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org ///