Skip to content

Conversation

devnexen
Copy link
Member

Not such as fix but taking more precautions.
Indeed, the arc4random has two little flaws in this platform,
one already caught upfront by the extension (ie size 0), also
internal use of ccrng_generate which can silently fail in few rare
cases.

Not such as fix but taking more precautions.
Indeed, the arc4random has two little flaws in this platform,
one already caught upfront by the extension (ie size 0), also
internal use of ccrng_generate which can silently fail in few rare
cases.
@devnexen devnexen force-pushed the random_macos_update branch from afa6eec to a1597e4 Compare December 25, 2021 06:34
…0.13

releases which need this header in addition.
@kocsismate
Copy link
Member

I've just checked https://opensource.apple.com/source/CommonCrypto/CommonCrypto-60165/include/Private/CommonRandomSPI.h.auto.html and I'm wondering why CCRandomGenerateBytes is marked with API_DEPRECATED_WITH_REPLACEMENT?

@devnexen
Copy link
Member Author

devnexen commented Jun 21, 2022

Means CCRandomGenerateBytes replaces all of these old calls (they have been removed I can see in the header locally only the new api is available).

@TimWolla
Copy link
Member

TimWolla commented Jul 8, 2022

see also: #8094

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants