On 1/10/19 5:15 PM, Ben L wrote: > Hi all, > > First time emailing gcc-patches, so I'm sorry if I get any of this wrong or if > there's obvious errors repeated in my patches. AFAICT I should be sending each > change individually rather than as one bulk patch, so I'm sorry about the spam > too. > > All of these changes were found by fuzzing libiberty's demanglers over the > past week, and I have at least one more that it's currently crashing out on > but I haven't had time to look into why yet. > > Obviously since this is my first time emailing I don't have write access to > commit any of these, so if any are approved then I'd be grateful if you can > commit them too. > > Thanks, > Ben > > -- > > 'typed_name' is checked before the loop, but not checked after every > iteration. This can cause a crash if the input buffer is malformed since > 'typed_name' can be assigned NULL. > > To fix this, break out of the loop if we see it's NULL and handle that case > afterwards. > > * cp-demangle (d_print_comp_inner): Guard against a NULL 'typed_name'. > * testsuite/demangle-expected: Add testcase. > THanks. I've installed this on the trunk.
jeff