(css) Fix server-side CSS cleaner

Fixes #4366
Alinto · Jan 17, 2018 · bca655f · bca655f
1 parent d4c8f50
commit bca655f
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/NEWS b/NEWS
@@ -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)
 

diff --git a/UI/MailPartViewers/UIxMailPartHTMLViewer.m b/UI/MailPartViewers/UIxMailPartHTMLViewer.m
@@ -288,8 +288,8 @@ - (void) _appendStyle: (unichar *) _chars
                 }
               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;