➜
This issue tracker has been migrated to GitHub,
and is currently read-only.
For more information,
see the GitHub FAQs in the Python's Developer Guide.
Created on 2008-07-16 22:59 by kristjan.jonsson, last changed 2022-04-11 14:56 by admin. This issue is now closed.
| Files | ||||
|---|---|---|---|---|
| File name | Uploaded | Description | Edit | |
| tmp6.patch | kristjan.jonsson, 2008-07-16 22:59 | patch for issue | ||
| Messages (2) | |||
|---|---|---|---|
| msg69853 - (view) | Author: Kristján Valur Jónsson (kristjan.jonsson) *  |
Date: 2008-07-16 22:59 | |
The CryptGenRandomBytes uses whatever data is already in the buffer as seed for the output. So, the buffer is effectively an in/out buffer. Now, since we are generating random data anyway, the fact that we are using an undefined seed for the data shouldn't matter. However, this does create a bunch of false positives for analysis tools such as Purify, that track the copying and usage of uninitialized data. An easy patch is to clear the buffer before submitting it to CryptGenRandomBytes, and is attached. |
|||
| msg70118 - (view) | Author: Amaury Forgeot d'Arc (amaury.forgeotdarc) * |
Date: 2008-07-21 21:14 | |
Committed as r65174 and r65175. (for trunk, I had to change PyBytes_AS_STRING into PyString_AS_STRING) Thanks! |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2022-04-11 14:56:36 | admin | set | github: 47637 |
| 2008-07-21 21:14:57 | amaury.forgeotdarc | set | status: open -> closed keywords: patch, patch, easy resolution: fixed messages: + msg70118 |
| 2008-07-21 13:04:25 | amaury.forgeotdarc | set | keywords:
patch, patch, easy assignee: amaury.forgeotdarc nosy: + amaury.forgeotdarc |
| 2008-07-16 22:59:42 | kristjan.jonsson | create | |