[riot-notifications] [RIOT-OS/RIOT] random: generate random seed at compile-time (#13255)
notifications at github.com
Tue May 26 17:58:40 CEST 2020
> Tbh this feature looks like a huge foot gun.
It will provide randomness during development, but once deployed all boards will behave the same.
> But making it default gives a false sense of randomness.
So does [setting it to `0x1`](https://github.com/RIOT-OS/RIOT/blob/master/sys/include/random.h#L42). Additionally, there is a [compile time warning](https://github.com/RIOT-OS/RIOT/pull/13255/files#diff-437ac60cf858166a11a50640fe808948R34) now. Varying seeds between builds and the warning improve the current situation. It is obvious that this mode should not be used for a real deployment.
> instantiable randomness (seeded by CPU_ID or application) that can be used to generate 'stable' (across reboots) IDs or application behavior.
CPUIDs are not always in place (very bad seeds anyway). Reseeding can be done by the application already, however, PRNGs need to run before your application code begins.
> Could we compromise on making this non-default?
Like, introducing a pseudomodule as suggested by @benpicco? I wouldn't mind. As indicated before, this mode should only be used for debugging and development anyway...
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications