grig crashes on mouse click between freq display digits
Bug #517816 reported by
Kamal Mostafa
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| grig (Debian) |
Fix Released
|
Unknown
|
|||
| grig (Ubuntu) |
Fix Released
|
Undecided
|
Iulian Udrea | ||
Bug Description
Binary package hint: grig
grig 0.7.2 Ubuntu Karmic
TESTCASE:
(Reproducible on amd64 but not i386)
1. Run grig
2. Mouse click in the space *between* the "145" and the "000" (not on any digit).
grig crashes with Segmentation fault.
Note also that mouse scroll-wheel-up events don't work (but scroll-wheel-down events do work) to change frequency digits.
Related branches
lp:~kamalmostafa/ubuntu/lucid/grig/fix-517816-crash
- Iulian Udrea: Approve
- Ubuntu branches: Pending requested
-
Diff: 41 lines (+10/-2)3 files modifieddebian/changelog (+7/-0)
debian/control (+2/-1)
src/rig-gui-lcd.c (+1/-1)
| Changed in grig (Ubuntu): | |
| assignee: | nobody → Kamal Mostafa (kamalmostafa) |
| status: | New → In Progress |
Revision history for this message
| Kamal Mostafa (kamalmostafa) wrote | #1 |
| Changed in grig (Ubuntu): | |
| assignee: | Kamal Mostafa (kamalmostafa) → nobody |
| status: | In Progress → Confirmed |
Revision history for this message
| Kamal Mostafa (kamalmostafa) wrote | #2 |
| Changed in grig (Ubuntu): | |
| status: | Confirmed → In Progress |
| assignee: | nobody → Iulian Udrea (iulian) |
Revision history for this message
| Iulian Udrea (iulian) wrote | #3 |
| Changed in grig (Ubuntu): | |
| status: | In Progress → Fix Committed |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in grig (Ubuntu): | |
| status: | Fix Committed → Fix Released |
| Changed in grig (Debian): | |
| status: | Unknown → New |
| Changed in grig (Debian): | |
| status: | New → Fix Released |
To post a comment you must log in.
Attached branch fixes the bug, which is caused by an invalid (i >= 0) comparison with variable 'i' declared as unsigned.
Patch forwarded upstream to https:/
Will forward to Debian pending Ubuntu merge.