icewm assert failure: *** buffer overflow detected ***: icewm terminated

Bug #447883 reported by Th. Sievers on 2009-10-10
This bug affects 6 people
Affects Status Importance Assigned to Milestone
icewm (Ubuntu)
Chad Miller

Bug Description

Binary package hint: icewm

9.10 beta amd64

ProblemType: Crash
Architecture: amd64
AssertionMessage: *** buffer overflow detected ***: icewm terminated
Date: Sat Oct 10 10:43:03 2009
DistroRelease: Ubuntu 9.10
ExecutablePath: /usr/bin/icewm
Package: icewm 1.2.37+1.3.4pre2-3
ProcCmdline: icewm
ProcVersionSignature: Ubuntu 2.6.31-13.43-generic
Signal: 6
SourcePackage: icewm
 raise () from /lib/
 abort () from /lib/
 ?? () from /lib/
 __fortify_fail () from /lib/
 __chk_fail () from /lib/
Title: icewm assert failure: *** buffer overflow detected ***: icewm terminated
Uname: Linux 2.6.31-13-generic x86_64
UserGroups: adm admin cdrom dialout lpadmin plugdev sambashare

Related branches

Th. Sievers (th-sievers) wrote :
Andreas Leitgeb (avl-logic) wrote :

One thing in advance: icewm did work on my other machine (a Pentium 4), but not on this one (Athlon XP (32bit)

On the Athlon machine, I then tried to compile it from some (older) source-tree. The resulting binary still crashed
(the same way). Then I added CXXOPTS=-g for icewm's configure, and this time the resulting binary didn't crash.

My (uneducated) guess would be, that something with g++ and optimization is perhaps wrong...

I tried to track down the reason and found a workaround in installing the old jaunty icewm-common package.
After some further investigations I found a better workaround in removing the message files from icewm-common (karmic) with:
find /usr/share/locale -name "icewm*" |xargs rm

May you please confirm this workaround ?

Changed in icewm (Ubuntu):
status: New → Confirmed

I have done a fresh lucid setup on a new machine without the workaround noted above.
No problem in using icewm.
I hope the problem is solved in lucid.

Lei Zhang (thestig-google) wrote :

Here's a patch to fix the bug. I have also sent it upstream:

Lei Zhang (thestig-google) wrote :

Upload the right patch on this second try.

The attachment "icewm.patch" seems to be a patch. If it isn't, please remove the "patch" flag from the attachment, remove the "patch" tag, and if you are a member of the ~ubuntu-reviewers, unsubscribe the team.

[This is an automated message performed by a Launchpad user owned by ~brian-murray, for any issues please contact him.]

tags: added: patch
Chad Miller (cmiller) on 2013-10-22
Changed in icewm (Ubuntu):
assignee: nobody → Chad Miller (cmiller)
Chad Miller (cmiller) wrote :

I reviewed Lei Zhang's (#6) patch. It needs "delete [] loadmsg;" at the end of CPUStatus::updateToolTip, but so does the original code.

Lei Zhang (thestig-google) wrote :

Right. Thanks for looking. I can either update the patch, or maybe I can take a second pass later and fix this and any other memory leaks I find. Thoughts on which way to go?

BTW, the latest icewm-devel mailing list discussion seems to be about finding a maintainer for icewm. :(
Until someone actually take ownership of the project and move it forward, it'll be up to the distro package maintainers to apply these patches to their builds.

To post a comment you must log in.
This report contains Public information  Edit
Everyone can see this information.

Duplicates of this bug

Other bug subscribers