[Bug 154819] Entering applications:// crashes konqueror

Rafael Fernández López ereslibre at kde.org
Sun Dec 30 03:07:02 CET 2007


------- 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=154819         
ereslibre kde org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #22768|0                           |1
        is obsolete|                            |



------- Additional Comments From ereslibre kde org  2007-12-30 03:07 -------
Created an attachment (id=22769)
 --> (http://bugs.kde.org/attachment.cgi?id=22769&action=view)
Patch improved for fixing the crash.

This new patch is far safer, but I'm not sure if it will comply with all
standards. Basically, safer compared to the previous because in the case that
_str wasn't finishing in '/', I had trailingSlashesToRemove = -1, and it was
later being called with a count() - (-1), resulting in a bye-bye ranges.

It also prevents from crashing when calling "protocol:", by adding a '/'. This
is the part I'm not sure of taking care of standards...

Do you think this patch can be committed ? If so, please speak up.


More information about the Kdelibs-bugs mailing list