linker optimisation causes crashes on armel with -Wl,-O1

Bug #512959 reported by Paul Larson
16
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Linaro Binutils
Expired
High
Unassigned
binutils (Ubuntu)
Expired
High
Unassigned
Lucid
Won't Fix
High
Unassigned
gvfs (Ubuntu)
Expired
Low
Unassigned
Lucid
Won't Fix
Low
Unassigned
nautilus (Ubuntu)
Invalid
Low
Unassigned
Lucid
Invalid
Low
Unassigned

Bug Description

Binary package hint: nautilus

Nautilus just restarts continuously, can get the round spinning cursor after logging into GDM, but that's as far as it seems to go.

ProblemType: Crash
Architecture: armel
AssertionMessage: *** stack smashing detected ***: nautilus terminated
Date: Tue Jan 26 14:04:34 2010
DistroRelease: Ubuntu 10.04
EcryptfsInUse: Yes
ExecutablePath: /usr/bin/nautilus
InstallationMedia: Ubuntu 10.04 "Lucid Lynx" - Alpha armel+dove (20100126)
Package: nautilus 1:2.29.2-0ubuntu1
ProcCmdline: nautilus
ProcCwd: /home/plars
ProcEnviron:
 LANG=en_US.utf8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-701.4-hostname
Signal: 6
SourcePackage: nautilus
StacktraceTop:
 raise () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
Title: nautilus assert failure: *** stack smashing detected ***: nautilus terminated
Uname: Linux 2.6.31-701-dove armv7l
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare

Revision history for this message
Paul Larson (pwlars) wrote :
Revision history for this message
Paul Larson (pwlars) wrote :

Marking public even though this hasn't been retraced (yet). I believe the retracer for armel is currently down, but hopefully we will get a trace on it soon.

visibility: private → public
Paul Larson (pwlars)
tags: added: daily
Revision history for this message
Sebastien Bacher (seb128) wrote :

what does the daily tag means there?

Revision history for this message
Paul Larson (pwlars) wrote :

Process thing that our team uses to tag bugs that were found in the course of testing daily snapshots

Revision history for this message
Sebastien Bacher (seb128) wrote :

Could you try to get a valgrind log of the issue? What image are you testing? Is that on real hardware or virtual boxes?

Changed in nautilus (Ubuntu):
importance: Undecided → Medium
status: New → Incomplete
Revision history for this message
Sebastien Bacher (seb128) wrote :

setting to incomplete until we get details on the crash

Revision history for this message
Paul Larson (pwlars) wrote :

This is real hardware, and unfortunately we don't have valgrind for armel

Revision history for this message
Paul Larson (pwlars) wrote :

This is blocking the use of X on both imx51, and dove at the moment. Would appreciate if we could raise this at least to "high"

Revision history for this message
Oliver Grawert (ogra) wrote :

weird, i dont see it on imx51

Revision history for this message
Sebastien Bacher (seb128) wrote :

sorry but the bug has virtually no useful detail and you are the only one having the issue, any change the box you are using could have issues? nautilus crash should not break desktop login. you can also try moving the nautilus.desktop entry to see if GNOME starts without this one...

Revision history for this message
Oliver Grawert (ogra) wrote :

ok, after a dist-upgrade i can confirm it.
during upgrade i got the packageset from http://paste.ubuntu.com/363941/

(not including nautilus though)
i'll try to downgrade package by package and see when it goes away

Revision history for this message
Paul Larson (pwlars) wrote :

after downgrading the gvfs packages to 1.5.1-0ubuntu2, I am able to get past this

Revision history for this message
Paul Larson (pwlars) wrote :

more specifically libgvfscommon0 seems to be the culprit

Revision history for this message
Paul Larson (pwlars) wrote :

With libgvfscommon0 1.5.1-0ubuntu2 installed, it works. However if I upgrade to 1.5.2-0ubuntu1, that's when I start to see the problem with nautilus.

Revision history for this message
Alexander Sack (asac) wrote :

seems to be caused by gvfs update. setting that to high

Changed in gvfs (Ubuntu):
status: New → Confirmed
importance: Undecided → High
Revision history for this message
Alexander Sack (asac) wrote :
Revision history for this message
Paul Larson (pwlars) wrote :

didn't have much luck getting a useful backtrace, even with all -dbgsym I could think of.

Alexander Sack (asac)
Changed in gvfs (Ubuntu Lucid):
milestone: none → lucid-alpha-3
Changed in nautilus (Ubuntu Lucid):
milestone: none → lucid-alpha-3
Revision history for this message
Paul Larson (pwlars) wrote :

I spent quite a bit of time on Friday trying to bisect mainline gvfs to see if I could determine where this broke. At the end of the exercise I ended up with current 1.5.2 upstream + the few patches in the package and I was still unable to reproduce the bug. As a sanity check, I reinstalled the current (known broken) libgvfscommon 1.5.2-0ubuntu1 package and still could not reproduce the problem.

I'm now working from a live image, where I can reproduce the problem again. I tried reverse applying the patch that Alexander posted on this bug, and I can still reproduce the problem. Still investigating.

Revision history for this message
Paul Larson (pwlars) wrote :

