View Issue Details

IDProjectCategoryView StatusLast Update
0006193Talerexchangepublic2021-09-02 16:14
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
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Product Version0.7.0 
Target Version0.7.1Fixed in Version0.7.1 
Summary0006193: memset for clearing key material
DescriptionIn exchange/src/lib/exchange_api_refresh_common.c:

 65 /* Finally, clean up a bit...
 66 (NOTE: compilers might optimize this away, so this is
 67 not providing any strong assurances that the key material
 68 is purged.) */
 69 memset (md,
 70 0,
 71 sizeof (struct MeltData));

glibc now has a function called explicit_bzero for this (since version 2.25 from Feb 2017).
TagsNo tags attached.

Required notifications

Address or linkRemarkAdded byNotifiedActions

Activities

Christian Grothoff

2020-04-21 12:49

manager   ~0015723

Ja, ist nur dann eben nicht so portabel. Die groessere Frage fuer mich ist, ob wir key-clearing ueberhaupt machen sollten...

Christian Grothoff

2020-04-21 12:56

manager   ~0015724

GNUNET_CRYPTO_zero_keys() macht bereits explicit_bzero(), sollten wir hier also verwenden.

Christian Grothoff

2020-04-21 12:57

manager   ~0015725

Fixed in c167fd8f..ee0d0dd3

Christian Grothoff

2021-09-02 16:14

manager   ~0018260

Fix committed to master branch.

Issue History

Date ModifiedUsernameFieldChange
2020-04-21 12:38fefeNew Issue
2020-04-21 12:38fefeStatusnew => assigned
2020-04-21 12:38fefeAssigned To => Christian Grothoff
2020-04-21 12:49Christian GrothoffNote Added: 0015723
2020-04-21 12:56Christian GrothoffNote Added: 0015724
2020-04-21 12:57Christian GrothoffNote Added: 0015725
2020-04-21 12:58Christian GrothoffStatusassigned => informing
2020-04-21 12:58Christian GrothoffResolutionopen => fixed
2020-04-21 12:58Christian GrothoffFixed in Version => 0.7.1
2020-04-21 12:58Christian GrothoffTarget Version => 0.7.1
2021-08-24 14:23Christian GrothoffStatusinforming => closed
2021-09-02 16:13Christian Grothoff
2021-09-02 16:14Christian GrothoffNote Added: 0018260