121556 2006-02-07 18:57:39 +0000 Kpdf kpart make Konqueror crash when viewing some embedded Pdf 2006-02-18 12:53:13 +0000 1 1 10 Unmaintained kpdf general unspecified openSUSE Linux RESOLVED FIXED NOR crash --- 1 fidel78 aacid greedkills konqueror-bugs-null 0 oldest_to_newest 412301 0 fidel78 2006-02-07 18:57:39 +0000 Version: (using KDE KDE 3.5.0) Installed from: SuSE RPMs OS: Linux At this web site: http://www.gazzettaufficiale.it/guri/cgi-bin/db2www/arti/arti.mac/report4?service=0&numeroarticolo=9999&subarticolo=1&progressivo=0&versione=1&redazione=06A01262&dataGazzetta=2006-02-06&numgu=30&cdimg=06A01262999901 Konqueror embeds Kpdf to view the pdf file. All seems ok, but if I try to go back with Konqueror "back" button for example, or I try to navigate with the links on the left side of the page, Konqueror crashs! This crash happens not only on this page, but also with other pages of www.gazzettaufficiale.it containing embedded pdf files. I'll post the crash backtrace shortly. 412303 1 14599 fidel78 2006-02-07 19:01:03 +0000 Created attachment 14599 Crash backtrace 412643 2 aacid 2006-02-08 23:20:19 +0000 I'd say the problem here is that konqueror removes things from kpdf foot too late in the process of "destroying" the html page to load the other and kpdf as problems with that, what do you konq guys think? Valgrindizin seems the "ui" of kpdf got destroyed before than the part itself ==10759== Invalid read of size 8 ==10759== at 0x1A194F92: KPDFDocument::closeDocument() (document.cpp:252) ==10759== by 0x1A19523C: KPDFDocument::~KPDFDocument() (document.cpp:127) ==10759== by 0x1A09886F: KPDF::Part::~Part() (part.cpp:294) ==10759== by 0x1205E6BA: KParts::Part::slotWidgetDestroyed() (in /home/kde35/instalado/lib/libkparts.so.2.1.0) ==10759== by 0x1205E85C: KParts::Part::qt_invoke(int, QUObject*) (in /home/kde35/instalado/lib/libkparts.so.2.1.0) ==10759== by 0x1205EAD9: KParts::ReadOnlyPart::qt_invoke(int, QUObject*) (in /home/kde35/instalado/lib/libkparts.so.2.1.0) ==10759== by 0x1A09D752: KPDF::Part::qt_invoke(int, QUObject*) (part.moc:214) ==10759== by 0x13E36878: QObject::activate_signal(QConnectionList*, QUObject*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13E372DF: QObject::activate_signal(int) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x14172315: QObject::destroyed() (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13E352CC: QObject::~QObject() (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13E6D04D: QWidget::~QWidget() (in /usr/lib/libqt-mt.so.3.3.4) ==10759== Address 0x19933B10 is 248 bytes inside a block of size 264 free'd ==10759== at 0x11B1DCCC: operator delete(void*) (vg_replace_malloc.c:246) ==10759== by 0x1A1A8EB5: PageView::~PageView() (pageview.cpp:175) ==10759== by 0x13E6CF87: QWidget::~QWidget() (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x11F053D2: QFrame::~QFrame() (qframe.h:48) ==10759== by 0x13F6C888: QSplitter::~QSplitter() (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13E338F4: QObject::event(QEvent*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13E6E36A: QWidget::event(QEvent*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13F6DC4E: QSplitter::event(QEvent*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13DD11E3: QApplication::internalNotify(QObject*, QEvent*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x13DD1F3F: QApplication::notify(QObject*, QEvent*) (in /usr/lib/libqt-mt.so.3.3.4) ==10759== by 0x1309E9E0: KApplication::notify(QObject*, QEvent*) (kapplication.cpp:550) ==10759== by 0x11CC698B: QApplication::sendEvent(QObject*, QEvent*) (qapplication.h:491) 415121 3 aacid 2006-02-18 11:34:43 +0000 *** Bug 122200 has been marked as a duplicate of this bug. *** 415136 4 aacid 2006-02-18 12:53:12 +0000 SVN commit 510876 by aacid: Reoder some deletes, add some removeObserver and use some magic QGuardPtr so that when embedded on konqueror and changing page it does crash, i still think that konqueror should not delete kpart created widgets behind its back, but probably it's easier to change kpdf than all the other code. BUGS: 121556 M +16 -11 part.cpp M +7 -3 part.h M +6 -0 ui/minibar.cpp M +2 -0 ui/minibar.h M +2 -1 ui/pageview.cpp M +2 -1 ui/thumbnaillist.cpp M +6 -1 ui/toc.cpp M +2 -1 ui/toc.h 14599 2006-02-07 19:01:03 +0000 2006-02-07 19:01:03 +0000 Crash backtrace Backtrace text/plain 4149 fidel78 VXNpbmcgaG9zdCBsaWJ0aHJlYWRfZGIgbGlicmFyeSAiL2xpYi90bHMvbGlidGhyZWFkX2RiLnNv LjEiLgpbVGhyZWFkIGRlYnVnZ2luZyB1c2luZyBsaWJ0aHJlYWRfZGIgZW5hYmxlZF0KW05ldyBU aHJlYWQgMTA5NjA4OTEyMCAoTFdQIDExMzgxKV0KW0tDcmFzaCBoYW5kbGVyXQojNyAgMHg0MjUy ZDE3MCBpbiBRVmFsdWVWZWN0b3JQcml2YXRlPFBhZ2VWaWV3SXRlbSo+OjpRVmFsdWVWZWN0b3JQ cml2YXRlICgpCiAgIGZyb20gL29wdC9rZGUzL2xpYi9rZGUzL2xpYmtwZGZwYXJ0LnNvCiM4ICAw eDQyNTJkMWUzIGluIFFWYWx1ZVZlY3RvcjxQYWdlVmlld0l0ZW0qPjo6ZGV0YWNoSW50ZXJuYWwg KCkKICAgZnJvbSAvb3B0L2tkZTMvbGliL2tkZTMvbGlia3BkZnBhcnQuc28KIzkgIDB4NDI1MjUx OTEgaW4gUGFnZVZpZXc6Om5vdGlmeVNldHVwICgpCiAgIGZyb20gL29wdC9rZGUzL2xpYi9rZGUz L2xpYmtwZGZwYXJ0LnNvCiMxMCAweDQyNTA2YjJhIGluIEtQREZEb2N1bWVudDo6Y2xvc2VEb2N1 bWVudCAoKQogICBmcm9tIC9vcHQva2RlMy9saWIva2RlMy9saWJrcGRmcGFydC5zbwojMTEgMHg0 MjUwN2MxOSBpbiBLUERGRG9jdW1lbnQ6On5LUERGRG9jdW1lbnQgKCkKICAgZnJvbSAvb3B0L2tk ZTMvbGliL2tkZTMvbGlia3BkZnBhcnQuc28KIzEyIDB4NDI0NDEwMzkgaW4gS1BERjo6UGFydDo6 flBhcnQgKCkgZnJvbSAvb3B0L2tkZTMvbGliL2tkZTMvbGlia3BkZnBhcnQuc28KIzEzIDB4NDAw NWUyNTkgaW4gS1BhcnRzOjpQYXJ0OjpzbG90V2lkZ2V0RGVzdHJveWVkICgpCiAgIGZyb20gL29w dC9rZGUzL2xpYi9saWJrcGFydHMuc28uMgojMTQgMHg0MDA1ZTJkOCBpbiBLUGFydHM6OlBhcnQ6 OnF0X2ludm9rZSAoKSBmcm9tIC9vcHQva2RlMy9saWIvbGlia3BhcnRzLnNvLjIKIzE1IDB4NDAw NWUzMzQgaW4gS1BhcnRzOjpSZWFkT25seVBhcnQ6OnF0X2ludm9rZSAoKQogICBmcm9tIC9vcHQv a2RlMy9saWIvbGlia3BhcnRzLnNvLjIKIzE2IDB4NDI0NDAyYzMgaW4gS1BERjo6UGFydDo6cXRf aW52b2tlICgpCiAgIGZyb20gL29wdC9rZGUzL2xpYi9rZGUzL2xpYmtwZGZwYXJ0LnNvCiMxNyAw eDQwYzVlYzRhIGluIFFPYmplY3Q6OmFjdGl2YXRlX3NpZ25hbCAoKQogICBmcm9tIC91c3IvbGli L3F0My9saWIvbGlicXQtbXQuc28uMwojMTggMHg0MGM1ZjRhNiBpbiBRT2JqZWN0OjphY3RpdmF0 ZV9zaWduYWwgKCkKICAgZnJvbSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10LnNvLjMKIzE5IDB4 NDBmYTYwNzkgaW4gUU9iamVjdDo6ZGVzdHJveWVkICgpIGZyb20gL3Vzci9saWIvcXQzL2xpYi9s aWJxdC1tdC5zby4zCiMyMCAweDQwYzYyNTNjIGluIFFPYmplY3Q6On5RT2JqZWN0ICgpIGZyb20g L3Vzci9saWIvcXQzL2xpYi9saWJxdC1tdC5zby4zCiMyMSAweDQwYzk5NmI2IGluIFFXaWRnZXQ6 On5RV2lkZ2V0ICgpIGZyb20gL3Vzci9saWIvcXQzL2xpYi9saWJxdC1tdC5zby4zCiMyMiAweDQw ZDg3ZTRkIGluIFFTcGxpdHRlcjo6flFTcGxpdHRlciAoKSBmcm9tIC91c3IvbGliL3F0My9saWIv bGlicXQtbXQuc28uMwojMjMgMHg0MGM5OTcxMiBpbiBRV2lkZ2V0Ojp+UVdpZGdldCAoKSBmcm9t IC91c3IvbGliL3F0My9saWIvbGlicXQtbXQuc28uMwojMjQgMHg0MGQ3ZDZiYiBpbiBRQ2xpcHBl cldpZGdldDo6flFDbGlwcGVyV2lkZ2V0ICgpCiAgIGZyb20gL3Vzci9saWIvcXQzL2xpYi9saWJx dC1tdC5zby4zCiMyNSAweDQwYzk5NzEyIGluIFFXaWRnZXQ6On5RV2lkZ2V0ICgpIGZyb20gL3Vz ci9saWIvcXQzL2xpYi9saWJxdC1tdC5zby4zCiMyNiAweDQwZDdkNzZiIGluIFFWaWV3cG9ydFdp ZGdldDo6flFWaWV3cG9ydFdpZGdldCAoKQogICBmcm9tIC91c3IvbGliL3F0My9saWIvbGlicXQt bXQuc28uMwojMjcgMHg0MGQ3YTljOCBpbiBRU2Nyb2xsVmlldzo6flFTY3JvbGxWaWV3ICgpCiAg IGZyb20gL3Vzci9saWIvcXQzL2xpYi9saWJxdC1tdC5zby4zCiMyOCAweDQxY2RmODNiIGluIEtI VE1MVmlldzo6fktIVE1MVmlldyAoKSBmcm9tIC9vcHQva2RlMy9saWIvbGlia2h0bWwuc28uNAoj MjkgMHg0MDA1ZTljYSBpbiBLUGFydHM6OlBhcnQ6On5QYXJ0ICgpIGZyb20gL29wdC9rZGUzL2xp Yi9saWJrcGFydHMuc28uMgojMzAgMHg0MDA2NWE2ZCBpbiBLUGFydHM6OlJlYWRPbmx5UGFydDo6 flJlYWRPbmx5UGFydCAoKQogICBmcm9tIC9vcHQva2RlMy9saWIvbGlia3BhcnRzLnNvLjIKIzMx IDB4NDFjZGVjZmUgaW4gS0hUTUxQYXJ0Ojp+S0hUTUxQYXJ0ICgpIGZyb20gL29wdC9rZGUzL2xp Yi9saWJraHRtbC5zby40CiMzMiAweDQxY2RkNzk3IGluIEtIVE1MUGFydDo6Y2xlYXIgKCkgZnJv bSAvb3B0L2tkZTMvbGliL2xpYmtodG1sLnNvLjQKIzMzIDB4NDFjZGRkZmIgaW4gS0hUTUxQYXJ0 OjpyZXN0b3JlU3RhdGUgKCkgZnJvbSAvb3B0L2tkZTMvbGliL2xpYmtodG1sLnNvLjQKIzM0IDB4 NDFjODQ4MTEgaW4gS0hUTUxQYXJ0QnJvd3NlckV4dGVuc2lvbjo6cmVzdG9yZVN0YXRlICgpCiAg IGZyb20gL29wdC9rZGUzL2xpYi9saWJraHRtbC5zby40CiMzNSAweDQxN2FmMmJhIGluIEtvbnFW aWV3OjpyZXN0b3JlSGlzdG9yeSAoKQogICBmcm9tIC9vcHQva2RlMy9saWIvbGlia2RlaW5pdF9r b25xdWVyb3Iuc28KIzM2IDB4NDE3YzAyNzMgaW4gS29ucVZpZXc6OmdvICgpIGZyb20gL29wdC9r ZGUzL2xpYi9saWJrZGVpbml0X2tvbnF1ZXJvci5zbwojMzcgMHg0MTdkMTQxNSBpbiBLb25xTWFp bldpbmRvdzo6c2xvdEdvSGlzdG9yeURlbGF5ZWQgKCkKICAgZnJvbSAvb3B0L2tkZTMvbGliL2xp YmtkZWluaXRfa29ucXVlcm9yLnNvCiMzOCAweDQxN2QyNDhkIGluIEtvbnFNYWluV2luZG93Ojpx dF9pbnZva2UgKCkKICAgZnJvbSAvb3B0L2tkZTMvbGliL2xpYmtkZWluaXRfa29ucXVlcm9yLnNv CiMzOSAweDQwYzVlY2ZlIGluIFFPYmplY3Q6OmFjdGl2YXRlX3NpZ25hbCAoKQogICBmcm9tIC91 c3IvbGliL3F0My9saWIvbGlicXQtbXQuc28uMwojNDAgMHg0MGZhNzNjMiBpbiBRU2lnbmFsOjpz aWduYWwgKCkgZnJvbSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10LnNvLjMKIzQxIDB4NDBjN2Jk MDMgaW4gUVNpZ25hbDo6YWN0aXZhdGUgKCkgZnJvbSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10 LnNvLjMKIzQyIDB4NDBjODM2ODMgaW4gUVNpbmdsZVNob3RUaW1lcjo6ZXZlbnQgKCkKICAgZnJv bSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10LnNvLjMKIzQzIDB4NDBiZmNiOWYgaW4gUUFwcGxp Y2F0aW9uOjppbnRlcm5hbE5vdGlmeSAoKQogICBmcm9tIC91c3IvbGliL3F0My9saWIvbGlicXQt bXQuc28uMwojNDQgMHg0MGJmZTc0OSBpbiBRQXBwbGljYXRpb246Om5vdGlmeSAoKSBmcm9tIC91 c3IvbGliL3F0My9saWIvbGlicXQtbXQuc28uMwojNDUgMHg0MDg4OTdhZSBpbiBLQXBwbGljYXRp b246Om5vdGlmeSAoKSBmcm9tIC9vcHQva2RlMy9saWIvbGlia2RlY29yZS5zby40CiM0NiAweDQw YmYxNDhjIGluIFFFdmVudExvb3A6OmFjdGl2YXRlVGltZXJzICgpCiAgIGZyb20gL3Vzci9saWIv cXQzL2xpYi9saWJxdC1tdC5zby4zCiM0NyAweDQwYmFiMTQzIGluIFFFdmVudExvb3A6OnByb2Nl c3NFdmVudHMgKCkKICAgZnJvbSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10LnNvLjMKIzQ4IDB4 NDBjMTQ4NDEgaW4gUUV2ZW50TG9vcDo6ZW50ZXJMb29wICgpIGZyb20gL3Vzci9saWIvcXQzL2xp Yi9saWJxdC1tdC5zby4zCiM0OSAweDQwYzE0Njg2IGluIFFFdmVudExvb3A6OmV4ZWMgKCkgZnJv bSAvdXNyL2xpYi9xdDMvbGliL2xpYnF0LW10LnNvLjMKIzUwIDB4NDBiZmU2MWYgaW4gUUFwcGxp Y2F0aW9uOjpleGVjICgpIGZyb20gL3Vzci9saWIvcXQzL2xpYi9saWJxdC1tdC5zby4zCiM1MSAw eDQxN2Q1NTNjIGluIGtkZW1haW4gKCkgZnJvbSAvb3B0L2tkZTMvbGliL2xpYmtkZWluaXRfa29u cXVlcm9yLnNvCiM1MiAweDQwMDE4OGI0IGluIGtkZWluaXRtYWluICgpIGZyb20gL29wdC9rZGUz L2xpYi9rZGUzL2tvbnF1ZXJvci5zbwojNTMgMHgwODA0ZWE0OCBpbiBsYXVuY2ggKCkKIzU0IDB4 MDgwNGYxNWMgaW4gaGFuZGxlX2xhdW5jaGVyX3JlcXVlc3QgKCkKIzU1IDB4MDgwNGY3MTkgaW4g aGFuZGxlX3JlcXVlc3RzICgpCiM1NiAweDA4MDRmZjkxIGluIG1haW4gKCkK