veracrypt / VeraCrypt-DCS

VeraCrypt EFI Bootloader for EFI Windows system encryption (LGPL)
GNU Lesser General Public License v3.0
134 stars 59 forks source link

void HaltPrint(const CHAR16* Msg) should also clear the bootParams #21

Closed DavidXanatos closed 4 years ago

DavidXanatos commented 4 years ago

void HaltPrint(const CHAR16* Msg) its triggered when something goes terribly wrong with the disk hook, so its the last thing happens and it triggers CleanSensitiveData() so far so good, but what with the bootParams they contain sensitive data to and are not being cleared. so the call should also MEM_BURNthe bootParams variable.

idrassi commented 4 years ago

@DavidXanatos The bootParams variable should indeed be clear like the others. I will push a fix for it.