View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0003743 | GNUnet | datastore service | public | 2015-03-28 21:05 | 2018-06-06 22:24 |
| Reporter | amatus | Assigned To | amatus | Due Date | |
| Priority | high | Severity | tweak | Reproducibility | N/A |
| Status | closed | Resolution | fixed | ||
| Product Version | Git master | ||||
| Fixed in Version | 0.11.0pre66 | ||||
| Summary | 0003743: plugin API get_key inconsistancy | ||||
| Description | In gnunet_datastore_plugin.h the key parameter to the get_key function is documented: @param key key to match, never NULL However GNUNET_DATASTORE_get_key allows key to be NULL: @param key maybe NULL (to match all entries) Which causes the GetMessage to not include a key which causes gnunet-datastore-service to call get_key with a NULL key. The mysql, postgres, and sqlite plugins assume key can never be NULL: GNUNET_assert (key != NULL); But the heap plugin allows key to be NULL. The only code which calls GNUNET_DATASTORE_get_key with a NULL key is gnunet-datastore.c which tries to copy the contents of one datastore to another. | ||||
| Tags | No tags attached. | ||||
| Well, so the answer is clear: we should allow key=NULL and make it iterate over all values. | |
| Fixed in 2dde0202c5590eeb051c1346f2b66293d83b87ce | |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2015-03-28 21:05 | amatus | New Issue | |
| 2015-03-28 23:27 | Christian Grothoff | Note Added: 0009057 | |
| 2015-03-28 23:28 | Christian Grothoff | Priority | normal => high |
| 2015-03-28 23:28 | Christian Grothoff | Status | new => confirmed |
| 2015-05-31 19:47 | Christian Grothoff | Severity | minor => tweak |
| 2017-03-19 21:34 | amatus | Assigned To | => amatus |
| 2017-03-19 21:34 | amatus | Status | confirmed => assigned |
| 2017-03-19 22:42 | amatus | Note Added: 0011954 | |
| 2017-03-19 22:43 | amatus | Status | assigned => resolved |
| 2017-03-19 22:43 | amatus | Resolution | open => fixed |
| 2017-03-19 22:43 | amatus | Fixed in Version | => 0.11.0pre66 |
| 2018-06-06 22:24 | Christian Grothoff | Status | resolved => closed |