Fix parsing large values for ChainPackage/@InstallSize.

Fixes #4388

3 files changed, 7 insertions, 3 deletions

diff --git a/src/wix/WixToolset.Core/Compiler_Bundle.cs b/src/wix/WixToolset.Core/Compiler_Bundle.cs
index a72d434d..03f0ca0e 100644
--- a/src/wix/WixToolset.Core/Compiler_Bundle.cs
+++ b/src/wix/WixToolset.Core/Compiler_Bundle.cs
@@ -1962,7 +1962,7 @@ namespace WixToolset.Core
             var perMachine = YesNoDefaultType.NotSet;
             string detectCondition = null;
             string protocol = null;
-            var installSize = CompilerConstants.IntegerNotSet;
+            long? installSize = null;
             string msuKB = null;
             var enableFeatureSelection = YesNoType.NotSet;
             var forcePerMachine = YesNoType.NotSet;
@@ -2082,7 +2082,7 @@ namespace WixToolset.Core
                         allowed = (packageType == WixBundlePackageType.Exe);
                         break;
                     case "InstallSize":
-                        installSize = this.Core.GetAttributeIntegerValue(sourceLineNumbers, attrib, 0, Int32.MaxValue);
+                        installSize = this.Core.GetAttributeLongValue(sourceLineNumbers, attrib, 0, Int64.MaxValue);
                         break;
                     case "KB":
                         msuKB = this.Core.GetAttributeValue(sourceLineNumbers, attrib);
@@ -2314,7 +2314,7 @@ namespace WixToolset.Core
                     chainPackageSymbol.PerMachine = perMachine;
                 }
 
-                if (CompilerConstants.IntegerNotSet != installSize)
+                if (installSize.HasValue)
                 {
                     chainPackageSymbol.InstallSize = installSize;
                 }
diff --git a/src/wix/test/WixToolsetTest.CoreIntegration/BundleFixture.cs b/src/wix/test/WixToolsetTest.CoreIntegration/BundleFixture.cs
index c9ae049e..0bea3af9 100644
--- a/src/wix/test/WixToolsetTest.CoreIntegration/BundleFixture.cs
+++ b/src/wix/test/WixToolsetTest.CoreIntegration/BundleFixture.cs
@@ -274,6 +274,9 @@ namespace WixToolsetTest.CoreIntegration
                     var intermediate = Intermediate.Load(wixOutput);
                     var section = intermediate.Sections.Single();
 
+                    var packageSymbol = section.Symbols.OfType<WixBundlePackageSymbol>().Where(x => x.Id.Id == "NetFx462Web").Single();
+                    Assert.Equal(Int64.MaxValue, packageSymbol.InstallSize);
+
                     var payloadSymbol = section.Symbols.OfType<WixBundlePayloadSymbol>().Where(x => x.Id.Id == "NetFx462Web").Single();
                     Assert.Equal(Int64.MaxValue, payloadSymbol.FileSize);
                 }
diff --git a/src/wix/test/WixToolsetTest.CoreIntegration/TestData/SingleExeBundle/SingleExeRemotePayload.wxs b/src/wix/test/WixToolsetTest.CoreIntegration/TestData/SingleExeBundle/SingleExeRemotePayload.wxs
index 0d459f02..29dc78e7 100644
--- a/src/wix/test/WixToolsetTest.CoreIntegration/TestData/SingleExeBundle/SingleExeRemotePayload.wxs
+++ b/src/wix/test/WixToolsetTest.CoreIntegration/TestData/SingleExeBundle/SingleExeRemotePayload.wxs
@@ -8,6 +8,7 @@
                 PerMachine="yes"
                 DetectCondition="A"
                 InstallCondition="B"
+                InstallSize="9223372036854775807"
                 Id="NetFx462Web"
                 Vital="yes"
                 Permanent="yes"