Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 295205 - app-editors/vim runtime ABRT with -D_FORTIFY_SOURCE=2
Summary: app-editors/vim runtime ABRT with -D_FORTIFY_SOURCE=2
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: High major (vote)
Assignee: Vim Maintainers
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-11-30 16:10 UTC by Jeremy Olexa (darkside) (RETIRED)
Modified: 2009-11-30 18:18 UTC (History)
0 users

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-11-30 16:10:24 UTC
http://www.mail-archive.com/vim_dev@googlegroups.com/msg04770.html
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=38136

With -O2:
"foo.txt" [New File]*** buffer overflow detected ***: vim terminated
Vim: Caught deadly signal ABRT

Vim: Finished.
Aborted

With -O2 -D_FORTIFY_SOURCE=1, it does not ABRT.

See the above links for upstream discussion. A quick fix would be to append -D_FORTIFY_SOURCE=1. I think.
Comment 1 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-11-30 16:17:34 UTC
The question is, why doesn't ftp://ftp.vim.org/pub/vim/patches/7.2/7.2.251 get applied?

This is in Gentoo Prefix, btw. Maybe we screwed something up... =/
Comment 2 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2009-11-30 16:26:10 UTC
Dang, our fault. While overriding src_prepare() in vim's ebuild, we never called vim_src_prepare() Doh!

Sorry for the noise!!
Comment 3 Jim Ramsay (lack) (RETIRED) gentoo-dev 2009-11-30 18:18:03 UTC
Heh, as soon as I saw your name, I thought to myself: is there a 'resolved: prefix_is_crazy'?

;)