[Bug c++/70909] Libiberty Demangler segfaults (4)

[mark at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=70909

Mark Wielaard <mark at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #40233|0                           |1
        is obsolete|                            |

--- Comment #40 from Mark Wielaard <mark at gcc dot gnu.org> ---
Created attachment 40316
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=40316&action=edit
infinite d_print_comp printing protection

(In reply to Markus Trippelsdorf from comment #39)
> Mark, could you please post your path to gcc-patches?
> Then there might be a chance to get it into binutils before 2.28 gets
> released.

I did rewrite the patch a little to remove the const arguments, add just the
testcases it fixes and allow up to 2 levels of recursion. That is attached.

But I still haven't figured out why we need to allow 2 levels of recursion for
some of the cases. See the XXX in the patch. I don't feel I can propose this
unless we figure out why the level needs to be 2 (and not 3 or...)

If we figure that out we should add the explanation to the comment. Then we can
propose it on gcc-patches.