diff options
author | Jakub Jelinek <jakub@redhat.com> | 2012-05-29 13:34:38 +0200 |
---|---|---|
committer | Jakub Jelinek <jakub@gcc.gnu.org> | 2012-05-29 13:34:38 +0200 |
commit | 92b05e72eab38351ab9a486e0e0ec9db11f4413b (patch) | |
tree | c2b60359132356dcde0e153b19936cc26691c1d9 | |
parent | 4f7f7aca357246cb38a1a4f9e1177f5adfd3f25e (diff) | |
download | gcc-92b05e72eab38351ab9a486e0e0ec9db11f4413b.zip gcc-92b05e72eab38351ab9a486e0e0ec9db11f4413b.tar.gz gcc-92b05e72eab38351ab9a486e0e0ec9db11f4413b.tar.bz2 |
re PR middle-end/53510 (OOM while compile some code)
PR middle-end/53510
* input.c (read_line): Use XRESIZEVEC instead of XNEWVEC
to avoid leaking memory. No need to handle memory allocation
failure. Double string_len on each reallocation instead of
adding 2.
* gcov.c (read_line): Likewise.
From-SVN: r187952
-rw-r--r-- | gcc/ChangeLog | 9 | ||||
-rw-r--r-- | gcc/gcov.c | 11 | ||||
-rw-r--r-- | gcc/input.c | 13 |
3 files changed, 14 insertions, 19 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 513405f..19048db 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,12 @@ +2012-05-29 Jakub Jelinek <jakub@redhat.com> + + PR middle-end/53510 + * input.c (read_line): Use XRESIZEVEC instead of XNEWVEC + to avoid leaking memory. No need to handle memory allocation + failure. Double string_len on each reallocation instead of + adding 2. + * gcov.c (read_line): Likewise. + 2012-05-29 Hans-Peter Nilsson <hp@axis.com> * config/cris/cris.h (TARGET_HAS_BREAK, TARGET_TRAP_USING_BREAK8): @@ -2219,15 +2219,8 @@ read_line (FILE *file) return string; } pos += len; - ptr = XNEWVEC (char, string_len * 2); - if (ptr) - { - memcpy (ptr, string, pos); - string = ptr; - string_len += 2; - } - else - pos = 0; + string = XRESIZEVEC (char, string, string_len * 2); + string_len *= 2; } return pos ? string : NULL; diff --git a/gcc/input.c b/gcc/input.c index 5f14489..ece7282 100644 --- a/gcc/input.c +++ b/gcc/input.c @@ -1,5 +1,5 @@ /* Data and functions related to line maps and input files. - Copyright (C) 2004, 2007, 2008, 2009, 2010, 2011 + Copyright (C) 2004, 2007, 2008, 2009, 2010, 2011, 2012 Free Software Foundation, Inc. This file is part of GCC. @@ -105,15 +105,8 @@ read_line (FILE *file) return string; } pos += len; - ptr = XNEWVEC (char, string_len * 2); - if (ptr) - { - memcpy (ptr, string, pos); - string = ptr; - string_len += 2; - } - else - pos = 0; + string = XRESIZEVEC (char, string, string_len * 2); + string_len *= 2; } return pos ? string : NULL; |