diff options
| author | Richard Henderson <[email protected]> | 2020-01-10 14:54:21 +0000 |
|---|---|---|
| committer | Theodore Ts'o <[email protected]> | 2020-01-25 12:18:51 -0500 |
| commit | 8dae77ac56fec1e085fca3ecea49abda1b4b73d2 (patch) | |
| tree | 2630f94c64052a93a771135e9c1e25f248e5fed9 | |
| parent | 98dcfce69729f9ce0fb14f96a39bbdba21429597 (diff) | |
powerpc: Mark archrandom.h functions __must_check
We must not use the pointer output without validating the
success of the random read.
Acked-by: Michael Ellerman <[email protected]>
Reviewed-by: Ard Biesheuvel <[email protected]>
Signed-off-by: Richard Henderson <[email protected]>
Signed-off-by: Mark Brown <[email protected]>
Link: https://lore.kernel.org/r/[email protected]
Signed-off-by: Theodore Ts'o <[email protected]>
| -rw-r--r-- | arch/powerpc/include/asm/archrandom.h | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/arch/powerpc/include/asm/archrandom.h b/arch/powerpc/include/asm/archrandom.h index f0f16b4fc5ea..9a53e29680f4 100644 --- a/arch/powerpc/include/asm/archrandom.h +++ b/arch/powerpc/include/asm/archrandom.h @@ -6,17 +6,17 @@ #include <asm/machdep.h> -static inline bool arch_get_random_long(unsigned long *v) +static inline bool __must_check arch_get_random_long(unsigned long *v) { return false; } -static inline bool arch_get_random_int(unsigned int *v) +static inline bool __must_check arch_get_random_int(unsigned int *v) { return false; } -static inline bool arch_get_random_seed_long(unsigned long *v) +static inline bool __must_check arch_get_random_seed_long(unsigned long *v) { if (ppc_md.get_random_seed) return ppc_md.get_random_seed(v); @@ -24,7 +24,7 @@ static inline bool arch_get_random_seed_long(unsigned long *v) return false; } -static inline bool arch_get_random_seed_int(unsigned int *v) +static inline bool __must_check arch_get_random_seed_int(unsigned int *v) { unsigned long val; bool rc; |