(fix) properly XML escape wide characters (fixes #3616)

ActiveSync/NSString+ActiveSync.m

@@ -43,99 +43,6 @@
 
 @implementation NSString (ActiveSync)
 
-//
-// This is a copy from NSString+XMLEscaping.m from SOPE.
-// The difference here is that we use wchar_t instead of unichar.
-// This is needed to get the rigth numeric character reference.
-// e.g. SMILING FACE WITH OPEN MOUTH
-//      ok: wchar_t -> 😃   wrong:  unichar -> � �
-//
-// We avoir naming it like the one in SOPE since if the ActiveSync
-// bundle is loaded, it'll overwrite the one provided by SOPE.
-//
-- (NSString *) _stringByEscapingXMLStringUsingCharacters {
-  register unsigned i, len, j;
-  register wchar_t *buf;
-  const wchar_t *chars;
-  unsigned escapeCount;
-
-  if ([self length] == 0) return @"";
-
-  NSData *data = [self dataUsingEncoding:NSUTF32StringEncoding];
-  chars = [data bytes];
-  len = [data length]/4;
-
-  /* check for characters to escape ... */
-  for (i = 0, escapeCount = 0; i < len; i++) {
-    switch (chars[i]) {
-      case '&': case '"': case '<': case '>': case '\r':
-        escapeCount++;
-        break;
-      default:
-        if (chars[i] < 0x20 || chars[i] > 127)
-          escapeCount++;
-        break;
-    }
-  }
-  if (escapeCount == 0 ) {
-    /* nothing to escape ... */
-    return [[self copy] autorelease];
-  }
-
-  buf = calloc((len + 5) + (escapeCount * 16), sizeof(wchar_t));
-  for (i = 0, j = 0; i < len; i++) {
-    switch (chars[i]) {
-      /* escape special chars */
-      case '\r':
-        buf[j] = '&'; j++; buf[j] = '#'; j++; buf[j] = '1'; j++;
-        buf[j] = '3'; j++; buf[j] = ';'; j++;
-        break;
-      case '&':
-        buf[j] = '&'; j++; buf[j] = 'a'; j++; buf[j] = 'm'; j++;
-        buf[j] = 'p'; j++; buf[j] = ';'; j++;
-        break;
-      case '"':
-        buf[j] = '&'; j++; buf[j] = 'q'; j++; buf[j] = 'u'; j++;
-        buf[j] = 'o'; j++; buf[j] = 't'; j++; buf[j] = ';'; j++;
-        break;
-      case '<':
-        buf[j] = '&'; j++; buf[j] = 'l'; j++; buf[j] = 't'; j++;
-        buf[j] = ';'; j++;
-        break;
-      case '>':
-        buf[j] = '&'; j++; buf[j] = 'g'; j++; buf[j] = 't'; j++;
-        buf[j] = ';'; j++;
-        break;
-
-      default:
-        /* escape big chars */
-        if (chars[i] > 127) {
-          unsigned char nbuf[32];
-          unsigned int k;
-
-          sprintf((char *)nbuf, "&#%i;", (int)chars[i]);
-          for (k = 0; nbuf[k] != '\0'; k++) {
-            buf[j] = nbuf[k];
-            j++;
-          }
-        }
-        else if (chars[i] == 0x9 || chars[i] == 0xA || chars[i] == 0xD || chars[i] >= 0x20) { // ignore any unsupported control character
-          /* nothing to escape */
-          buf[j] = chars[i];
-          j++;
-        }
-        break;
-    }
-  }
-
-  self = [[NSString alloc] initWithBytesNoCopy: buf
-                                        length: (j*sizeof(wchar_t))
-                                      encoding: NSUTF32StringEncoding
-                                  freeWhenDone: YES];
-
-  return [self autorelease];
-}
-
 - (NSString *) sanitizedServerIdWithType: (SOGoMicrosoftActiveSyncFolderType) folderType
 {
   if (folderType == ActiveSyncEventFolder)
@@ -155,7 +62,7 @@ - (NSString *) sanitizedServerIdWithType: (SOGoMicrosoftActiveSyncFolderType) fo
 
 - (NSString *) activeSyncRepresentationInContext: (WOContext *) context
 {
-  return [self _stringByEscapingXMLStringUsingCharacters];
+  return [self safeStringByEscapingXMLString];
 }
 
 - (int) activeSyncFolderType

SoObjects/Appointments/SOGoAppointmentFolder.m

@@ -2140,7 +2140,7 @@ - (NSDictionary *) freebusyResponseForRecipient: (iCalPerson *) recipient
       [content addObject: davElementWithContent (@"request-status", XMLNS_CALDAV,
 						 @"2.0;Success")];
       [content addObject: davElementWithContent (@"calendar-data", XMLNS_CALDAV,
-                                                 [calendarData stringByEscapingXMLString])];
+                                                 [calendarData safeStringByEscapingXMLString])];
     }
   else
       [content addObject:

SoObjects/Contacts/SOGoContactSourceFolder.m

@@ -498,7 +498,7 @@ - (NSString **) _properties: (NSString **) properties
       methodSel = SOGoSelectorForPropertyGetter (*currentProperty);
       if (methodSel && [ldifEntry respondsToSelector: methodSel])
         *currentValue = [[ldifEntry performSelector: methodSel]
-                          stringByEscapingXMLString];
+                          safeStringByEscapingXMLString];
       currentProperty++;
       currentValue++;
     }

