Don't use int constants with a long data type.
Bug #625798 reported by
Michael Bienia
This bug affects 2 people
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Tickcount |
Triaged
|
High
|
Unassigned | ||
| tickcount (Ubuntu) |
Fix Released
|
Medium
|
Thierry Carrez | ||
Bug Description
A debug session in #ubuntu-devel initiated by Emmet Hikory for the cause of the FTBFS for powerpc, revealing a but in the tickcount code.
tickcount's C code uses "long" for the variables but uses UINT_MAX for the wrap-around computation. The same applies to the test suite for it, it also uses UINT_MAX. This works for 32bit because sizeof(
Related branches
Revision history for this message
| Michael Bienia (geser) wrote | #1 |
Revision history for this message
| Michael Bienia (geser) wrote | #2 |
Revision history for this message
| Emmet Hikory (persia) wrote | #3 |
| Changed in tickcount (Ubuntu): | |
| status: | New → Triaged |
| importance: | Undecided → Medium |
| Changed in tickcount (Ubuntu): | |
| assignee: | nobody → Thierry Carrez (ttx) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad Janitor (janitor) wrote | #4 |
| Changed in tickcount (Ubuntu): | |
| status: | In Progress → Fix Released |
| Changed in tickcount: | |
| status: | New → Triaged |
| importance: | Undecided → High |
| tags: | added: easy |
To post a comment you must log in.
I test-built with the upstream patch (but a slightly different debdiff) against maverick on i386, amd64, armel, and powerpc. In all cases, the package built successfully, including passing all tests.