View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0009637 | GNUnet | ARM service | public | 2025-03-18 13:51 | 2025-04-11 12:43 |
| Reporter | fefe | Assigned To | Christian Grothoff | Due Date | |
| Priority | normal | Severity | minor | Reproducibility | unable to reproduce |
| Status | closed | Resolution | fixed | ||
| Product Version | Git master | ||||
| Target Version | 0.24.1 | Fixed in Version | 0.24.1 | ||
| Summary | 0009637: process_refunds_cb: can prd->cr_head be NULL? | ||||
| Description | The case is handled by the for loop: 503 for (cr = prd->cr_head; 504 NULL != cr; 505 cr = cr->next) 506 if (cr->refund_serial == refund_serial) 507 return; but further down we just dereference cr_head->prev: 523 GNUNET_CONTAINER_DLL_insert (prd->cr_head, 524 prd->cr_tail, 525 cr); | ||||
| Tags | No tags attached. | ||||
| Nope, we don't. You're reading the macro wrong. It checks if cr_head is NULL. Our DLLs always work like that, basically head/tail start out as NULL, that's totally normal for that macro. | |
| I analyzed the issue a bit more. This confused the static analyzer because it does not understand that head and tail are NULL at the same time. You check whether tail is NULL and then dereference head->prev. Adding an assertion that either both or none of head and tail are NULL would fix this. | |
| Fixed in GNUnet via 611583c70..db8b6c7a5. | |
| 0.24.1 | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2025-03-18 13:51 | fefe | New Issue | |
| 2025-03-18 13:51 | fefe | Status | new => assigned |
| 2025-03-18 13:51 | fefe | Assigned To | => Christian Grothoff |
| 2025-03-19 01:11 | Christian Grothoff | Note Added: 0024241 | |
| 2025-03-19 01:11 | Christian Grothoff | Status | assigned => resolved |
| 2025-03-19 01:11 | Christian Grothoff | Resolution | open => no change required |
| 2025-03-19 01:11 | Christian Grothoff | Fixed in Version | => 1.0 |
| 2025-03-19 01:11 | Christian Grothoff | Reproducibility | have not tried => unable to reproduce |
| 2025-03-19 01:11 | Christian Grothoff | Product Version | => git (master) |
| 2025-03-19 01:11 | Christian Grothoff | Target Version | => 1.0 |
| 2025-03-19 09:23 | fefe | Status | resolved => feedback |
| 2025-03-19 09:23 | fefe | Resolution | no change required => reopened |
| 2025-03-19 09:23 | fefe | Note Added: 0024242 | |
| 2025-03-19 11:25 | Christian Grothoff | Note Added: 0024243 | |
| 2025-03-19 11:25 | Christian Grothoff | Status | feedback => resolved |
| 2025-03-19 11:25 | Christian Grothoff | Resolution | reopened => fixed |
| 2025-03-19 11:25 | Christian Grothoff | Project | Taler => GNUnet |
| 2025-03-19 11:25 | Christian Grothoff | Category | mechant backend => General |
| 2025-03-19 11:26 | Christian Grothoff | Category | General => ARM service |
| 2025-03-19 11:26 | Christian Grothoff | Product Version | git (master) => Git master |
| 2025-03-19 11:26 | Christian Grothoff | Fixed in Version | 1.0 => 0.25.0 |
| 2025-03-19 11:26 | Christian Grothoff | Target Version | 1.0 => 0.25.0 |
| 2025-03-20 18:43 | schanzen | Target Version | 0.25.0 => 0.24.1 |
| 2025-04-11 12:32 | schanzen | Fixed in Version | 0.25.0 => 0.24.1 |
| 2025-04-11 12:43 | schanzen | Note Added: 0024510 | |
| 2025-04-11 12:43 | schanzen | Status | resolved => closed |