diff options
author | Mark Wielaard <mark@klomp.org> | 2017-04-21 09:01:56 +0000 |
---|---|---|
committer | Mark Wielaard <mark@gcc.gnu.org> | 2017-04-21 09:01:56 +0000 |
commit | 13b6ef76dc275232310ebfca27db08aeef9b858c (patch) | |
tree | a7877742e13eda04830c6b97ef5b5822412562dd /libiberty/cp-demangle.c | |
parent | 1240212c6b2ad61a9f8f918192fbb67d45405cc2 (diff) | |
download | gcc-13b6ef76dc275232310ebfca27db08aeef9b858c.tar.gz |
libiberty: Always return NULL if d_add_substitution fails.
d_add_substitution can fail for various reasons, like when the subs array
is full. If d_add_substitution fails d_substitution should return NULL
early and not try to continue. Every other call of d_add_substitution
is handled in the same way.
libiberty/ChangeLog:
* cp-demangle.c (d_substitution): Return NULL if d_add_substitution
fails.
From-SVN: r247055
Diffstat (limited to 'libiberty/cp-demangle.c')
-rw-r--r-- | libiberty/cp-demangle.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libiberty/cp-demangle.c b/libiberty/cp-demangle.c index 2c7d5c5faab..aeff7a79d74 100644 --- a/libiberty/cp-demangle.c +++ b/libiberty/cp-demangle.c @@ -3891,7 +3891,8 @@ d_substitution (struct d_info *di, int prefix) /* If there are ABI tags on the abbreviation, it becomes a substitution candidate. */ dc = d_abi_tags (di, dc); - d_add_substitution (di, dc); + if (! d_add_substitution (di, dc)) + return NULL; } return dc; } |