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

[Th. Sievers]

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
ProcEnviron:
 LANGUAGE=de_DE.UTF-8
 LANG=de_DE.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-13.43-generic
Signal: 6
SourcePackage: icewm
StacktraceTop:
 raise () from /lib/libc.so.6
 abort () from /lib/libc.so.6
 ?? () from /lib/libc.so.6
 __fortify_fail () from /lib/libc.so.6
 __chk_fail () from /lib/libc.so.6
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

[Andreas Leitgeb (avl-logic)]

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...

[H.-Dirk Schmitt (dirk-computer42)]

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 ?

[H.-Dirk Schmitt (dirk-computer42)]

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)]

Here's a patch to fix the bug. I have also sent it upstream: https://sourceforge.net/p/icewm/patches/241/

[Lei Zhang (thestig-google)]

Upload the right patch on this second try.

[Ubuntu Foundations Team Bug Bot (crichton)]

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.]

[Chad Miller (cmiller)]

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)]

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.