diff options
author | Bob Arnson <bob@firegiant.com> | 2020-09-01 18:31:26 -0400 |
---|---|---|
committer | Bob Arnson <bob@firegiant.com> | 2020-09-01 18:34:22 -0400 |
commit | e1ff7245ee176c18bd0b773a7e11df6bb95d7b7e (patch) | |
tree | 017787984ffabf5076765c27775fb75127b7dcb3 | |
parent | 8634b7f60adad8d09834e77aa0e4a0f26451f845 (diff) | |
download | wix-e1ff7245ee176c18bd0b773a7e11df6bb95d7b7e.tar.gz wix-e1ff7245ee176c18bd0b773a7e11df6bb95d7b7e.tar.bz2 wix-e1ff7245ee176c18bd0b773a7e11df6bb95d7b7e.zip |
Fix check for valid 8.3 names for >3 extensions.
-rw-r--r-- | src/WixToolset.Core/Common.cs | 2 | ||||
-rw-r--r-- | src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs | 4 | ||||
-rw-r--r-- | src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs | 4 |
3 files changed, 9 insertions, 1 deletions
diff --git a/src/WixToolset.Core/Common.cs b/src/WixToolset.Core/Common.cs index 6efc7571..1bb895be 100644 --- a/src/WixToolset.Core/Common.cs +++ b/src/WixToolset.Core/Common.cs | |||
@@ -253,7 +253,7 @@ namespace WixToolset.Core | |||
253 | { | 253 | { |
254 | return filename.Length < 9; | 254 | return filename.Length < 9; |
255 | } | 255 | } |
256 | else if (expectedDot > 8 || filename[expectedDot] != '.') | 256 | else if (expectedDot > 8 || filename[expectedDot] != '.' || expectedDot + 4 < filename.Length) |
257 | { | 257 | { |
258 | return false; | 258 | return false; |
259 | } | 259 | } |
diff --git a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs index 158687cf..f5353c87 100644 --- a/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs +++ b/src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs | |||
@@ -371,6 +371,10 @@ namespace WixToolsetTest.CoreIntegration | |||
371 | WixAssert.CompareLineByLine(new[] | 371 | WixAssert.CompareLineByLine(new[] |
372 | { | 372 | { |
373 | "Directory:DUPLICATENAMEANDSHORTNAME\tINSTALLFOLDER\tduplicat", | 373 | "Directory:DUPLICATENAMEANDSHORTNAME\tINSTALLFOLDER\tduplicat", |
374 | "Directory:Folder1\tINSTALLFOLDER\tFolder.1", | ||
375 | "Directory:Folder12\tINSTALLFOLDER\tFolder.12", | ||
376 | "Directory:Folder123\tINSTALLFOLDER\tFolder.123", | ||
377 | "Directory:Folder1234\tINSTALLFOLDER\tyakwclwy|Folder.1234", | ||
374 | "Directory:INSTALLFOLDER\tProgramFiles6432Folder\t1egc1laj|MsiPackage", | 378 | "Directory:INSTALLFOLDER\tProgramFiles6432Folder\t1egc1laj|MsiPackage", |
375 | "Directory:NAMEANDSHORTNAME\tINSTALLFOLDER\tSHORTNAM|NameAndShortName", | 379 | "Directory:NAMEANDSHORTNAME\tINSTALLFOLDER\tSHORTNAM|NameAndShortName", |
376 | "Directory:NAMEANDSHORTSOURCENAME\tINSTALLFOLDER\tNAMEASSN|NameAndShortSourceName", | 380 | "Directory:NAMEANDSHORTSOURCENAME\tINSTALLFOLDER\tNAMEASSN|NameAndShortSourceName", |
diff --git a/src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs b/src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs index a217fa34..2f277956 100644 --- a/src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs +++ b/src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs | |||
@@ -17,6 +17,10 @@ | |||
17 | <Directory Id="NAMEANDSHORTSOURCENAME" Name="NameAndShortSourceName" ShortName="NAMEASSN"></Directory> | 17 | <Directory Id="NAMEANDSHORTSOURCENAME" Name="NameAndShortSourceName" ShortName="NAMEASSN"></Directory> |
18 | <Directory Id="SHORTNAMEANDLONGSOURCENAME" SourceName="ShortNameAndLongSourceName" Name="SHNALSNM"></Directory> | 18 | <Directory Id="SHORTNAMEANDLONGSOURCENAME" SourceName="ShortNameAndLongSourceName" Name="SHNALSNM"></Directory> |
19 | <Directory Id="SOURCENAMEWITHSHORTVALUE" SourceName="SRTSRCVL"></Directory> | 19 | <Directory Id="SOURCENAMEWITHSHORTVALUE" SourceName="SRTSRCVL"></Directory> |
20 | <Directory Id="Folder1" Name="Folder.1"></Directory> | ||
21 | <Directory Id="Folder12" Name="Folder.12"></Directory> | ||
22 | <Directory Id="Folder123" Name="Folder.123"></Directory> | ||
23 | <Directory Id="Folder1234" Name="Folder.1234"></Directory> | ||
20 | </DirectoryRef> | 24 | </DirectoryRef> |
21 | </Fragment> | 25 | </Fragment> |
22 | </Wix> | 26 | </Wix> |