Bug 295301 - crash on adding new bookmark
Summary: crash on adding new bookmark
Status: RESOLVED FIXED
Alias: None
Product: rekonq
Classification: Applications
Component: general (show other bugs)
Version: 0.9.0
Platform: Compiled Sources Linux
: NOR crash
Target Milestone: ---
Assignee: Andrea Diamantini
URL:
Keywords:
: 295894 (view as bug list)
Depends on:
Blocks:
 
Reported: 2012-03-04 14:25 UTC by Heinz Wiesinger
Modified: 2012-03-29 22:50 UTC (History)
3 users (show)

See Also:
Latest Commit:
Version Fixed In: 0.9.1


Attachments
crash report (12.62 KB, application/octet-stream)
2012-03-04 14:25 UTC, Heinz Wiesinger
Details
my crash report (8.32 KB, application/octet-stream)
2012-03-06 18:34 UTC, Paul W
Details
proposed fix (866 bytes, patch)
2012-03-11 09:54 UTC, Andrea Diamantini
Details
crash report (11.94 KB, application/octet-stream)
2012-03-11 10:49 UTC, Heinz Wiesinger
Details
New crash information added by DrKonqi (11.35 KB, text/plain)
2012-03-29 22:50 UTC, Justin Curtis Mathews
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Heinz Wiesinger 2012-03-04 14:25:06 UTC
Created attachment 69280 [details]
crash report

When adding a new bookmark, rekonq crashes after the bookmark is added (it's available after restarting rekonq). See attached crash report.
Comment 1 Andrea Diamantini 2012-03-05 21:49:39 UTC
uhm... I cannot reproduce here. And this crash is really strange. Heinz, you can reproduce it with rekonq 0.9 compiled from "Branches.0.9" branch just adding a new bookmark (eg: clicking on the star on the urlbar)?
Comment 2 Paul W 2012-03-06 18:33:42 UTC
I can confirm this bug. I compiled rekonq from Branches.0.9 and after adding some bookmarks it crashed.

Hower it is not easy to reproduce because it seems to be random, e.g it does not crash for while, but short time after, every time I add a bookmark it does crash.
Comment 3 Paul W 2012-03-06 18:34:44 UTC
Created attachment 69330 [details]
my crash report
Comment 4 Andrea Diamantini 2012-03-11 09:53:43 UTC
Can you please test a patch for me, too better understand the problem? If you prefer a diff patch, I'm attaching it here. It should work in master and in Branches.0.9.
If you prefer a git branch, it is 'Bug_295301_Proposed_Fix" on rekonq remote repo.

Thanks.
Comment 5 Andrea Diamantini 2012-03-11 09:54:15 UTC
Created attachment 69488 [details]
proposed fix
Comment 6 Andrea Diamantini 2012-03-11 09:57:53 UTC
Git commit ab7534b58128d99a0ec2cd77a981cba28d3df97b by Andrea Diamantini.
Committed on 11/03/2012 at 10:11.
Pushed by adjam into branch 'Bug_295301_Proposed_Fix'.

Check WebTab::view() existence.

I don't think this is needed in any way, according to code and docs.
But same strange crashes (eg: bug 295301) suggest I'm wrong.
This patch needs testing.

M  +5    -1    src/webtab.cpp
M  +7    -1    src/webtab.h

http://commits.kde.org/rekonq/ab7534b58128d99a0ec2cd77a981cba28d3df97b
Comment 7 Heinz Wiesinger 2012-03-11 10:21:39 UTC
Patch applied on top of Branches.0.9 and adding bookmarks works fine now :)
Comment 8 Heinz Wiesinger 2012-03-11 10:49:43 UTC
Created attachment 69490 [details]
crash report

Hrm, did again crash :/
It was working for some pages now, so there might be still some improvement over the previous situation. New crash report attached.
Comment 9 Andrea Diamantini 2012-03-12 23:21:13 UTC
Git commit f6cc2c5c13fbc219a6bc3391994b6cdeca44dd10 by Andrea Diamantini.
Committed on 13/03/2012 at 00:23.
Pushed by adjam into branch 'master'.

Get sure urlbar(s) are deleted when related webtab is

Heinz, Paul. Please test actual master before backporting this to
0.9 series. Many thanks for your help and for this bug report.
I'm crossing fingers to hear good news from your tests ;)

