fix: use Valuelength for parseString instead of whole String size (#96)

* fix: use Valuelength for parseString instead of whole string length

If a resource has a empty string, parsing value of string is parsed incorrectly(getting a next String part as string).
As String.ValueLength is length in utf-16 string without null value, use 2*(s.ValueLength+1).

* chore: add sample file for stringversioninfo fix

* chore: add sample file to version_test

Author: Snshadow

test/YourPhone.Exp.WinRT.dll

@@ -280,7 +280,7 @@ func (pe *File) parseString(rva uint32, e ResourceDirectoryEntry) (string, strin
 		return "", "", 0, err
 	}
 	valueOffset := alignDword(uint32(2*(len(key)+1))+offset+StringLength, e.Data.Struct.OffsetToData)
-	b, err = pe.ReadBytesAtOffset(valueOffset, uint32(s.Length))
+	b, err = pe.ReadBytesAtOffset(valueOffset, uint32(2*(s.ValueLength+1)))
 	if err != nil {
 		return "", "", 0, err
 	}

version.go

@@ -43,6 +43,11 @@ var peVersionResourceTests = []struct {
 		nil,
 		map[string]string{"Assembly Version": "7.3.4.500", "Comments": "PowerShell on Windows top-level project", "CompanyName": "Microsoft Corporation", "FileDescription": "pwsh", "FileVersion": "7.3.4.500", "InternalName": "pwsh.dll", "LegalCopyright": "(c) Microsoft Corporation.", "OriginalFilename": "pwsh.dll", "ProductName": "PowerShell", "ProductVersion": "7.3.4 SHA: b59f05d5a1b2fceca231f75c53c203a02edf6203"},
 	},
+	{
+		getAbsoluteFilePath("test/YourPhone.Exp.WinRT.dll"),
+		nil,
+		map[string]string{"CompanyName": "Microsoft Corporation", "FileDescription": "", "FileVersion": "1.24052.124.0", "OriginalFilename": "YourPhone.Exp.WinRT.dll", "LegalCopyright": "© Microsoft Corporation.  All rights reserved.", "InternalName": "YourPhone.Exp.WinRT", "ProductName": "Microsoft Phone Link", "ProductVersion": "1.24052.124.0"},
+	},
 }
 
 func TestParseVersionResources(t *testing.T) {