eggdrop crash on i386

Bug #885329 reported by Stefan Lithén
32
This bug affects 5 people
Affects Status Importance Assigned to Milestone
eggdrop (Debian)
Fix Released
Unknown
eggdrop (Ubuntu)
Fix Released
Medium
Vibhav Pant
Oneiric
Won't Fix
Medium
Unassigned
Precise
Fix Released
Undecided
Vibhav Pant

Bug Description

[Impact]
Eggdrop crashes when trying to start on a i386 machine
[Development Fix]
=== modified file 'src/tclhash.c'
--- src/tclhash.c 2008-08-18 00:26:01 +0000
+++ src/tclhash.c 2011-11-06 05:20:30 +0000
@@ -1202,7 +1202,7 @@
for (i = 0; cc[i].name; i++) {
egg_snprintf(p, sizeof p, "*%s:%s", tl->name,
cc[i].funcname ? cc[i].funcname : cc[i].name);
- l = nmalloc(Tcl_ScanElement(p, &k));
+ l = nmalloc(Tcl_ScanElement(p, &k) + 1);
Tcl_ConvertElement(p, l, k | TCL_DONT_USE_BRACES);
table[0].cdata = (void *) cc[i].func;
add_cd_tcl_cmds(table);
@@ -1220,7 +1220,7 @@
for (i = 0; cc[i].name; i++) {
egg_snprintf(p, sizeof p, "*%s:%s", table->name,
cc[i].funcname ? cc[i].funcname : cc[i].name);
- l = nmalloc(Tcl_ScanElement(p, &k));
+ l = nmalloc(Tcl_ScanElement(p, &k) + 1);
Tcl_ConvertElement(p, l, k | TCL_DONT_USE_BRACES);
Tcl_DeleteCommand(interp, p);
unbind_bind_entry(table, cc[i].flags, cc[i].name, l);

[Stable Fix]
Same as the development fix

[Test Case]
Install eggdrop on an i386 machine
Launch eggdrop from the terminal
[Regression Potential]

[Original Report]Eggdrop crashes when trying to start on a i386 machine after update from ubuntu 11.04 to ubuntu 11.10
It works on my other updated amd64 machine, but it also crashes on my updated virtual i386 machine (virtualbox).

Here is output from when trying to start the eggdrop: http://pastebin.com/0cV4AYm9

ProblemType: Bug
DistroRelease: Ubuntu 11.10
Package: eggdrop 1.6.19-1.2ubuntu2
ProcVersionSignature: Ubuntu 3.0.0-12.20-generic 3.0.4
Uname: Linux 3.0.0-12-generic i686
ApportVersion: 1.23-0ubuntu4
Architecture: i386
Date: Wed Nov 2 17:45:55 2011
EcryptfsInUse: Yes
ExecutablePath: /usr/bin/eggdrop
InstallationMedia: Xubuntu 11.10 "Oneiric Ocelot" - Release i386 (20111012)
SourcePackage: eggdrop
UpgradeStatus: No upgrade log present (probably fresh install)

Related branches

Revision history for this message
Stefan Lithén (spacebug-se-deactivatedaccount) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in eggdrop (Ubuntu):
status: New → Confirmed
Revision history for this message
Paul Sladen (sladen) wrote :

To reproduce:

  1. Do 'sudo apt-get install eggdrop'
  2.Run 'eggdrop'

Revision history for this message
Daniel Holbach (dholbach) wrote :

This seems to be fixed in Debian - is somebody looking into working on a merge?

Revision history for this message
Ben Kietzman (ben-kietzman) wrote :

This is happening on my server as well.

Changed in eggdrop (Debian):
status: Unknown → Fix Released
Revision history for this message
Sami Nieminen (sami-nieminen) wrote :

I am experiencing the same crash. Is this being worked on?

Changed in eggdrop (Ubuntu Oneiric):
status: New → Confirmed
Vibhav Pant (vibhavp)
Changed in eggdrop (Ubuntu):
assignee: nobody → Vibhav Pant (vibhavp)
Vibhav Pant (vibhavp)
Changed in eggdrop (Ubuntu Oneiric):
assignee: nobody → Vibhav Pant (vibhavp)
Bryce Harrington (bryce)
description: updated
Vibhav Pant (vibhavp)
Changed in eggdrop (Ubuntu Oneiric):
assignee: Vibhav Pant (vibhavp) → nobody
description: updated
Revision history for this message
Dave Gilbert (ubuntu-treblig) wrote :

severe impact, non core->medium

Changed in eggdrop (Ubuntu):
importance: Undecided → Medium
status: Confirmed → Triaged
Changed in eggdrop (Ubuntu Oneiric):
importance: Undecided → Medium
status: Confirmed → Triaged
Revision history for this message
Vibhav Pant (vibhavp) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package eggdrop - 1.6.19-1.2ubuntu4

---------------
eggdrop (1.6.19-1.2ubuntu4) quantal; urgency=low

  * debian/patches/01tcl8.5.10.patch (LP: #885329)
    - Fixes startup crash caused by unexpected behaviour in Tcl 8.5.10
 -- Vibhav Pant <email address hidden> Sun, 20 May 2012 19:46:49 +0530

Changed in eggdrop (Ubuntu):
status: Triaged → Fix Released
Julian Taylor (jtaylor)
Changed in eggdrop (Ubuntu Precise):
status: New → Triaged
Revision history for this message
Vibhav Pant (vibhavp) wrote :
Changed in eggdrop (Ubuntu Precise):
assignee: nobody → Vibhav Pant (vibhavp)
Revision history for this message
Chris Halse Rogers (raof) wrote : Please test proposed package

Hello Stefan, or anyone else affected,

Accepted eggdrop into precise-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in eggdrop (Ubuntu Precise):
status: Triaged → Fix Committed
tags: added: verification-needed
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Hello Stefan, or anyone else affected,

Accepted eggdrop into oneiric-proposed. The package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

description: updated
Changed in eggdrop (Ubuntu Oneiric):
status: Triaged → Fix Committed
Revision history for this message
Stefan Lithén (spacebug-se-deactivatedaccount) wrote :

I just tested it now and it works.

Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Stefan, did you test on oneiric or precise? Thanks!

Revision history for this message
Sami Nieminen (sami-nieminen) wrote :

I tested on precise and is working now. Thanks for the fix.

Revision history for this message
Stefan Lithén (spacebug-se-deactivatedaccount) wrote :

precise

Martin Pitt (pitti)
tags: added: verification-done
removed: verification-needed
tags: added: verification-done-precise verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package eggdrop - 1.6.19-1.2ubuntu3.1

---------------
eggdrop (1.6.19-1.2ubuntu3.1) precise-proposed; urgency=low

  [ Vibhav Pant ]
  * debian/patches/01tcl8.5.10.patch (LP: #885329)
    - Fixes startup crash caused by unexpected behaviour in Tcl 8.5.10

  [ Julian Taylor ]
  * fix a bug in gcc-format-string-warnings.patch
    patch wrongly replaced "..%d" with "%s"
 -- Julian Taylor <email address hidden> Sun, 20 May 2012 20:10:09 +0200

Changed in eggdrop (Ubuntu Precise):
status: Fix Committed → Fix Released
Steve Langasek (vorlon)
tags: removed: verification-done verification-done-precise
Revision history for this message
Brian Murray (brian-murray) wrote :

This bug report still needs verification in Oneiric.

Revision history for this message
Brian Murray (brian-murray) wrote : Verification still needed

The fix for this bug has been awaiting testing feedback in the -proposed repository for oneiric for more than 90 days. Please test this fix and update the bug appropriately with the results. In the event that the fix for this bug is still not verified 15 days from now, the package will be removed from the -proposed repository.

tags: added: removal-candidate
Revision history for this message
Brian Murray (brian-murray) wrote :

The version of eggdrop in oneiric-proposed has been removed as this bug report was not verified in a timely fashion.

tags: removed: verification-needed
tags: removed: removal-candidate
Changed in eggdrop (Ubuntu Oneiric):
status: Fix Committed → Triaged
Revision history for this message
Rolf Leggewie (r0lf) wrote :

oneiric has seen the end of its life and is no longer receiving any updates. Marking the oneiric task for this ticket as "Won't Fix".

Changed in eggdrop (Ubuntu Oneiric):
status: Triaged → Won't Fix
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.