SoObjects/Contacts/SOGoFolder+CardDAV.m

@@ -28,6 +28,7 @@
 #import <DOM/DOMNode.h>
 #import <SaxObjC/SaxObjC.h>
 
+#import <SOGo/NSString+Utilities.h>
 #import <SOGo/SOGoUser.h>
 #import <SOGo/WOResponse+SOGo.h>
 
@@ -67,7 +68,7 @@ - (void) _appendObject: (NSDictionary *) object
                            [component davEntityTag]];
       [r appendContentString: etagLine];
       [r appendContentString: @"<C:address-data>"];
-      contactString = [[component contentAsString] stringByEscapingXMLString];
+      contactString = [[component contentAsString] safeStringByEscapingXMLString];
       [r appendContentString: contactString];
       [r appendContentString: @"</C:address-data>"
                         @"<C:addressbook-data>"];

SoObjects/SOGo/NSString+Utilities.h

@@ -54,6 +54,7 @@
 
 /* Unicode safety */
 - (NSString *) safeString;
+- (NSString *) safeStringByEscapingXMLString;
 
 /* JSON */
 - (NSString *) jsonRepresentation;

SoObjects/SOGo/NSString+Utilities.m

@@ -319,6 +319,100 @@ - (NSString *) safeString
   return AUTORELEASE(s);
 }
 
+
+//
+// This is a copy from NSString+XMLEscaping.m from SOPE.
+// The difference here is that we use wchar_t instead of unichar.
+// This is needed to get the rigth numeric character reference.
+// e.g. SMILING FACE WITH OPEN MOUTH
+//      ok: wchar_t -> 😃   wrong:  unichar -> � �
+//
+// We avoid naming it like the one in SOPE since if the ActiveSync
+// bundle is loaded, it'll overwrite the one provided by SOPE.
+//
+- (NSString *) safeStringByEscapingXMLString {
+  register unsigned i, len, j;
+  register wchar_t *buf;
+  const wchar_t *chars;
+  unsigned escapeCount;
+
+  if ([self length] == 0) return @"";
+
+  NSData *data = [self dataUsingEncoding:NSUTF32StringEncoding];
+  chars = [data bytes];
+  len = [data length]/4;
+
+  /* check for characters to escape ... */
+  for (i = 0, escapeCount = 0; i < len; i++) {
+    switch (chars[i]) {
+      case '&': case '"': case '<': case '>': case '\r':
+        escapeCount++;
+        break;
+      default:
+        if (chars[i] < 0x20 || chars[i] > 127)
+          escapeCount++;
+        break;
+    }
+  }
+  if (escapeCount == 0 ) {
+    /* nothing to escape ... */
+    return [[self copy] autorelease];
+  }
+
+  buf = calloc((len + 5) + (escapeCount * 16), sizeof(wchar_t));
+  for (i = 0, j = 0; i < len; i++) {
+    switch (chars[i]) {
+      /* escape special chars */
+      case '\r':
+        buf[j] = '&'; j++; buf[j] = '#'; j++; buf[j] = '1'; j++;
+        buf[j] = '3'; j++; buf[j] = ';'; j++;
+        break;
+      case '&':
+        buf[j] = '&'; j++; buf[j] = 'a'; j++; buf[j] = 'm'; j++;
+        buf[j] = 'p'; j++; buf[j] = ';'; j++;
+        break;
+      case '"':
+        buf[j] = '&'; j++; buf[j] = 'q'; j++; buf[j] = 'u'; j++;
+        buf[j] = 'o'; j++; buf[j] = 't'; j++; buf[j] = ';'; j++;
+        break;
+      case '<':
+        buf[j] = '&'; j++; buf[j] = 'l'; j++; buf[j] = 't'; j++;
+        buf[j] = ';'; j++;
+        break;
+      case '>':
+        buf[j] = '&'; j++; buf[j] = 'g'; j++; buf[j] = 't'; j++;
+        buf[j] = ';'; j++;
+        break;
+
+      default:
+        /* escape big chars */
+        if (chars[i] > 127) {
+          unsigned char nbuf[32];
+          unsigned int k;
+
+          sprintf((char *)nbuf, "&#%i;", (int)chars[i]);
+          for (k = 0; nbuf[k] != '\0'; k++) {
+            buf[j] = nbuf[k];
+            j++;
+          }
+        }
+        else if (chars[i] == 0x9 || chars[i] == 0xA || chars[i] == 0xD || chars[i] >= 0x20) { // ignore any unsupported control character
+          /* nothing to escape */
+          buf[j] = chars[i];
+          j++;
+        }
+        break;
+    }
+  }
+
+  self = [[NSString alloc] initWithBytesNoCopy: buf
+                                        length: (j*sizeof(wchar_t))
+                                      encoding: NSUTF32StringEncoding
+                                  freeWhenDone: YES];
+
+  return [self autorelease];
+}
+
 - (NSString *) jsonRepresentation
 {
   NSString *cleanedString;

SoObjects/SOGo/SOGoGCSFolder.m

@@ -2076,7 +2076,7 @@ - (NSString **) _properties: (NSString **) properties
       methodSel = SOGoSelectorForPropertyGetter (*currentProperty);
       if (methodSel && [sogoObject respondsToSelector: methodSel])
         *currentValue = [[sogoObject performSelector: methodSel]
-                          stringByEscapingXMLString];
+                          safeStringByEscapingXMLString];
       currentProperty++;
       currentValue++;
     }

