Code robustness question

[hasherezade]

Code robustness question

[hasherezade]

I make checks for the cases which are the most casual.

So, it is gonna be a common error to supply a bad path to the PE file - that's why I make checks around opening file and reading from file (the access may be denied for whatever reasons). I don't do checks for the the things which are less casual, or corner cases (such as attempt to load big files which size cannot be retrieved by GetFileSize).

I do check for failure in relocating the PE - because some PEs has no relocation table, and in such case they will not be loadable by this way (as we cannot guarantee at which base we will be able to load them). Relocation will fail also if we try to load a PE of different bitness than a loader. Those are common cases the user of this tool will encounter while playing around with this code.

I want this code to be as simple as possible, so apart from those common things, I don't want to litter it by too many checks. Otherwise it may be difficult for the reader to sieve out what part of code is about processing the PE, and what is about error checking. I also wrote about it in the comment on the top of the file to make the purpose of this code clear.

/*
A demo PE-loader for the training: https://github.com/hasherezade/malware_training_vol1
WARNING: This is a basic example. For the sake of simplicity, some checks have been omitted.
*/