tree bfc183646276fceac80d279c7911f5cfabebe6b0
parent 84fd65724027a360207d3c294cc5af35a071ade0
author Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> 1591260209 +0200
committer Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com> 1592297556 +0200

Skip redundant checks for NULL f_rng

Unless MBEDTLS_ECP_NO_INTERNAL_RNG is defined, it's no longer possible for
f_rng to be NULL at the places that randomize coordinates.

Eliminate the NULL check in this case:
- it makes it clearer to reviewers that randomization always happens (unless
  the user opted out at compile time)
- a NULL check in a place where it's easy to prove the value is never NULL
  might upset or confuse static analyzers (including humans)
- removing the check saves a bit of code size

Signed-off-by: Manuel Pégourié-Gonnard <manuel.pegourie-gonnard@arm.com>