Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

meta_pe: fix rich header length check for hash calculation #50

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Commits on Sep 23, 2016

  1. meta_pe: fix rich header length check for hash calculation

    The original Rich Signature write-up:
      http://www.ntcore.com/files/richsign.htm
    searches 400 bytes for the "Rich" string.  The pefile module
    searches 128 bytes for the string.  I have found that 128 is
    sometimes not enough, and 400 feels rather large, so I have
    choosen a round (hex) value in between.
    
    I also take a lesson from the original write-up and search for
    NULL values, but added a search for the PE header as well.
    
    Note that because we rely on pefile module for the Rich Header
    Values, that array will be incomplete, and thus differ from that
    used for hash calculation, until pefile itself is fixed.
    knowmalware committed Sep 23, 2016
    Configuration menu
    Copy the full SHA
    759a626 View commit details
    Browse the repository at this point in the history

Commits on Sep 2, 2017

  1. Simplify rich header hash calculation with latest pefile.

    The latest version of pefile provides easy access to the
    deobfuscated rich header by accessing the 'clear_data' key
    of the parsed rich header.
    knowmalware committed Sep 2, 2017
    Configuration menu
    Copy the full SHA
    d99c819 View commit details
    Browse the repository at this point in the history