search

binarly-io / efiXplorer

IDA plugin for UEFI firmware analysis and reverse engineering automation
GNU General Public License v3.0
908 stars 106 forks source link

Improperly closing the file lead double-free crash. #52

Closed cc-crack closed 2 years ago

cc-crack commented 2 years ago

Hi, When I tested in the ida 7.7 linux environment, I found that the efixloader would crash in some cases. I checked the code and found that load_file closes the li pointer passed in externally and reuses it, but ida closes this pointer again after load_file completes. I've submitted a PR to fix this.

yeggor commented 2 years ago

Hello, thank you. I've merged your PR.