Fix #155

mganss · Jan 17, 2019 · cabae35 · cabae35
1 parent f4eeace
commit cabae35
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/HtmlSanitizer/HtmlSanitizer.cs b/src/HtmlSanitizer/HtmlSanitizer.cs
@@ -538,6 +538,7 @@ private void DoSanitize(IHtmlDocument dom, IElement context, string baseUrl = ""
  }
 
  // sanitize the style attribute
+ var oldStyleEmpty = string.IsNullOrEmpty(tag.GetAttribute("style"));
  SanitizeStyle(tag, baseUrl);
 
  var checkClasses = AllowedCssClasses != null;
@@ -564,9 +565,9 @@ private void DoSanitize(IHtmlDocument dom, IElement context, string baseUrl = ""
  if (!tag.ClassList.Any())
  RemoveAttribute(tag, attribute, RemoveReason.ClassAttributeEmpty);
  }
- else if (string.IsNullOrEmpty(attribute.Value))
+ else if (!oldStyleEmpty && attribute.Name == "style" && string.IsNullOrEmpty(attribute.Value))
  {
- tag.RemoveAttribute(attribute.Name);
+ RemoveAttribute(tag, attribute, RemoveReason.StyleAttributeEmpty);
  }
  }
  }

diff --git a/src/HtmlSanitizer/RemoveReason.cs b/src/HtmlSanitizer/RemoveReason.cs
@@ -32,6 +32,10 @@ public enum RemoveReason
  /// <summary>
  /// The class attribute is empty
  /// </summary>
- ClassAttributeEmpty
+ ClassAttributeEmpty,
+ /// <summary>
+ /// The style attribute is empty
+ /// </summary>
+ StyleAttributeEmpty,
  }
 }