wisk / medusa

An open source interactive disassembler
Other
1.04k stars 92 forks source link

I want add support windows pdb files #4

Closed ivan-kulikov-dev closed 8 years ago

ivan-kulikov-dev commented 10 years ago

Hi.I have question about architecture.I want add support windows pdb files. My algoritm is: 1)parse PE_DIRECTORY_ENTRY_DEBUG 2)try download pdb file from microsoft site 3)valid pdb file 4)if file not system,then try load user pdb file.(filename.pdb) 5)parse pdb file and add comments

questions: 1)Pe loader parse PE_DIRECTORY_ENTRY_DEBUG.Can be add IsHasDebugSymbols to Loader?Download symbols with TaskManager?(AddTask)

wisk commented 10 years ago

Hello,

That's a really good idea! I don't think a new method for the class Loader is needed, the method Loader::Map should be enough to parse the PDB and fills the Document. However, as you noted, the user can define _NT_SYMBOL_PATH to specify a remote server which could provide the PDB. This behavior requires a specific task and API to download file.

Tell me what you need, I'll try to implement it.

BTW, are you going to implement it from scratch or relying on an external library?

ivan-kulikov-dev commented 10 years ago

No, i'll rewrite this https://code.google.com/p/pdbparser/ .

ivan-kulikov-dev commented 10 years ago

https://github.com/gunmetal313/medusa/tree/addpdb my branch

wisk commented 8 years ago

Hi gunmetal313,

Because I didn't have news regarding this feature for a long time, I close this issue. Feel free to re-open it if you have made some progresses.

Thanks.