View Issue Details

IDProjectCategoryView StatusLast Update
0008089Talerexchangepublic2024-03-07 20:47
Reporterfefe Assigned ToChristian Grothoff  
PrioritynormalSeverityminorReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.9.4Fixed in Version0.9.4 
Summary0008089: reserve_batch_withdraw_ok: index should be size_t
DescriptionThis is in exchange/src/lib/exchange_api_batch_withdraw2.c:

110 unsigned int index;

123 json_array_foreach (ja, index, j)

which expands to:

243 #define json_array_foreach(array, index, value) \
244 for (index = 0; \
245 index < json_array_size(array) && (value = json_array_get(array, index)); \
246 index++)

json_array_size returns a size_t, so index should be a size_t, too.
TagsNo tags attached.

Relationships

child of 0008112 assignedfefe Merchant security review 

Activities

fefe

2024-01-15 16:29

developer   ~0020876

Further down in the function:

137 for (unsigned int i = 0; i<index; i++)
138 TALER_blinded_denom_sig_free (&blind_sigs[i]);

i should be a size_t, too.

Christian Grothoff

2024-01-15 16:44

manager   ~0020877

Fixed in 6c1400044..4cf9e0000

Issue History

Date Modified Username Field Change
2024-01-15 16:27 fefe New Issue
2024-01-15 16:27 fefe Status new => assigned
2024-01-15 16:27 fefe Assigned To => Christian Grothoff
2024-01-15 16:29 fefe Note Added: 0020876
2024-01-15 16:44 Christian Grothoff Status assigned => resolved
2024-01-15 16:44 Christian Grothoff Resolution open => fixed
2024-01-15 16:44 Christian Grothoff Fixed in Version => 0.9.4
2024-01-15 16:44 Christian Grothoff Note Added: 0020877
2024-01-15 16:44 Christian Grothoff Product Version => git (master)
2024-01-15 16:44 Christian Grothoff Target Version => 0.9.4
2024-01-18 23:32 Christian Grothoff Relationship added child of 0008112
2024-03-07 20:47 Christian Grothoff Status resolved => closed