evolution crashed with SIGSEGV in strcmp()
- Jaunty (9.04)
- Bug #359658
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
evolution-indicator (Ubuntu) |
Fix Released
|
Medium
|
Ted Gould | ||
Jaunty |
Won't Fix
|
Medium
|
C de-Avillez |
Bug Description
Binary package hint: evolution
I didn't actually do anything - Evolution mail viewer was open on a mail message with a lot of large embedded images for about an hour and then it crashed, so I doubt it has anything to do with the content of the message
ProblemType: Crash
Architecture: amd64
DistroRelease: Ubuntu 9.04
ExecutablePath: /usr/bin/evolution
NonfreeKernelMo
Package: evolution 2.26.0-0ubuntu4
ProcCmdline: evolution --component=mail
ProcEnviron:
LANG=en_US.UTF-8
SHELL=/bin/bash
Signal: 11
SourcePackage: evolution
StacktraceTop:
strcmp () from /lib/libc.so.6
g_str_equal () from /usr/lib/
org_gnome_
?? () from /usr/lib/
e_event_emit ()
Title: evolution crashed with SIGSEGV in strcmp()
Uname: Linux 2.6.28-11-generic x86_64
UserGroups: adm admin cdrom dialout fuse lpadmin plugdev sambashare vboxusers
Oded Arbel (oded-geek) wrote : | #1 |
- Dependencies.txt Edit (5.5 KiB, text/plain; charset="utf-8")
- Disassembly.txt Edit (655 bytes, text/plain; charset="utf-8")
- ProcMaps.txt Edit (99.1 KiB, text/plain; charset="utf-8")
- ProcStatus.txt Edit (785 bytes, text/plain; charset="utf-8")
- Registers.txt Edit (945 bytes, text/plain; charset="utf-8")
- Stacktrace.txt Edit (1.3 KiB, text/plain; charset="utf-8")
- ThreadStacktrace.txt Edit (13.9 KiB, text/plain; charset="utf-8")
Apport retracing service (apport) wrote : Stacktrace.txt (retraced) | #2 |
Apport retracing service (apport) wrote : ThreadStacktrace.txt (retraced) | #3 |
Changed in evolution (Ubuntu): | |
importance: | Undecided → Medium |
tags: | removed: need-amd64-retrace |
Pedro Villavicencio (pedro) wrote : | #4 |
isn't this more like an evolution-indicator issue?
visibility: | private → public |
affects: | evolution (Ubuntu) → evolution-indicator (Ubuntu) |
Changed in evolution-indicator (Ubuntu): | |
assignee: | nobody → Ted Gould (ted-gould) |
Ted Gould (ted) wrote : | #5 |
Hello,
I've got a bug fix that I have put in my PPA:
https:/
Could those who can recreate this test to see if this fixes the bug?
For sponsors/release team I've proposed a merge of that package here:
https:/
Thanks everyone!
Ted
greenhunter (tierfreunde-hagenburg) wrote : | #6 |
hi,
I installed both packags. And did only one test sending myself on email and switching to another desktop. So no crash. But I didn't get a notification about new mail either.
Sebastien Bacher (seb128) wrote : | #7 |
the change has been uploaded now
Changed in evolution-indicator (Ubuntu Jaunty): | |
status: | New → Fix Committed |
greenhunter (tierfreunde-hagenburg) wrote : | #8 |
oh, oh
First I don't get any notifcations from evolution about new mails at all now.
Second evolution crashed while look into my new mails with courser up/down. (Possible a bug in new-mail counter.) I'm trying to reproduce it.
Ted Gould (ted) wrote : Re: [Bug 359658] Re: evolution crashed with SIGSEGV in strcmp() | #9 |
One of the reasons that you might not be getting notifications of new
mails is that the notifications aren't sent if any Evolution window is
in focus. So, you need to make sure you're working in another window
when looking.
greenhunter (tierfreunde-hagenburg) wrote : | #10 |
there is zero notification about emails in evolution. I tried diff workspaces, opened and focused other windows and min/maxed evolution.
Other notifications work well: evo-calender, printer, pidgin, volume...
Ted Gould (ted) wrote : | #11 |
On Wed, 2009-04-22 at 13:35 +0000, greenhunter wrote:
> there is zero notification about emails in evolution. I tried diff
> workspaces, opened and focused other windows and min/maxed evolution.
>
> Other notifications work well: evo-calender, printer, pidgin, volume...
Can you run evolution from the command line? The output there should
have information on why Evolution isn't sending notifications. Thanks!
greenhunter (tierfreunde-hagenburg) wrote : | #12 |
right,
s@z:~$ evolution
** (evolution:4144): DEBUG: mailto URL command: evolution %s
** (evolution:4144): DEBUG: mailto URL program: evolution
** (evolution:4144): DEBUG: EI: SHELL STARTUP
** (evolution:4144): DEBUG: EI: mail_read_notify
** (evolution:4144): DEBUG: MAIL SERVER: Count changed: 0
** (evolution:4144): DEBUG: Evolution is focused
** (evolution:4144): DEBUG: Evolution is focused
regardless what its real status is: evolution always thinks it is focused.
greenhunter (tierfreunde-hagenburg) wrote : | #13 |
hi
I was able to gather some new information:
Evolution only thinks it is on top, if "inbox" is currently selected. And it doesn't matter if evo is on another workspace or minimised.
If I select ex. trash folder in evo, create a new mail or do something else in another window. Notifications work properly.
Martin Pitt (pitti) wrote : | #14 |
The patch looks weird to me:
- if (g_str_equal (res_name, "evolution"))
+ if (g_strcmp0 (res_name, "evolution"))
g_str_equal() returns TRUE for identical strings, while g_strcmp0() returns 0 for identical strings, which is considered as "false". I don't see anywhere in the bug trail the intent to negate the condition.
Changed in evolution-indicator (Ubuntu Jaunty): | |
status: | Fix Committed → Incomplete |
Ted Gould (ted) wrote : | #15 |
Please try the updated package in the bugfix PPA. If you've still got
it installed, just update :)
greenhunter (tierfreunde-hagenburg) wrote : | #16 |
Is the build only of the fix for amd only? I need 386.
Ted Gould (ted) wrote : | #17 |
On Thu, 2009-04-23 at 20:41 +0000, greenhunter wrote:
> Is the build only of the fix for amd only? I need 386.
No, i386, AMD and LPIA. The x86 deb is here:
https:/
greenhunter (tierfreunde-hagenburg) wrote : | #18 |
nice,
finally it works like expected.
btw, why is there no notification-
At the moment notifications are totally useless if you don't watch the screen all the time.
And the number of new mails in evo inbox aren't shown in the not-applet.
Firefoxs old notification is really annoying among this new one.
Updates should be shown as well that way.
Martin Pitt (pitti) wrote : | #19 |
Ted, did you see my question in comment 14 about the inverted logic?
Ted Gould (ted) wrote : | #20 |
On Fri, 2009-04-24 at 10:16 +0000, Martin Pitt wrote:
> Ted, did you see my question in comment 14 about the inverted logic?
Yes, sorry. That's fixed in the new version.
Martin Pitt (pitti) wrote : | #21 |
Thanks, Ted. I rejected the current upload. Can you please send a new merge request for your fix?
Changed in evolution-indicator (Ubuntu): | |
status: | Incomplete → In Progress |
Changed in evolution-indicator (Ubuntu Jaunty): | |
status: | Incomplete → In Progress |
Launchpad Janitor (janitor) wrote : | #22 |
This bug was fixed in the package evolution-indicator - 0.1.15-0ubuntu1
---------------
evolution-indicator (0.1.15-0ubuntu1) karmic; urgency=low
* Upstream release 0.1.15: Brown paper bag. Missing a '!' in
previous release. Now really fixes LP: #359658
* Upstream release 0.1.14: Change to g_strcmp0 to check for a NULL
string and not crash when getting it. LP: #359658
-- Ted Gould <email address hidden> Mon, 11 May 2009 08:20:11 +0200
Changed in evolution-indicator (Ubuntu): | |
status: | In Progress → Fix Released |
Lars Oliver Hansen (lolh) wrote : | #23 |
Bug 381144 isn't solved by this issue.
A month ago Evolution suddenly didn't want to start anymore. Nothing displayed. dmesg shows evolution[8484]: segfault at 0 ip 00007f6a831a5092 sp 00007fff97d1bf98 error 4 in libc-2.
This is still true after the fix for this bug is applied.
I had tried reinstalling libc and evolution. Without success. evolution still segfaults. I haven't had local access to my e-mails for over one month now. Ubuntu Jaunty 9.04.
Please reinvestigate.
C de-Avillez (hggdh2) wrote : | #24 |
@Lars:
(1) what version of Evolution are you running?
(2) what version of Ubuntu are you running?
thanks.
Lars Oliver Hansen (lolh) wrote : | #25 |
Ubuntu 9.04 (jaunty)
Kernel Linux 2.6.28-11-generic
Gnome 2.26.1
latest updates as of today (all verses without backported) but I added Teds repo.
So I have evolution which came with Ubuntu 9.04 on the day it was released.
C de-Avillez (hggdh2) wrote : | #26 |
Lars, can you please open a new bug on that? I would like to see a stacktrace on it. You will need to enable apport: before trying to reproduce the issue:
sudo force_start=1 /etc/init.d/apport restart
then try to start Evolution. When you get the segfault, apport should kick in and ask you to file a new bug. Do it, and please subscribe myself (hggdh2).
Laurent Bonnaud (laurent-bonnaud) wrote : | #27 |
On Tue, 2009-06-16 at 15:01 +0000, C de-Avillez wrote:
> Lars, can you please open a new bug on that? I would like to see a
> stacktrace on it.
The bug I reported and merged has a good backtrace given by apport. See
bug #385263.
--
Laurent Bonnaud.
http://
C de-Avillez (hggdh2) wrote : | #28 |
Laurent, thanks, but bug 385263 seems to be a real duplicate of this one.
Unless... you *did* apply Ted's fix, and it still fails... in this case I really need a new bug, with a new stacktrace, so that we can follow the code and see if it is still a special case of this one, or a brand new issue.
krishan (srikrishanmalik) wrote : | #29 |
<assuming this is a duplicate of 363894 >
I am also hitting this after recent updates to ubuntu 9.04, here is the trace.
#0 0xb68de078 in strcmp () from /lib/tls/
#1 0xb6a28c64 in g_str_equal () from /usr/lib/
#2 0xb055db14 in org_gnome_
#3 0xb7d378e1 in ?? () from /usr/lib/
#4 0xb7d37e29 in e_plugin_invoke () from /usr/lib/
#5 0xb7d305da in ?? () from /usr/lib/
#6 0xb7d30761 in e_event_emit () from /usr/lib/
#7 0xb5a4c248 in ?? () from /usr/lib/
#8 0xb5a4d833 in ?? () from /usr/lib/
#9 0xb5a4f8aa in ?? () from /usr/lib/
#10 0xb6a01c81 in ?? () from /usr/lib/
#11 0xb6a03b88 in g_main_
#12 0xb6a070eb in ?? () from /usr/lib/
#13 0xb6a075ba in g_main_loop_run () from /usr/lib/
#14 0xb73eecc3 in bonobo_main () from /usr/lib/
#15 0x0805d563 in ?? ()
#16 0xb687d775 in __libc_start_main () from /lib/tls/
#17 0x08050221 in ?? ()
(gdb)
Brian J. Murrell (brian-interlinx) wrote : | #30 |
The package listed in comment #17 reporting to fix this:
https:/
seems to be no longer available.
On another note, why was this simply not put into jaunty-updates? I wonder how many other bugs that i run into on a frequent basis are fixed by somebody and simply not given to the users. You really are doing Ubuntu a disservice fixing all of these bugs but not actually giving them to the users.
I find Ubuntu so frustrating some days with all the bugs that it makes me wonder, for a very dark moment if i should not be using Windows. At least I get bug fixes from them.
Sebastien Bacher (seb128) wrote : | #31 |
> On another note, why was this simply not put into jaunty-updates? I wonder how many other bugs that i run into on a frequent basis are fixed by somebody and simply not given to the users
Because every single fix can't be backported to stable, it would require lot of efforts which we would put at fixing new bugs and lower the quality of the next versions which create a circle of lower quality
Concerning the no bug fixes comment try to compare the number of updates you get in both system you will see that you probably get as many bug fixed, microsoft doesn't fix any single issue in their updates either
Brian J. Murrell (brian-interlinx) wrote : | #32 |
On Thu, 2009-10-08 at 13:55 +0000, Sebastien Bacher wrote:
> > On another note, why was this simply not put into jaunty-updates? I
> wonder how many other bugs that i run into on a frequent basis are fixed
> by somebody and simply not given to the users
>
> Because every single fix can't be backported to stable,
What backporting? The dude had a package already created and ready to
go. The only thing left to do was actually give it to the users.
> it would require
> lot of efforts which we would put at fixing new bugs and lower the
> quality of the next versions which create a circle of lower quality
So the current situation, a current release which most users are using,
rife with bugs (that have been fixed and are just floating around in
PPAs even!) is better?
I agree that the next release always needs to be solid too, but the
reality is that the next release is always full of "shiny new" with it's
own set of bugs, so the user only ever has the choice of trading in old
bugs for new ones. There's never an answer to the user that just wants
something that's stable and will receive bug fixes without getting a
whole raft of new bugs.
I guess ultimately, if an organization doesn't have enough manpower to
maintain the current release and develop the next release, then the
organization's goals are too lofty for the resources the organization
has to achieve them.
> Concerning the no bug fixes comment try to compare the number of updates
> you get in both system you will see that you probably get as many bug
> fixed, microsoft doesn't fix any single issue in their updates either
Yeah. Ultimately, I do believe Windows is just as bad as the linux
desktop and would not trade the transparency of linux for the blackbox
of Windows. Some days though, it's very frustrating to be a linux
desktop user.
Evolution is a pig that doesn't scale and plain just doesn't work
(vfolders are still broken). Daily, Firefox 3.5 starts out at a few
hundred MB of memory and by the end of the day (or three) it's consuming
2-3G of memory. And that's just the most visible apps/bugs.
But I digress.
mvandemar (michael-endlesspoetry) wrote : | #33 |
Ok, this is very frustrating. I reported this identical behavior here:
https:/
"Apport retracing service" reported the following:
> "This crash has the same stack trace characteristics as bug #359658.
> However, the latter was already fixed in an earlier package version
> than the one in this report. This might be a regression or because
> the problem is in a dependent package."
So, does the phrase "already fixed in an earlier package version" mean something different on Launchpad than it would pretty much anywhere else? What am I missing here? Because a few days later it was marked as a duplicate of this bug. Is it the same bug, and was simply never fixed? Was the info about it being fixed simply wrong?
Also, to add to the symptoms, I have definitely determined over the past month that, in my case anyways, Evolution *only* crashes when a) vlc is running, and b) a new email comes in. I tried turning off the notifier applet to see if that helped, but it still crashes randomly under those circumstances. It is usually within 30-45 minutes that it crashes, although of course that is dependent on whether or not I actually get an email in that timeframe. Sometimes apport generates a report on the crash, sometimes not.
Can anyone else confirm that they are experiencing the same symptoms? Or can anyone else say that they definitely do not have vlc and yet still get this issue?
I am available to help test, if someone wants to give me a scenario to set up, if it will help.
Sebastien Bacher (seb128) wrote : | #34 |
> What backporting? The dude had a package already created and ready to go. The only thing left to do was actually give it to the users.
The stable update process require testing and validation of the updates by different team to assure there is no regression in the change
> So the current situation, a current release which most users are using, rife with bugs (that have been fixed and are just floating around in PPAs even!) is better?
The situation is not ideal but that's what can be done with the current manpower knowing that ubuntu doesn't write most of the software it's shipping you get unhappy users either way either because they lack features available in new versions if you work on old version only by fixing bugs or because bugs are not fixed on all versions when working on updates
Sebastien Bacher (seb128) wrote : | #35 |
let's also not turn this bug in a discussion about updates, the bottom line will be a manpower limitation anyway and a choice to do between updates, the number of softwares you ship and the versions you work on...
Changed in evolution-indicator (Ubuntu Jaunty): | |
assignee: | Ted Gould (ted) → nobody |
Sebastien Bacher (seb128) wrote : | #36 |
unassigning from ted since he's not working on the jaunty update there
Changed in evolution-indicator (Ubuntu Jaunty): | |
assignee: | nobody → Ubuntu Desktop Bugs (desktop-bugs) |
assignee: | Ubuntu Desktop Bugs (desktop-bugs) → nobody |
Sebastien Bacher (seb128) wrote : | #37 |
on a side note is somebody wanting to work on the jaunty update?
Martin Pitt (pitti) wrote : | #38 |
Brian J. Murrell [2009-10-08 14:40 -0000]:
> What backporting? The dude had a package already created and ready to
> go. The only thing left to do was actually give it to the users.
Where? There was a wrong patch sent here, but no jaunty package or
debdiff ever...
C de-Avillez (hggdh2) wrote : | #39 |
I will work on getting Ted's fix ported to Jaunty.
Changed in evolution-indicator (Ubuntu Jaunty): | |
assignee: | nobody → C de-Avillez (hggdh2) |
C de-Avillez (hggdh2) wrote : | #40 |
I have published evolution-indicator 0.1.13-
I would like someone to test and confirm it is working. I will only provide a debdiff when I have such a confirmation.
Thank you.
Thiago Figueiro (thiagocsf) wrote : | #41 |
I've installed evolution-
I will report back in a few days.
greenhunter (tierfreunde-hagenburg) wrote : | #42 |
Why don't you use evolution-
Thiago Figueiro (thiagocsf) wrote : | #43 |
2009/10/12 greenhunter <email address hidden>:
> Why don't you use evolution-
> zero chrashes for me.
Dunno - aptitude never offered to upgrade. I'll check why.
Thanks for the tip.
Cheers,
Thiago.
C de-Avillez (hggdh2) wrote : | #44 |
2009/10/12 greenhunter <email address hidden>:
> Why don't you use evolution-
> zero chrashes for me.
Because e-i 0.1.15 is for Karmic. And it is obsolete, anyway.
Although you personally can grab a version for a newer release and use it, we cannot.
C de-Avillez (hggdh2) wrote : | #45 |
Unassigning myself from the Jaunty task. There does not seem to be a lot of interest in this fix for Jaunty.
Changed in evolution-indicator (Ubuntu Jaunty): | |
assignee: | C de-Avillez (hggdh2) → Ubuntu Desktop Bugs (desktop-bugs) |
status: | In Progress → Triaged |
mvandemar (michael-endlesspoetry) wrote : | #46 |
C de-Avillez wrote on 2009-10-09:
> I have published evolution-indicator 0.1.13-
> on my PPA (https:/
> field.series_
> I would like someone to test and confirm it is working. I will only
> provide a debdiff when I have such a confirmation.
> Thank you.
Sorry for not testing this sooner. I just updated tonight using this fix and tested, and I can definitely confirm that it did *not* work for me.
What I cannot confirm, however, is that I gave the same bug as what everyone else is reporting. Evolution *only* crashes for me if VLC media player is running, and a new email comes in. Under those circumstances it crashes about 1-2 out of every 3 times, and it still happens after this update.
If this is not what others are experiencing, then the bug I originally reported (https:/
C de-Avillez (hggdh2) wrote : | #47 |
@mvandemar: thank you for your feedback. Indeed, bug 424099 seems to be this one -- which means the fix I cherry-picked did not do the trick. I will have to go back to it. :-(
Also, did the failure produce a crash (i.e., was apport triggered)? If so, I would like you to open a new bug via apport, and report here the bug #. I would like to see the backtrace.
mvandemar (michael-endlesspoetry) wrote : | #48 |
Bug 424099 contains a backtrace that I submitted via apport. I can try and force another crash and see if it comes up again, but I think I quit seeing the apport window for this crash a while ago. Is it possible that it got flagged as "already submitted" on my machine, or that I inadvertently told it not to display the window anymore, and if so, how can I clear that?
Also, is there a way to show a more complete report of what is going on at the time of the crash? If the last apport didn't show it, I am not sure what a new one would have that would be differently. I am willing to install whatever debugging/tracing apps you need me to.
C de-Avillez (hggdh2) wrote : | #49 |
bug 424099 has the crash *before* applying my proposed e-i fix, so it will not help me here. I need the crash with my fix in place, so that I can follow the backtrace. I checked the build logs, and the fix is indeed being applied.
One thing you can do: please verify if under /var/crash you have *any* files matching the pattern '*evolution-
1. please remove these files;
2. run 'sudo force_start=1 /etc/init.d/apport restart' to enable apport crash triggering on Jaunty;
3. repeat the error. Apport should trigger, and offer to report the bug. Please do so, and give me the bug number.
Additionally, if you want to try local debugging: the easiest way would be, after the crash (and after submitting the new bug), to run 'sudo apport-retrace -g /var/crash/
bt full
and
thread apply all bt full
and give me the output.
Thank you.
mvandemar (michael-endlesspoetry) wrote : | #50 |
And now, of course, I cannot replicate the crash. I am starting to think that I may have had Evolution running when I did the update, failed to restart before testing, and therefore the process running when it crashed was still the pre-patch one.
I will continue to test over the next couple of days, but as I said before it was not difficult to get it to bomb before at all.
C de-Avillez (hggdh2) wrote : | #51 |
Thank you. This is actually one thing I thought of, but decided to wait for the backtrace. Yes, this might have something to do with it.
As you propose, let's wait a couple of days. Thank you, very much.
mvandemar (michael-endlesspoetry) wrote : | #52 |
Zero crashes so far. Looks like it is working fine.
C de-Avillez (hggdh2) wrote : | #53 |
- debdiff for evo-indicator Jaunty Edit (1.7 KiB, text/plain)
Good. I am attaching the debdiff for this. Thank you very much.
Martin Pitt (pitti) wrote : | #54 |
Thanks! Uploaded.
Changed in evolution-indicator (Ubuntu Jaunty): | |
assignee: | Ubuntu Desktop Bugs (desktop-bugs) → C de-Avillez (hggdh2) |
status: | Triaged → Fix Committed |
tags: | added: verification-needed |
Martin Pitt (pitti) wrote : | #55 |
Accepted evolution-indicator into jaunty-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https:/
xermán (xerman.soto) wrote : | #56 |
Hello there,
In Ubuntu 904x86_64, Evolution just crashed. Nothing unusual happened before the crash. The process was the following:
1- Turn on Computer.
2- Start Evolution and Empathy.
3- Check, write and send some email in Evolution.
4- Launch Xsane in another Workspace.
5- Before pressing the "Preview" in Xsane, the crash came out.
Other things happening in Evolution are:
1- It keeps Folders in the IN Folder marked as "has emails not read" when it has none until evolution restarts.
2- Every time the app starts, checking contact list and selecting 1 contact from the list makes the contact list scroll up and down from begining to end of list before showing the contact information on the Preview Pane.
3- If the Preview Pane is not shown, the contact list just does de scroll up and down for several times and then stops.
Thanks all.
Best regards
Xermán
Martin Pitt (pitti) wrote : | #57 |
Anyone who can test the proposed package? If not, the proposed update will be removed again. Thanks!
Martin Pitt (pitti) wrote : | #58 |
This update has been in jaunty-proposed for half a year or longer, without any testing feedback. I removed the jaunty-proposed package again.
Changed in evolution-indicator (Ubuntu Jaunty): | |
status: | Fix Committed → Won't Fix |
Oded Arbel (oded-geek) wrote : | #59 |
I'm sorry, but I didn't encounter this problem again on my system.
My personal opinion as a developer is that changing bad code for good code is a good idea, and even if the original problem is not reproduceable and even if the patch is not being tested, that fix still makes sense and should go in - preventing SIGSEGV should be no. 1 priority of developers.
Here's the GNOME official documentation of g_str_equal() (the call being replaced by the patch):
---8<---
Note that this function is primarily meant as a hash table comparison function. For a general-purpose, NULL-safe string comparison function, see g_strcmp0().
---8<---
Which as I read it means: don't ever use g_str_equal() as a general string comparison method. IMHO no testing feedback should be required to apply this patch, because it makes sense the original code is bad.
StacktraceTop: strcmp () at ../sysdeps/ x86_64/ strcmp. S:29 mail_new_ notify (
IA__g_str_equal (v1=0x0, v2=0x7f8b6b72a2f2)
org_gnome_
epl_invoke (ep=0x1c3cde0,
e_event_emit (emp=0x1c3d2c0,