https://sourceware.org/bugzilla/show_bug.cgi?id=32260

--- Comment #4 from Jonny Weir <jonny.weir at clearpool dot io> ---
(In reply to Alan Modra from comment #3)
> Looks to me that merge.c:177 should be unsigned int newnb ...
> By calculating newnb as unsigned long we get to a count of 2^32 and overflow
> table->nbuckets without triggering an error in sec_merge_maybe_resize

Interesting, I have made this change the output now looks like this:

XXX bfdtab->count=1749 table->nbuckets=524288
XXX bfdtab->count=1750 table->nbuckets=524288
XXX bfdtab->count=1751 table->nbuckets=524288
XXX sec->size=2872400096
XXX added=1436200049
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(20)
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(33)
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(41)
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(48)
...
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(391)
/bin/ld: /tmp/ccrnRHZp.ltrans0.ltrans.o: access beyond end of merged section
(391)
/bin/ld: final link failed
collect2: error: ld returned 1 exit status

No more assertion failures, but the access errors remain.

The next set estimations are still very large.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

Reply via email to