[Korganizer-devel] [Bug 140688] calendar file grows by 20 megabytes causes korganizer to crash (all system memory exhausted) when any operation on it is performed

[Reinhold Kainhofer]

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
         
http://bugs.kde.org/show_bug.cgi?id=140688         
reinhold kainhofer com changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED



------- Additional Comments From reinhold kainhofer com  2007-01-27 00:55 -------
SVN commit 627541 by kainhofe:

When loading any url/uri (attachments, audio alarm files, procedure alarm application), make sure we load it as utf8 and not latin1.

By default, QString(const char*) treats the string as latin1, so we need to use QString::fromUtf8 instead.

BUG: 140688


 M  +2 -2      icalformatimpl.cpp  


--- branches/KDE/3.5/kdepim/libkcal/icalformatimpl.cpp #627540:627541
 @ -1204,11 +1204,11  @
     if (isurl == 0)
       attachment = new Attachment((const char*)icalattach_get_data(a));
     else {
-      attachment = new Attachment(QString(icalattach_get_url(a)));
+      attachment = new Attachment(QString::fromUtf8(icalattach_get_url(a)));
     }
   }
   else if ( value_kind == ICAL_URI_VALUE ) {
-    attachment = new Attachment(QString(icalvalue_get_uri(icalproperty_get_value(attach))));
+    attachment = new Attachment(QString::fromUtf8(icalvalue_get_uri(icalproperty_get_value(attach))));
   }
 
   icalparameter *p = icalproperty_get_first_parameter(attach, ICAL_FMTTYPE_PARAMETER);