Narrowed it down to -O1. If I remove -O1 from the LDFLAGS in debian/rules, then it works (defaults to -O2).

Revision history for this message
Loïc Minier (lool) wrote :

LDFLAGS or CFLAGS? LDFLAGS default to no optimization (-O0) not O2.

Revision history for this message
Paul Larson (pwlars) wrote :

This was all I changed from debian/rules:
-LDFLAGS += -Wl,-z,defs -Wl,-O1 -Wl,--as-needed
+LDFLAGS += -Wl,-z,defs -Wl,--as-needed

I noticed something scroll by saying that it was defaulting to -O2

Revision history for this message
Matthias Klose (doko) wrote : Re: [Bug 512959] Re: nautilus assert failure: *** stack smashing detected ***: nautilus terminated

On 02.02.2010 19:07, Paul Larson wrote:
> This was all I changed from debian/rules:
> -LDFLAGS += -Wl,-z,defs -Wl,-O1 -Wl,--as-needed
> +LDFLAGS += -Wl,-z,defs -Wl,--as-needed
>
> I noticed something scroll by saying that it was defaulting to -O2

that would be surprising, -Wl,-O2 doesn't exist.

Revision history for this message
Loïc Minier (lool) wrote : Re: nautilus assert failure: *** stack smashing detected ***: nautilus terminated

I've uploaded this and am hence lowering importance, but this needs to be filed with binutils upstream.
nautilus (1:2.29.2-0ubuntu2) lucid; urgency=low

  * Only add -Wl,-O1 to LDFLAGS if DEB_HOST_ARCH != armel; thanks Paul Larson;
    LP #512959.

 -- Loïc Minier <email address hidden> Tue, 02 Feb 2010 19:45:44 +0100

Changed in nautilus (Ubuntu Lucid):
status: Incomplete → New
importance: Medium → Low
status: New → Confirmed
Revision history for this message
Paul Larson (pwlars) wrote :

No, I didn't change that in nautilus, it was in gvfs

Revision history for this message
Paul Larson (pwlars) wrote :

Marking the nautilus task invalid since this is really happening in gvfs/binutils

Changed in nautilus (Ubuntu Lucid):
status: Confirmed → Invalid
Revision history for this message
Loïc Minier (lool) wrote :

Revert the nautilus change and redid the change for gvfs; sorry, was focused on the bug title and that matched the nautilus rules, my bad.

Revision history for this message
Apport retracing service (apport) wrote :

StacktraceTop:
 raise () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
 ?? () from /lib/libc.so.6

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote :

StacktraceTop:
 raise () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
 ?? () from /lib/libc.so.6

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote :

StacktraceTop:
 raise () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
 ?? () from /lib/libc.so.6

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
tags: removed: need-armel-retrace
tags: added: iso-testing
Revision history for this message
Martin Pitt (pitti) wrote : Re: nautilus assert failure: *** stack smashing detected ***: nautilus terminated

Bug in armel binutils; keeping gvfs task open to revert the workaround once it gets fixed, but that's not release critical.

Changed in gvfs (Ubuntu Lucid):
milestone: lucid-alpha-3 → none
status: Confirmed → Won't Fix
Changed in gvfs (Ubuntu):
status: Confirmed → Triaged
Changed in gvfs (Ubuntu Lucid):
importance: High → Low
Changed in nautilus (Ubuntu Lucid):
milestone: lucid-alpha-3 → none
Martin Pitt (pitti)
summary: - nautilus assert failure: *** stack smashing detected ***: nautilus
- terminated
+ causes crashes on armel with -Wl,-O1
tags: added: patch
tags: removed: patch
Changed in gvfs (Ubuntu):
importance: High → Low
Revision history for this message
Alexander Sack (asac) wrote : Re: causes crashes on armel with -Wl,-O1

we should look if there is a simple way to reproduce it.

Changed in binutils (Ubuntu Lucid):
importance: Undecided → High
status: New → Triaged
Matthias Klose (doko)
tags: added: toolchain
Revision history for this message
Andrew Stubbs (ams-codesourcery) wrote :

High priority for investigation. Possibly change importance once understood.

Changed in binutils-linaro:
importance: Undecided → High
Michael Hope (michaelh1)
summary: - causes crashes on armel with -Wl,-O1
+ linker optimisation causes crashes on armel with -Wl,-O1
Revision history for this message
Tobin Davis (gruemaster) wrote :

Is this bug still valid? If not, it will close automatically.

Changed in binutils (Ubuntu Lucid):
status: Triaged → Won't Fix
Changed in gvfs (Ubuntu):
status: Triaged → Incomplete
Changed in binutils (Ubuntu):
status: Triaged → Incomplete
Changed in binutils-linaro:
status: New → Incomplete
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for Linaro Binutils because there has been no activity for 60 days.]

Changed in binutils-linaro:
status: Incomplete → Expired
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for binutils (Ubuntu) because there has been no activity for 60 days.]

Changed in binutils (Ubuntu):
status: Incomplete → Expired
Revision history for this message
Launchpad Janitor (janitor) wrote :

[Expired for gvfs (Ubuntu) because there has been no activity for 60 days.]

Changed in gvfs (Ubuntu):
status: Incomplete → Expired
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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