gcalctool crashed with SIGSEGV in math_equation_insert()

Bug #651455 reported by ojab
24
This bug affects 2 people
Affects Status Importance Assigned to Milestone
GCalctool
Fix Released
Critical
gcalctool (Ubuntu)
Fix Released
Medium
Unassigned
Maverick
Fix Released
Undecided
Unassigned

Bug Description

Impact: Calculator crashes when pasting and nothing in paste buffer
Resolution: Upstream code change (patch in comments below)
TEST CASE
1. Open calculator
2. Place the cursor in the test area
3. Ensure that the text area is empty
4. Press CTRL+C then press Ctrl+V
5. The calculator must not crash.

Risk: Low, simple a null pointer check

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

StacktraceTop:
 math_equation_insert (equation=0x1df5780, text=0x0) at math-equation.c:892
 request_text_received_func (clipboard=0x209c6a0,
 selection_received (widget=0x1fdd260,
 g_closure_invoke ()
 ?? () from /usr/lib/libgobject-2.0.so.0

Revision history for this message
Apport retracing service (apport) wrote : Stacktrace.txt
Revision history for this message
Apport retracing service (apport) wrote : ThreadStacktrace.txt
Changed in gcalctool (Ubuntu):
importance: Undecided → Medium
tags: removed: need-amd64-retrace
visibility: private → public
Revision history for this message
Pedro Villavicencio (pedro) wrote :

this is known and fixed upstream as of today: https://bugzilla.gnome.org/show_bug.cgi?id=629381

Changed in gcalctool (Ubuntu):
status: New → Fix Committed
Changed in gcalctool:
importance: Unknown → Critical
status: Unknown → Fix Released
Revision history for this message
Robert Ancell (robert-ancell) wrote :

Fixes paste when buffer empty

description: updated
Revision history for this message
Martin Pitt (pitti) wrote : Please test proposed package

Accepted gcalctool into maverick-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 gcalctool (Ubuntu Maverick):
status: New → Fix Committed
tags: added: verification-needed
Revision history for this message
Ara Pulido (ara) wrote :

Tested with gcalctool (5.32.0-0ubuntu2)
Pasting an empty buffer no longer makes the application crash.

Also, I run the mago regression tests on it, and they all passed.

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

Accepted gcalctool into maverick-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!

tags: removed: verification-done
tags: added: verification-needed
Revision history for this message
Jean-Baptiste Lallement (jibel) wrote :

SRU verification for Maverick:
I have reproduced the problem with gcalctool 5.32.0-0ubuntu1 in maverick and have verified that the version of gcalctool 5.32.0-0ubuntu3 in -proposed fixes the issue.

Marking as verification-done

tags: added: verification-done
removed: verification-needed
description: updated
Greg A (etulfetulf)
Changed in gcalctool (Ubuntu Maverick):
status: Fix Committed → Fix Released
Changed in gcalctool (Ubuntu):
status: Fix Committed → Fix Released
tags: added: testcase
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Duplicates of this bug

Other bug subscribers

Remote bug watches

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