From: Timo Kokkonen Date: Wed, 8 Oct 2003 12:29:27 +0000 (+0300) Subject: re PR bootstrap/12490 (buffer overflow in scan-decls.c) X-Git-Tag: releases/gcc-3.4.0~3202 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=10dbf393161d85616f515ceb176e550c8686bf42;p=thirdparty%2Fgcc.git re PR bootstrap/12490 (buffer overflow in scan-decls.c) PR bootstrap/12490 * scan-decls.c (MAX_EXTERN_C_BRACES): New preprocessor constant to define the size of the extern_C_braces array. Set it to 200. (scan_decls): Abort when extern_C_braces_length is out-of-bounds. Co-Authored-By: Eric Botcazou From-SVN: r72224 --- diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5b2e01a6f676..72fd7ae770bd 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,11 @@ +2003-10-08 Timo Kokkonen + Eric Botcazou + + PR bootstrap/12490 + * scan-decls.c (MAX_EXTERN_C_BRACES): New preprocessor constant + to define the size of the extern_C_braces array. Set it to 200. + (scan_decls): Abort when extern_C_braces_length is out-of-bounds. + 2003-10-08 Carlo Wood * Makefile.in (gengtype-lex.c): flex 2.5.4[a] doesn't understand diff --git a/gcc/scan-decls.c b/gcc/scan-decls.c index 14f64e8cb2e1..ebd69cb05c11 100644 --- a/gcc/scan-decls.c +++ b/gcc/scan-decls.c @@ -34,7 +34,9 @@ int brace_nesting = 0; indicate the (brace nesting levels of) left braces that were prefixed by extern "C". */ int extern_C_braces_length = 0; -char extern_C_braces[20]; +/* 20 is not enough anymore on Solaris 9. */ +#define MAX_EXTERN_C_BRACES 200 +char extern_C_braces[MAX_EXTERN_C_BRACES]; #define in_extern_C_brace (extern_C_braces_length>0) /* True if the function declaration currently being scanned is @@ -220,6 +222,12 @@ scan_decls (cpp_reader *pfile, int argc ATTRIBUTE_UNUSED, brace_nesting++; extern_C_braces[extern_C_braces_length++] = brace_nesting; + if (extern_C_braces_length >= MAX_EXTERN_C_BRACES) + { + fprintf (stderr, + "Internal error: out-of-bounds index\n"); + exit (FATAL_EXIT_CODE); + } goto new_statement; } }