Fixed lgtm issues

[siredmar]

Fixes #51 Also fixed crashes on binary file read

[lgtm-com[bot]]

This pull request introduces 1 alert and fixes 29 when merging cc3104180e476aa0f89d3b233dc9b7d9f176662a into 8737c15db291b38d48ff1aecfc0f0c436243346e - view on LGTM.com

new alerts:

• 1 for Mismatching new/free or malloc/delete

fixed alerts:

• 20 for Empty branch of conditional
• 5 for Multiplication result converted to larger type
• 2 for Comparison result is always the same
• 1 for Resource not released in destructor
• 1 for Non-virtual destructor in base class