Fixed overflow potential on larger ELF files

konrad-kruczynski / elfsharp

Pure managed C# library for reading ELF, UImage, Mach-O binaries.

https://elfsharp.it

Other

151 stars 56 forks source link

Closed FoxCouncil closed 6 years ago

FoxCouncil commented 6 years ago

Had a 400MB ELF file cause an overflow in this part of the file load process.

Hope this helps, lots of love!

.fox

konrad-kruczynski commented 6 years ago

Thanks for the patch! I'll look at it in the evening.

konrad-kruczynski commented 6 years ago

The patch is generally ok. I'll merge it, but first I will add some things:

konrad-kruczynski commented 6 years ago

Once again thanks for the work.