diff --git a/binutils/ChangeLog b/binutils/ChangeLog index 35d3fada640..7ce0326ac49 100644 --- a/binutils/ChangeLog +++ b/binutils/ChangeLog @@ -1,3 +1,9 @@ +2020-03-05 Alan Modra + + PR 25629 + * objcopy.c (filter_symbols): Don't segfault on NULL + prefix_symbols_string. + 2020-03-04 Christian Eggers * objcopy.c (copy_object): Convert from bytes to octets for diff --git a/binutils/objcopy.c b/binutils/objcopy.c index daee5707637..3b9f1b73ca9 100644 --- a/binutils/objcopy.c +++ b/binutils/objcopy.c @@ -1580,9 +1580,14 @@ filter_symbols (bfd *abfd, bfd *obfd, asymbol **osyms, if (add_leading_char || prefix_symbols_string) { char *n, *ptr; + size_t len = strlen (name) + 1; - ptr = n = (char *) xmalloc (1 + strlen (prefix_symbols_string) - + strlen (name) + 1); + if (add_leading_char) + len++; + if (prefix_symbols_string) + len += strlen (prefix_symbols_string); + + ptr = n = (char *) xmalloc (len); if (add_leading_char) *ptr++ = bfd_get_symbol_leading_char (obfd);