SoObjects/SOGo/SOGoObject.m

@@ -1255,7 +1255,7 @@ - (NSString *) davRecordForUser: (NSString *) user
       if (!cn)
 	cn = user;
       [userRecord appendFormat: @"<displayName>%@</displayName>",
-		  [cn stringByEscapingXMLString]];
+		  [cn safeStringByEscapingXMLString]];
     }
 
   if (![params containsObject: @"noemail"])

SoObjects/SOGo/SOGoUserFolder.m

@@ -37,6 +37,7 @@
 
 #import "NSArray+Utilities.h"
 #import "NSDictionary+Utilities.h"
+#import "NSString+Utilities.h"
 #import "SOGoUserManager.h"
 #import "SOGoPermissions.h"
 #import "SOGoSystemDefaults.h"
@@ -271,7 +272,7 @@ - (void) _appendFolders: (NSArray *) folders
       [r appendContentString: @"<D:status>HTTP/1.1 200 OK</D:status>"];
       [r appendContentString: @"<D:prop><D:displayname>"];
       data = [currentFolder objectForKey: @"displayName"];
-      [r appendContentString: [data stringByEscapingXMLString]];
+      [r appendContentString: [data safeStringByEscapingXMLString]];
       [r appendContentString: @"</D:displayname></D:prop></D:propstat>"];
 
       /* Remove this once extensions 0.8x are no longer used */
@@ -284,12 +285,12 @@ - (void) _appendFolders: (NSArray *) folders
       ownerUser = [SOGoUser userWithLogin: [currentFolder objectForKey: @"owner"]
 			    roles: nil];
       data = [ownerUser cn];
-      [r appendContentString: [data stringByEscapingXMLString]];
+      [r appendContentString: [data safeStringByEscapingXMLString]];
       [r appendContentString: @"</ownerdisplayname>"];
 
       [r appendContentString: @"<D:displayname>"];
       data = [currentFolder objectForKey: @"displayName"];
-      [r appendContentString: [data stringByEscapingXMLString]];
+      [r appendContentString: [data safeStringByEscapingXMLString]];
       [r appendContentString: @"</D:displayname>"];
       /* end of temporary compatibility hack */
 
@@ -421,14 +422,14 @@ - (NSString *) _davFetchUsersMatching: (NSString *) user
                      [field stringByEscapingXMLString]];
               field = [currentUser objectForKey: @"cn"];
               [fetch appendFormat: @"<displayName>%@</displayName>",
-                     [field stringByEscapingXMLString]];
+                     [field safeStringByEscapingXMLString]];
               field = [currentUser objectForKey: @"c_email"];
               [fetch appendFormat: @"<email>%@</email>",
                      [field stringByEscapingXMLString]];
               field = [currentUser objectForKey: @"c_info"];
               if ([field length])
                 [fetch appendFormat: @"<info>%@</info>",
-                       [field stringByEscapingXMLString]];
+                       [field safeStringByEscapingXMLString]];
               [fetch appendString: @"</user>"];
             }
         }