On 1/10/19 5:14 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 > > -- > > Passing "_ZZaSFvOEES_" to cplus_demangle() without the DMGL_PARAMS flag causes > a crash due to d_right (dc) returning NULL inside d_encoding(). > > Check for this case and handle it as an error rather than crashing when trying > to dereference the right side's type. > > * cp-demangle.c (d_encoding): Guard against NULL return values from > d_right (dc). > * testsuite/demangle-expected: Add testcase. > THanks. I've installed this on the trunk.
Jeff
