TimeZoneInfo: Avoid cloning privately-created ArgumentRule[] arrays

src/mscorlib/src/System/TimeZoneInfo.cs

@@ -1266,16 +1266,13 @@ private TimeZoneInfo(
  Boolean adjustmentRulesSupportDst;
  ValidateTimeZoneInfo(id, baseUtcOffset, adjustmentRules, out adjustmentRulesSupportDst);
 
- if (!disableDaylightSavingTime && adjustmentRules != null && adjustmentRules.Length > 0) {
- m_adjustmentRules = (AdjustmentRule[])adjustmentRules.Clone();
- }
-
  m_id = id;
  m_baseUtcOffset = baseUtcOffset;
  m_displayName = displayName;
  m_standardDisplayName = standardDisplayName;
  m_daylightDisplayName = (disableDaylightSavingTime ? null : daylightDisplayName);
  m_supportsDaylightSavingTime = adjustmentRulesSupportDst && !disableDaylightSavingTime;
+ m_adjustmentRules = adjustmentRules;
  }
 
  // -------- SECTION: factory methods -----------------*
@@ -1316,7 +1313,7 @@ static public TimeZoneInfo CreateCustomTimeZone(
  String daylightDisplayName,
  AdjustmentRule [] adjustmentRules) {
 
- return new TimeZoneInfo(
+ return CreateCustomTimeZone(
  id,
  baseUtcOffset,
  displayName,
@@ -1345,7 +1342,11 @@ static public TimeZoneInfo CreateCustomTimeZone(
  AdjustmentRule [] adjustmentRules,
  Boolean disableDaylightSavingTime) {
 
- return new TimeZoneInfo(
+ if (!disableDaylightSavingTime && adjustmentRules?.Length > 0) {
+ adjustmentRules = (AdjustmentRule[])adjustmentRules.Clone();
+ }
+
+ return new TimeZoneInfo(
  id,
  baseUtcOffset,
  displayName,
@@ -5086,7 +5087,7 @@ static public TimeZoneInfo GetDeserializedTimeZoneInfo(String source) {
  AdjustmentRule[] rules = s.GetNextAdjustmentRuleArrayValue(false);
 
  try { 
- return TimeZoneInfo.CreateCustomTimeZone(id, baseUtcOffset, displayName, standardName, daylightName, rules);
+ return new TimeZoneInfo(id, baseUtcOffset, displayName, standardName, daylightName, rules, disableDaylightSavingTime: false);
  }
  catch (ArgumentException ex) {
  throw new SerializationException(Environment.GetResourceString("Serialization_InvalidData"), ex);