View Issue Details

IDProjectCategoryView StatusLast Update
0006155GNUnetutil librarypublic2020-07-09 07:17
Reporterfefe Assigned ToChristian Grothoff Due Date
PrioritynormalSeverityminorReproducibilityN/A
Status closedResolutionfixed 
Product VersionGit master 
Target Version0.13.0Fixed in Version0.13.0 
Summary0006155: Integer overflow in GNUNET_STRINGS_pp2s
Description  99 char *
 100 GNUNET_STRINGS_pp2s (const struct GNUNET_PeerIdentity *pids,
 101 unsigned int num_pids)
 102 {
 103 char *buf;
 104 size_t off;
 105 size_t plen = num_pids * 5 + 1;
 106
 107 off = 0;
 108 buf = GNUNET_malloc (plen);

The arithmetic on line 105 can overflow. The integer promotion from 32-bit int to 64-bit size_t will only happen at the assignment, after the multiplication has already overflown.
TagsNo tags attached.

Activities

Christian Grothoff

2020-04-03 16:50

manager   ~0015511

Ok, so you are doing an audit of libgnunetutil now. Because that code I am 100% sure is NOT reachable for Taler ;-).

Christian Grothoff

2020-04-03 16:50

manager   ~0015512

Fixed in bc01117ce..6fd24eef2. Let me know if giving you Git commit on gnunet.git would be more productive for these kinds of bugs.

schanzen

2020-07-09 07:17

administrator   ~0016432

0.13.0 released

Issue History

Date ModifiedUsernameFieldChange
2020-04-03 10:25fefeNew Issue
2020-04-03 16:50Christian GrothoffNote Added: 0015511
2020-04-03 16:50Christian GrothoffNote Added: 0015512
2020-04-03 16:51Christian GrothoffAssigned To => Christian Grothoff
2020-04-03 16:51Christian GrothoffStatusnew => resolved
2020-04-03 16:51Christian GrothoffResolutionopen => fixed
2020-04-03 16:51Christian GrothoffFixed in Version => 0.12.2
2020-04-03 16:51Christian GrothoffTarget Version => 0.12.2
2020-04-23 08:45schanzenFixed in Version0.12.2 => 0.13.0
2020-04-23 08:47schanzenTarget Version0.12.2 => 0.13.0
2020-05-31 22:49AdminknoxIssue cloned: 0006320
2020-05-31 22:52AdminknoxIssue cloned: 0006352
2020-07-09 07:17schanzenNote Added: 0016432
2020-07-09 07:17schanzenStatusresolved => closed