To: vim-dev@vim.org

Subject: Patch 7.2.251

Fcc: outbox

From: Bram Moolenaar <Bram@moolenaar.net>

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    ///