View Issue Details

IDProjectCategoryView StatusLast Update
0008106Talerexchangepublic2024-03-07 19:47
Reporterfefe Assigned ToChristian Grothoff Resolution deadline
ImplementerReviewer 
Release signed byDeployed by 
VerifierCommunicator 
Data protection authority notification requiredNo  
CSIRT notification requiredNo  
Estimated effort (hours)Actual effort (hours)0 
Fast-trackNo
Work packagenone
PrioritylowSeveritytrivialReproducibilityhave not tried
Status closedResolutionfixed 
Product Versiongit (master) 
Target Version0.9.4Fixed in Version0.9.4 
Summary0008106: denoms_cmp: comment is wrong
DescriptionThis is in exchange/src/lib/exchange_api_handle.c:

 715 /**
 716 * Compare two denomination keys. Ignores revocation data.
 717 *
 718 * @param denom1 first denomination key
 719 * @param denom2 second denomination key
 720 * @return 0 if the two keys are equal (not necessarily
 721 * the same object), 1 otherwise.
 722 */

The function can also return any other value.

 744 return GNUNET_memcmp (&tmp1,
 745 &tmp2);

... which calls memcmp. memcmp can return arbitrary values.
If you only want to test for equality, the comment should state "nonzero otherwise"
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Relationships

child of 0008112 implementingoec Merchant security review 

Activities

Christian Grothoff

2024-01-18 21:25

manager   ~0020909

Fixed in dbf84c51..53e679c9

Issue History

Date ModifiedUsernameFieldChange
2024-01-18 13:57fefeNew Issue
2024-01-18 13:57fefeStatusnew => assigned
2024-01-18 13:57fefeAssigned To => Christian Grothoff
2024-01-18 21:25Christian GrothoffNote Added: 0020909
2024-01-18 21:25Christian GrothoffStatusassigned => informing
2024-01-18 21:25Christian GrothoffResolutionopen => fixed
2024-01-18 21:25Christian GrothoffFixed in Version => 0.9.4
2024-01-18 21:25Christian GrothoffProduct Version => git (master)
2024-01-18 21:25Christian GrothoffTarget Version => 0.9.4
2024-01-18 22:31Christian GrothoffRelationship addedchild of 0008112
2024-03-07 19:47Christian GrothoffStatusinforming => closed