Date: Tue, 26 Nov 2013 17:11:43 +0000 (UTC) From: Matthew D Fleming <mdf@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r258658 - head/contrib/gcclibs/libcpp Message-ID: <201311261711.rAQHBh92058754@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: mdf Date: Tue Nov 26 17:11:43 2013 New Revision: 258658 URL: http://svnweb.freebsd.org/changeset/base/258658 Log: Fix a segfault / internal compiler error. Among other causes, when gcc throws a warning before parsing any tokens, the cur_token pointer is at the beginning of malloc'd memory. Dereferencing cur_token[-1] can cause a segfault. Code taken from OpenBSD http://www.openbsd.org/cgi-bin/cvsweb/src/gnu/gcc/libcpp/errors.c which was a more complete fix than the one I originally coded. MFC after: 1 week Modified: head/contrib/gcclibs/libcpp/errors.c Modified: head/contrib/gcclibs/libcpp/errors.c ============================================================================== --- head/contrib/gcclibs/libcpp/errors.c Tue Nov 26 16:13:48 2013 (r258657) +++ head/contrib/gcclibs/libcpp/errors.c Tue Nov 26 17:11:43 2013 (r258658) @@ -153,7 +153,20 @@ cpp_error (cpp_reader * pfile, int level } else { - src_loc = pfile->cur_token[-1].src_loc; + /* Find actual previous token. */ + cpp_token *t; + + if (pfile->cur_token != pfile->cur_run->base) + t = pfile->cur_token - 1; + else + { + if (pfile->cur_run->prev != NULL) + t = pfile->cur_run->prev->limit; + else + t = NULL; + } + /* Retrieve corresponding source location, unless we failed. */ + src_loc = t ? t->src_loc : 0; } if (_cpp_begin_message (pfile, level, src_loc, 0))
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201311261711.rAQHBh92058754>