tinymce crashes konqueror 4.2

Bug #322542 reported by Philippe Clérié
24
This bug affects 1 person
Affects Status Importance Assigned to Milestone
kdelibs
Unknown
Medium
kde4libs (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

Binary package hint: kdebase

I'm on intrepid using the ppa packages for KDE 4.2. So it might be inappropriate to report a bug against jaunty. My apologies if it is.

I am developing an application with django and I'm trying to use tinymce with the admin interface and elsewhere. It works fine with Firefox. Doesn't work at all on Konqueror 3.5 and yes, that's a known issue, and it crashes 4.2, with SIGSEGV. Crash Handler says the backtrace is of no use.

I believe rich text support is new in konqueror so I thought you'd want to know about this issue.

Regards
Philippe

Revision history for this message
In , Constantin Berzan (cberzan) wrote :

Version: (using KDE KDE 3.5.6)
Installed from: Ubuntu Packages
OS: Linux

TinyMCE (http://tinymce.moxiecode.com/index.php) is a JavaScript-based WYSIWYG editor which is used on wordpress.com and other sites. The compatibility chart (http://wiki.moxiecode.com/index.php/TinyMCE:Compatiblity) says it works with IE, Firefox and Safari.
When used within Konqueror, it doesn't appear to work at all (all I see is HTML code). Changing the User Agent to any of the above mentioned browsers doesn't make it work, either.
There are example editors which you can test at http://tinymce.moxiecode.com/example.php (Load it in Firefox to see how it's supposed to look like)

The TinyMCE developers aren't willing to fix this (http://sourceforge.net/tracker/index.php?func=detail&aid=1652067&group_id=103281&atid=635682), because Konqueror has such a small userbase in their opinion. But if Konqueror and Safari both use variants of KHTML to render pages, why doesn't the editor work when identifying as Safari?
I really wish it wouldn't be necessary to use Firefox for posting on Wordpress.

Revision history for this message
In , Jose Padilla (sargepl) wrote :

The same with KDE 3.5.7 (k)ubuntu packages. Tiny MCE doesn't work with konqueror. I'm web developer and I use Tiny MCE for my web interfaces but it's a limitation that this piece of software only works on top of IE/Firefox/Safari. I think the problem could be the JavaScript support or something like this. Tiny MCE is popular software so it would be very important that finally it works here, in konqueror.

Revision history for this message
In , Cedramanpower (cedramanpower) wrote :

*** This bug has been confirmed by popular vote. ***

Revision history for this message
In , Francesco-doffizi (francesco-doffizi) wrote :

I think it should be fixed some way. Too many website working fine with other (Firefox) browsers don't in Konqueror and I think this won't help this browser.
I like it's fast to load and well integrated in my kde environment, but I'd like too to see it capable to view Web 2.0 pages such as for example some page with auction commands in Ebay (and not latestsiteintheworld.com...).
Even a hook to use gecko (is kecko project dead?) insite konqueror as needed could be wonderful...

Revision history for this message
In , FiNeX (finex) wrote :

Please, vote all for bug #48302 :-)

*** This bug has been marked as a duplicate of 48302 ***

Revision history for this message
In , SadEagle (maksim-kde) wrote :

Actually, please don't. #48302 is pretty much fixed, but additional work is needed for specific editors.

Revision history for this message
In , Tommi Tervo (tommi-tervo) wrote :

*** Bug 170537 has been marked as a duplicate of this bug. ***

Revision history for this message
In , Anders Lund (anders-alweb) wrote :

Why was my report marked as a dup og this? I do not claim to be anything else than konqueror/khtml, that is, my UA string is the default.

Revision history for this message
In , SadEagle (maksim-kde) wrote :

We're using this as a tracker bug for tinyMCE issues... Anyway, seems like there is an XSS check denying tinyMCE from working on the web.... Which of course didn't happen when I was testing locally.

Revision history for this message
In , SadEagle (maksim-kde) wrote :

SVN commit 887751 by orlovich:

- Fix domain propagation for iframe's with javascript: URLs and similar cases.
Makes tinyMCE bootup when on web.
- Properly permit XML documents in XSS checks.

CCBUG:141523

 M +2 -6 ecma/kjs_window.cpp
 M +24 -16 khtml_part.cpp
 M +2 -0 khtmlpart_p.h

WebSVN link: http://websvn.kde.org/?view=rev&revision=887751

Revision history for this message
In , Fleet-d (fleet-d) wrote :

I also tried to use TinyMCE Editor with Konqueror. I don't know what's the matter.

You can try it yourself here: http://tinymce.moxiecode.com/examples/full.php

It loads the editor, but in read-only mode. You cannot click anything or edit anything. When I enable JavaScript errors, it tellme that there's an error in tiny_mce.js line 1, but when I click on Debug it freezes. It also freezese when I open the javascript debugger manually then select tiny_mce.js from loaded scripts.

Since TinyMCE is a widely used editor, it would be great to fix to work it...

Revision history for this message
In , SadEagle (maksim-kde) wrote :

re: comment #10: that should be fixed by the change in comment #9. Please try with 4.2 beta 2.

Revision history for this message
In , Fleet-d (fleet-d) wrote :

Ah it really loads up and now it's not in read-only mode..

But as soon as I start typing and press Enter several times, it simply crashes. You can verify that by visiting http://tinymce.moxiecode.com/examples/full.php
I simply had to position the cursor after the word "Google", pressed enter, some text, enter again, etc and it crashed...

Revision history for this message
In , FiNeX (finex) wrote :
Download full text (6.4 KiB)

Crash confirmed using current trunk r898537.

This is my backtrace:

Application: Konqueror (konqueror), signal SIGABRT
0x00007fe761a24001 in nanosleep () from /lib/libc.so.6
[Current thread is 0 (LWP 16927)]

Thread 2 (Thread 0x7fe751cf4950 (LWP 16989)):
#0 0x00007fe763e1ffed in pthread_cond_timedwait@@GLIBC_2.3.2 () from /lib/libpthread.so.0
#1 0x00007fe76408dae7 in QWaitCondition::wait () from /usr/lib/libQtCore.so.4
#2 0x00007fe7640851f9 in ?? () from /usr/lib/libQtCore.so.4
#3 0x00007fe76408cd79 in ?? () from /usr/lib/libQtCore.so.4
#4 0x00007fe763e1bfc7 in start_thread () from /lib/libpthread.so.0
#5 0x00007fe761a545dd in clone () from /lib/libc.so.6
#6 0x0000000000000000 in ?? ()

Thread 1 (Thread 0x7fe766a20750 (LWP 16927)):
[KCrash Handler]
#5 0x00007fe7619b70c5 in raise () from /lib/libc.so.6
#6 0x00007fe7619b85e3 in abort () from /lib/libc.so.6
#7 0x00007fe7619b00e9 in __assert_fail () from /lib/libc.so.6
#8 0x00007fe755ba19d7 in khtml::DeleteCollapsibleWhitespaceCommandImpl::deleteWhitespace (this=0x29b2b70, pos=<value optimized out>)
    at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:1011
#9 0x00007fe755ba58ca in khtml::DeleteCollapsibleWhitespaceCommandImpl::doApply (this=0x29b2b70) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:1058
#10 0x00007fe755b9725f in khtml::EditCommandImpl::apply (this=0x29b2b70) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:251
#11 0x00007fe755b9e0ce in khtml::CompositeEditCommandImpl::applyCommandToComposite (this=0x2a41470, cmd=@0x7fff6eb5cfb0) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:377
#12 0x00007fe755b9e8fd in khtml::CompositeEditCommandImpl::deleteCollapsibleWhitespace (this=0x2a41470) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:503
#13 0x00007fe755ba54b5 in khtml::InputTextCommandImpl::execute (this=0x2a41470, text=@0x7fff6eb5d2e0) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:1645
#14 0x00007fe755b9e44d in khtml::TypingCommandImpl::insertText (this=0x29b0c30, text=@0x7fff6eb5d2e0) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting_impl.cpp:2378
#15 0x00007fe755b9668d in khtml::TypingCommand::insertText (document=0x2818d80, text=@0x7fff6eb5d2e0) at /home/test/KDE4/src/kdelibs/khtml/editing/htmlediting.cpp:740
#16 0x00007fe755b91b7d in DOM::Editor::handleKeyEvent (this=0x28f32b0, _ke=<value optimized out>) at /home/test/KDE4/src/kdelibs/khtml/editing/editor.cpp:528
#17 0x00007fe755a04dc9 in DOM::ElementImpl::defaultEventHandler (this=0x2715a80, e=0x29cbcb0) at /home/test/KDE4/src/kdelibs/khtml/xml/dom_elementimpl.cpp:787
#18 0x00007fe7559fc80c in DOM::NodeImpl::dispatchGenericEvent (this=0x2715a80, evt=0x29cbcb0) at /home/test/KDE4/src/kdelibs/khtml/xml/dom_nodeimpl.cpp:524
#19 0x00007fe7559fc9de in DOM::NodeImpl::dispatchEvent (this=0x2715a80, evt=0x29cbcb0, exceptioncode=@0x7fff6eb5d4c4, tempEvent=true) at /home/test/KDE4/src/kdelibs/khtml/xml/dom_nodeimpl.cpp:451
#20 0x00007fe7559fd310 in DOM::NodeImpl::dispatchKeyEvent (this=0x2715a80, key=0x29fa000, keypress=<value optimized out>) at /home/test/KDE4/src/kdelibs/khtml/xml/dom_nodeimpl.cpp:690
#21 0x...

Read more...

Changed in kdebase:
importance: Undecided → Medium
status: New → Triaged
Changed in kdelibs:
status: Unknown → Confirmed
Revision history for this message
In , SadEagle (maksim-kde) wrote :

SVN commit 948882 by orlovich:

Make sure that iframes created with javascript: URLs enter the proper mode ---
in particular HTML, not XHTML one. Us doing otherwise caused tinyMCE to
get confused (due to a paragraph being <p> and not <P>) and remove & clone text node
being typed on, which caused the editting code to get really confused and blow up.
This, in particular, affected GSoC editor page. Now typing in should be fine
(though I would still be careful with richtext there)

I do think the mode detection code needs rework, and the editing code will
need heavy work, but that's for an another time.

CCBUG:188445
CCBUG:141523

 M +4 -1 khtml_part.cpp

WebSVN link: http://websvn.kde.org/?view=rev&revision=948882

Revision history for this message
In , SadEagle (maksim-kde) wrote :

SVN commit 948884 by orlovich:

automatically merged revision 948882:
Make sure that iframes created with javascript: URLs enter the proper mode ---
in particular HTML, not XHTML one. Us doing otherwise caused tinyMCE to
get confused (due to a paragraph being <p> and not <P>) and remove & clone text node
being typed on, which caused the editting code to get really confused and blow up.
This, in particular, affected GSoC editor page. Now typing in should be fine
(though I would still be careful with richtext there)

I do think the mode detection code needs rework, and the editing code will
need heavy work, but that's for an another time.

CCBUG:188445
CCBUG:141523

 M +4 -1 khtml_part.cpp

WebSVN link: http://websvn.kde.org/?view=rev&revision=948884

Revision history for this message
Jonathan Thomas (echidnaman) wrote :

For our purposes this one should be fixed once our KDE 4.2.3 packages are released.

Changed in kde4libs (Ubuntu):
status: Triaged → Fix Committed
Revision history for this message
Jonathan Thomas (echidnaman) wrote :

Fix released to the development version of Kubuntu 9.10. (Also available for Kubuntu 9.04: http://www.kubuntu.org/news/kde-4.2.3)

Changed in kde4libs (Ubuntu):
status: Fix Committed → Fix Released
Revision history for this message
In , Tommi Tervo (tommi-tervo) wrote :

*** Bug 202207 has been marked as a duplicate of this bug. ***

Changed in kdelibs:
importance: Unknown → Medium
Revision history for this message
In , Justin Zobel (justin-zobel) wrote :

Thank you for the bug report.

As this report hasn't seen any changes in 10 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.

Revision history for this message
In , Cullmann-t (cullmann-t) wrote :

Dear user,

KHTML (and KJS) was a long time more or less unmaintained and got removed in KF6.

Please migrate to use a QWebEngine based HTML component.

We will do no further fixes or improvements to the KF5 branches of these components beside important security fixes.

For security issues, please see:

https://kde.org/info/security/

Sorry that we did not fix this issue during the life-time of KHTML.

Greetings
Christoph Cullmann

Changed in kdelibs:
status: Confirmed → Unknown
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.