M  +1    -1    src/mainview.cpp
M  +4    -2    src/urlbar/urlbar.cpp
M  +36   -0    src/webpage.cpp
M  +9    -29   src/webpage.h
M  +27   -9    src/webtab.cpp
M  +9    -12   src/webtab.h
M  +8    -7    src/webview.cpp
M  +1    -0    src/webview.h

http://commits.kde.org/rekonq/f6cc2c5c13fbc219a6bc3391994b6cdeca44dd10
Comment 10 Andrea Diamantini 2012-03-13 09:06:47 UTC
*** Bug 295894 has been marked as a duplicate of this bug. ***
Comment 11 Andrea Diamantini 2012-03-13 09:55:03 UTC
Git commit 707c0b1761e2823e4253bcb441a1a54478e5b76d by Andrea Diamantini.
Committed on 13/03/2012 at 00:23.
Pushed by adjam into branch 'Branches.0.9'.

Get sure urlbar(s) are deleted when related webtab is

Heinz, Paul. Please test actual master before backporting this to
0.9 series. Many thanks for your help and for this bug report.
I'm crossing fingers to hear good news from your tests ;)

M  +1    -1    src/mainview.cpp
M  +4    -2    src/urlbar/urlbar.cpp
M  +36   -0    src/webpage.cpp
M  +9    -29   src/webpage.h
M  +27   -9    src/webtab.cpp
M  +9    -12   src/webtab.h
M  +8    -7    src/webview.cpp
M  +1    -0    src/webview.h

http://commits.kde.org/rekonq/707c0b1761e2823e4253bcb441a1a54478e5b76d
Comment 12 Andrea Diamantini 2012-03-18 05:26:20 UTC
Any news here? Can someone of you reproduce bug in current master or Branches.0.9?
Comment 13 Heinz Wiesinger 2012-03-18 09:50:47 UTC
Sorry, been busy this week. I'm running the version from Branches.0.9 for a couple days now and so far no crash.
Comment 14 rodneyp 2012-03-19 01:53:01 UTC
On Sun, 18 Mar 2012 05:26:20 Andrea Diamantini wrote:
> https://bugs.kde.org/show_bug.cgi?id=295301
> --- Comment #12 from Andrea Diamantini <adjam7@gmail.com> ---
> Any news here? Can someone of you reproduce bug in current master or
Branches.0.9?

Not readily. I was using the latest 0.9 available from  repositories for
openSUSE 12.1.  No updated app has been released there or in their
unstable branch.


Rod
Comment 15 Andrea Diamantini 2012-03-20 09:00:59 UTC
You'll find it fixed in the upcoming 0.9.1
Comment 16 Justin Curtis Mathews 2012-03-29 22:50:52 UTC
Created attachment 70003 [details]
New crash information added by DrKonqi

rekonq (0.9.0) on KDE Platform 4.8.1 (4.8.1) using Qt 4.8.0

- What I was doing when the application crashed: Pressing the back button the W key and the Command key on my Apple Keyboard.

- Unusual behavior I noticed: Closed unexpectedly.

- Custom settings of the application: None But I am Running and AMD 'DIY' setup that is 64-bit. And I am running Kubuntu 12.04 Beta 1 but I have upgraded it to Beta 2 since then.

-- Backtrace (Reduced):
#7  0x00007f8f50ea67a5 in page (this=0x2d1c4e0) at /build/buildd/rekonq-0.9.0-1/src/webtab.h:71
#8  WebTab::url (this=<optimized out>) at /build/buildd/rekonq-0.9.0-1/src/webtab.cpp:108
#9  0x00007f8f50ed5286 in UrlBar::loadFinished (this=0x2d20ee0) at /build/buildd/rekonq-0.9.0-1/src/urlbar/urlbar.cpp:354
#10 0x00007f8f50ed5da5 in qt_static_metacall (_a=<optimized out>, _id=<optimized out>, _o=<optimized out>, _c=<optimized out>) at /build/buildd/rekonq-0.9.0-1/obj-x86_64-linux-gnu/src/urlbar.moc:162
#11 UrlBar::qt_static_metacall (_o=<optimized out>, _c=<optimized out>, _id=<optimized out>, _a=<optimized out>) at /build/buildd/rekonq-0.9.0-1/obj-x86_64-linux-gnu/src/urlbar.moc:151