View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0004002 | Taler | wallet (WebExtension) | public | 2015-10-09 11:48 | 2019-12-26 20:37 |
| Reporter | Marcello Stanisci | Assigned To | Marcello Stanisci | Resolution deadline | |
| Implementer | Reviewer | ||||
| Release signed by | Deployed by | ||||
| Verifier | Communicator | ||||
| Data protection authority notification required | No | ||||
| CSIRT notification required | No | ||||
| Estimated effort (hours) | Actual effort (hours) | 0 | |||
| Fast-track | No | ||||
| Work package | none | ||||
| Priority | normal | Severity | minor | Reproducibility | have not tried |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.0 | Fixed in Version | 0.0 | ||
| Summary | 0004002: check over/under flows in multiplication between doubles | ||||
| Description | grep for CHECKOVERUNDERFLOW in wrap.c | ||||
| Tags | No tags attached. | ||||
| See also: https://gcc.gnu.org/onlinedocs/gcc/Integer-Overflow-Builtins.html | |
| addressed in e5aea94..39084f1 The solution in the link above doesn't regard operations on 'double' types. The following solution has been adopted c = a * b if (a<1 || b<1){ if (c >= max (a, b)) // is overflow } else{ if (c < max (a, b)) // is overflow } The solution should catch also underflows, since it simply checks if everything looks normal. Shall this bug be closed? | |
| Determined mathematically that overflow is impossible in this case. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-10-09 11:48 | Marcello Stanisci | New Issue | |
| 2015-10-09 11:48 | Marcello Stanisci | Status | new => assigned |
| 2015-10-09 11:48 | Marcello Stanisci | Assigned To | => Marcello Stanisci |
| 2015-10-22 23:14 | Christian Grothoff | Target Version | 0.2 => 0.1 |
| 2015-11-17 13:18 | Christian Grothoff | Assigned To | Marcello Stanisci => tg |
| 2015-11-17 13:19 | Christian Grothoff | Note Added: 0009912 | |
| 2015-11-17 14:30 | Christian Grothoff | Assigned To | tg => Marcello Stanisci |
| 2015-12-10 13:02 | Marcello Stanisci | Note Added: 0009992 | |
| 2015-12-10 13:02 | Marcello Stanisci | Status | assigned => feedback |
| 2015-12-17 10:47 | Christian Grothoff | Note Added: 0010009 | |
| 2015-12-17 10:47 | Christian Grothoff | Status | feedback => informing |
| 2015-12-17 10:47 | Christian Grothoff | Fixed in Version | => 0.0 |
| 2015-12-17 10:47 | Christian Grothoff | Resolution | open => fixed |
| 2015-12-17 10:48 | Christian Grothoff | Target Version | 0.1 => 0.0 |
| 2015-12-17 11:03 | Christian Grothoff | Status | informing => closed |
| 2019-12-26 20:37 | Florian Dold | Category | wallet (browser-based) => wallet (WebExtensions) |
| 2023-04-13 18:37 | Florian Dold | Category | wallet (WebExtensions) => wallet (WebExtension) |