(css) Fix server-side CSS cleaner

Fixes #4366
2018-01-17 13:58:42 -05:00 · 2018-01-17 13:58:42 -05:00 · bca655ffdd
parent d4c8f5089b
commit bca655ffdd
2 changed files with 3 additions and 2 deletions
--- a/1
+++ b/1
@ -44,6 +44,7 @@ Bug fixes
 - [web] removed null characters from JSON responses
 - [web] fixed advanced mailbox search when mailbox name is very long
 - [web] fixed handling of public access rights of Calendars (#4344)
+ - [web] fixed server-side CSS sanitization of messages (#4366)
 - [eas] hebrew folders encoding problem using EAS (#4240)
 - [eas] avoid sync requests for shared folders every second (#4275)

--- a/UI/MailPartViewers/UIxMailPartHTMLViewer.m
+++ b/UI/MailPartViewers/UIxMailPartHTMLViewer.m
@ -288,8 +288,8 @@ _xmlCharsetForCharset (NSString *charset)
                }
              else
                {
-                  // Prefix CSS rule
-                  length = (currentChar - start);
+                  // Prefix CSS rule including ending curly bracket
+                  length = (currentChar - start) + 1;
                  [declaration appendString: [NSString stringWithCharacters: start length: length]];
                  [css appendFormat: @".SOGoHTMLMail-CSS-Delimiter %@\n", declaration];
                  start = currentChar;