aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBob Arnson <bob@firegiant.com>2020-09-01 18:31:26 -0400
committerBob Arnson <bob@firegiant.com>2020-09-01 18:34:22 -0400
commite1ff7245ee176c18bd0b773a7e11df6bb95d7b7e (patch)
tree017787984ffabf5076765c27775fb75127b7dcb3
parent8634b7f60adad8d09834e77aa0e4a0f26451f845 (diff)
downloadwix-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.cs2
-rw-r--r--src/test/WixToolsetTest.CoreIntegration/MsiQueryFixture.cs4
-rw-r--r--src/test/WixToolsetTest.CoreIntegration/TestData/DefaultDir/DefaultDir.wxs4
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>