pixmap crashes on startup in strlen()

Bug #248382 reported by Coen Bijlsma
6
Affects Status Importance Assigned to Milestone
pixmap (Debian)
Fix Released
Unknown
pixmap (Ubuntu)
Fix Released
Undecided
Unassigned
Hardy
Fix Released
Undecided
Unassigned
Intrepid
Fix Released
Undecided
Unassigned

Bug Description

Binary package hint: pixmap

Hi,

I just installed pixmap on my computer and it crashes on startup (SIGSEV).
I am using Ubuntu 8.04.1 and the package version is as follows:
pixmap:
  Installed: 2.6pl4-14.2
  Candidate: 2.6pl4-14.2
  Version table:
 *** 2.6pl4-14.2 0
        500 http://nl.archive.ubuntu.com hardy/universe Packages
        100 /var/lib/dpkg/status

The full gdb stack I got was (I also attached valgrind logs):
Command name abbreviations are allowed if unambiguous.
(gdb) run
Starting program: /usr/bin/pixmap
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)
(no debugging symbols found)

Program received signal SIGSEGV, Segmentation fault.
0x00007f15dc678060 in strlen () from /lib/libc.so.6

Revision history for this message
Coen Bijlsma (coenbijlsma) wrote :
Changed in pixmap:
assignee: nobody → norsetto
status: New → In Progress
Changed in pixmap:
status: Unknown → New
Changed in pixmap:
assignee: norsetto → nobody
status: In Progress → Fix Committed
status: New → Confirmed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package pixmap - 2.6pl4-17ubuntu1

---------------
pixmap (2.6pl4-17ubuntu1) intrepid; urgency=low

  * Fix sigsegv due to getenv not being prototyped correctly (LP: #248382):
    - add stdlib.h header file to PixEdit.c
  * Modify Maintainer value to match the DebianMaintainerField
    specification.

 -- Cesare Tirabassi <email address hidden> Mon, 14 Jul 2008 23:31:32 +0200

Changed in pixmap:
status: Fix Committed → Fix Released
Revision history for this message
Cesare Tirabassi (norsetto) wrote :

motu-sru is subscribed to give clearance for uploading the fix to hardy-proposed:

1. The pixmap package sigsegv in hardy on amd64 arches.

2. The bug has been fixed in the release version (overriding the previous hack).

3. A minimal patch applicable to the stable version of the package is attached.

4. TEST CASE:
A simple test case is defined here. **You must have an amd64 arch for this to make sense***

4.1 In Hardy, install the pixmap (2.6pl4-14.2) package with your preferred package manager.
4.2 Launch the application (for instance with pixmap from the command line). The application will crash.
4.3 Enable the hardy-proposed repository. For instance, you can do this by adding the following line to /etc/apt/sources.list:
  deb http://archive.ubuntu.com/ubuntu/ hardy-proposed universe
Update your local cache and upgrade pixmap.
4.4 Launch the application again. This time it should start with no problems.
4.5 Disable the hardy-proposed repository.

5. The fix is to add a standard header file, no regressions are expected.

WARNING: Any user willing to help with testing is more than welcome. Please be aware that there might be some time before the updated packages will be available in your mirror. The sru needs to be approved and the package manually copied by an archive admin, this may take from 1 to several days to be processed.

Revision history for this message
Luca Falavigna (dktrkranz) wrote :

ACK from motu-sru.

Revision history for this message
Martin Pitt (pitti) wrote :

Accepted into -proposed, please test and give feedback here. Please see https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in pixmap:
status: Confirmed → Fix Committed
Revision history for this message
Vadim Peretokin (vperetokin) wrote :

I can confirm that this worked on Ubuntu 8.04.1, 64bit. Here's a log, with me enabling -proposed, and upgrading the application and it then running OK:

vadi@ubuntu-laptop:~$ pixmap
Segmentation fault
vadi@ubuntu-laptop:~$ pixmap
Warning: Cannot convert string "Term" to type Pixmap
Warning: Cannot convert string "Excl" to type Pixmap
vadi@ubuntu-laptop:~$

Revision history for this message
Albert Damen (albrt) wrote :

+1
- using AMD64
- pixmap from hardy release segfaults at start
- pixmap from hardy-proposed starts ok
- I was able to make, save and re-open a simple drawing.

Revision history for this message
Cesare Tirabassi (norsetto) wrote :

Thanks all for testing. We have the required quorum (including me) so I'm setting this as verification-done.

Revision history for this message
Martin Pitt (pitti) wrote :

Copied to hardy-updates.

Changed in pixmap:
status: Fix Committed → Fix Released
Changed in pixmap (Debian):
status: New → Fix Released
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.