n0fate / chainbreaker

Mac OS X Keychain Forensic Tool
GNU General Public License v2.0
833 stars 160 forks source link

Fix some bytes/str conversion issue #36

Open hexchain opened 1 year ago

hexchain commented 1 year ago

The output of the keychain password hash has been broken because using a bytes object for %s produces repr-type output. This PR fixes that.

Also, export file names were all numbers because bytes[n] is an int and str(bytes[n]) is just the integer converted to a string. Since we expect PrintNames to always be a bytes object, we should be able to use decode and ignore errors.