dump_dict function dump "Version Information"'s problem

[alexlongguo]

load a pe file and print dump_dict["Version Information"]:
[ [{ 'Structure': 'VS_VERSIONINFO', 'Length': { 'FileOffset': 1145400, 'Offset': 0, 'Value': 820 }, 'ValueLength': { 'FileOffset': 1145402, 'Offset': 2, 'Value': 52 }, 'Type': { 'FileOffset': 1145404, 'Offset': 4, 'Value': 0 } }, { 'Structure': 'VS_FIXEDFILEINFO', 'Signature': { 'FileOffset': 1145440, 'Offset': 0, 'Value': 4277077181 }, 'StrucVersion': { 'FileOffset': 1145444, 'Offset': 4, 'Value': 65536 }, 'FileVersionMS': { 'FileOffset': 1145448, 'Offset': 8, 'Value': 65536 }, 'FileVersionLS': { 'FileOffset': 1145452, 'Offset': 12, 'Value': 1 }, 'ProductVersionMS': { 'FileOffset': 1145456, 'Offset': 16, 'Value': 65536 }, 'ProductVersionLS': { 'FileOffset': 1145460, 'Offset': 20, 'Value': 1 }, 'FileFlagsMask': { 'FileOffset': 1145464, 'Offset': 24, 'Value': 63 }, 'FileFlags': { 'FileOffset': 1145468, 'Offset': 28, 'Value': 0 }, 'FileOS': { 'FileOffset': 1145472, 'Offset': 32, 'Value': 4 }, 'FileType': { 'FileOffset': 1145476, 'Offset': 36, 'Value': 1 }, 'FileSubtype': { 'FileOffset': 1145480, 'Offset': 40, 'Value': 0 }, 'FileDateMS': { 'FileOffset': 1145484, 'Offset': 44, 'Value': 0 }, 'FileDateLS': { 'FileOffset': 1145488, 'Offset': 48, 'Value': 0 } }, [{ 'Structure': 'StringFileInfo', 'Length': { 'FileOffset': 1145492, 'Offset': 0, 'Value': 660 }, 'ValueLength': { 'FileOffset': 1145494, 'Offset': 2, 'Value': 0 }, 'Type': { 'FileOffset': 1145496, 'Offset': 4, 'Value': 1 } }, 'Structure', 'Length', 'ValueLength', 'Type', { 'LangID': b '080404b0', b 'Comments': b '', b 'CompanyName': b '', b 'FileDescription': b 'xxx, b 'FileVersion': b '1, 0, 0, 1', b 'InternalName': b 'xxx', b 'LegalCopyright': b '(C) 2012', b 'LegalTrademarks': b '', b 'OriginalFilename': b 'xxx.EXE', b 'PrivateBuild': b '', b 'ProductName': b 'xxx', b 'ProductVersion': b '1, 0, 0, 1', b 'SpecialBuild': b '' }, { 'Structure': 'VarFileInfo', 'Length': { 'FileOffset': 1146152, 'Offset': 0, 'Value': 68 }, 'ValueLength': { 'FileOffset': 1146154, 'Offset': 2, 'Value': 0 }, 'Type': { 'FileOffset': 1146156, 'Offset': 4, 'Value': 1 } }, 'Structure', 'Length', 'ValueLength', 'Type', { b 'Translation': '0x0804 0x04b0' }] ] ]
There are 2 wrong data:
}, 'Structure', 'Length', 'ValueLength', 'Type', {

This data is not a dict.
After modified these 2 lines of code it looks correct
1.pefile.py : fileinfo_list.extend(st_entry.dump_dict()) => fileinfo_list.append(st_entry.dump_dict())
2.pefile.py : fileinfo_list.extend(var_entry.dump_dict()) => fileinfo_list.append(var_entry.dump_dict())
[ [{ 'Structure': 'VS_VERSIONINFO', 'Length': { 'FileOffset': 1145400, 'Offset': 0, 'Value': 820 }, 'ValueLength': { 'FileOffset': 1145402, 'Offset': 2, 'Value': 52 }, 'Type': { 'FileOffset': 1145404, 'Offset': 4, 'Value': 0 } }, { 'Structure': 'VS_FIXEDFILEINFO', 'Signature': { 'FileOffset': 1145440, 'Offset': 0, 'Value': 4277077181 }, 'StrucVersion': { 'FileOffset': 1145444, 'Offset': 4, 'Value': 65536 }, 'FileVersionMS': { 'FileOffset': 1145448, 'Offset': 8, 'Value': 65536 }, 'FileVersionLS': { 'FileOffset': 1145452, 'Offset': 12, 'Value': 1 }, 'ProductVersionMS': { 'FileOffset': 1145456, 'Offset': 16, 'Value': 65536 }, 'ProductVersionLS': { 'FileOffset': 1145460, 'Offset': 20, 'Value': 1 }, 'FileFlagsMask': { 'FileOffset': 1145464, 'Offset': 24, 'Value': 63 }, 'FileFlags': { 'FileOffset': 1145468, 'Offset': 28, 'Value': 0 }, 'FileOS': { 'FileOffset': 1145472, 'Offset': 32, 'Value': 4 }, 'FileType': { 'FileOffset': 1145476, 'Offset': 36, 'Value': 1 }, 'FileSubtype': { 'FileOffset': 1145480, 'Offset': 40, 'Value': 0 }, 'FileDateMS': { 'FileOffset': 1145484, 'Offset': 44, 'Value': 0 }, 'FileDateLS': { 'FileOffset': 1145488, 'Offset': 48, 'Value': 0 } }, [{ 'Structure': 'StringFileInfo', 'Length': { 'FileOffset': 1145492, 'Offset': 0, 'Value': 660 }, 'ValueLength': { 'FileOffset': 1145494, 'Offset': 2, 'Value': 0 }, 'Type': { 'FileOffset': 1145496, 'Offset': 4, 'Value': 1 } }, { 'Structure': 'StringTable', 'Length': { 'FileOffset': 1145528, 'Offset': 0, 'Value': 624 }, 'ValueLength': { 'FileOffset': 1145530, 'Offset': 2, 'Value': 0 }, 'Type': { 'FileOffset': 1145532, 'Offset': 4, 'Value': 1 } }, { 'LangID': b '080404b0', b 'Comments': b '', b 'CompanyName': b '', b 'FileDescription': b 'xxx', b 'FileVersion': b '1, 0, 0, 1', b 'InternalName': b 'xxx', b 'LegalCopyright': b '(C) 2012', b 'LegalTrademarks': b '', b 'OriginalFilename': b 'xxx.EXE', b 'PrivateBuild': b '', b 'ProductName': b 'xxx', b 'ProductVersion': b '1, 0, 0, 1', b 'SpecialBuild': b '' }, { 'Structure': 'VarFileInfo', 'Length': { 'FileOffset': 1146152, 'Offset': 0, 'Value': 68 }, 'ValueLength': { 'FileOffset': 1146154, 'Offset': 2, 'Value': 0 }, 'Type': { 'FileOffset': 1146156, 'Offset': 4, 'Value': 1 } }, { 'Structure': 'Var', 'Length': { 'FileOffset': 1146184, 'Offset': 0, 'Value': 36 }, 'ValueLength': { 'FileOffset': 1146186, 'Offset': 2, 'Value': 4 }, 'Type': { 'FileOffset': 1146188, 'Offset': 4, 'Value': 0 } }, { b 'Translation': '0x0804 0x04b0' }] ] ]

Before modified pefile.py code:

After modified pefile.py code: