View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0006187 | Taler | exchange | public | 2020-04-17 14:42 | 2021-09-02 16:14 |
| Reporter | oec | Assigned To | Christian Grothoff | 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 | tweak | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Product Version | git (master) | ||||
| Target Version | 0.7.1 | Fixed in Version | 0.7.1 | ||
| Summary | 0006187: Redundant copy of dki->issue.properties.fee_deposit | ||||
| Description | In exchange/taler-exchange-httpd_deposit.c: 428 TALER_amount_ntoh (&deposit.deposit_fee, 429 &dki->issue.properties.fee_deposit); 430 if (GNUNET_YES != 431 TALER_amount_cmp_currency (&deposit.amount_with_fee, 432 &deposit.deposit_fee) ) ... 455 TALER_amount_ntoh (&deposit.deposit_fee, 456 &dki->issue.properties.fee_deposit); The copies in line 428-429 and 455-456 are redundant. For better readability I suggest to remove of the copy in 428-429 and compare directly against &dki->issue.properties.fee_deposit in 431-432. | ||||
| Tags | No tags attached. | ||||
| The suggested fix doesn't work, as the cmp_currency function only works for the 'host byte order' version of 'account'. So we'd get a type error in that case. Anyway, I'll remove the redundant initialization in lines 455/456, nice catch. | |
| Fixed in 04d4922d..7a46b3fb | |
| Fix committed to master branch. | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2020-04-17 14:42 | oec | New Issue | |
| 2020-04-17 14:42 | oec | Status | new => assigned |
| 2020-04-17 14:42 | oec | Assigned To | => Christian Grothoff |
| 2020-04-17 16:14 | Christian Grothoff | Severity | minor => tweak |
| 2020-04-17 16:14 | Christian Grothoff | Product Version | => git (master) |
| 2020-04-17 16:19 | Christian Grothoff | Note Added: 0015680 | |
| 2020-04-17 16:20 | Christian Grothoff | Status | assigned => informing |
| 2020-04-17 16:20 | Christian Grothoff | Resolution | open => fixed |
| 2020-04-17 16:20 | Christian Grothoff | Fixed in Version | => 0.7.1 |
| 2020-04-17 16:20 | Christian Grothoff | Note Added: 0015681 | |
| 2020-04-17 16:20 | Christian Grothoff | Target Version | => 0.7.1 |
| 2021-08-24 14:23 | Christian Grothoff | Status | informing => closed |
| 2021-09-02 16:13 | Christian Grothoff | ||
| 2021-09-02 16:14 | Christian Grothoff | Note Added: 0018264 |