commit d3a9edd: [Minor] Another asan fix for rdrand

Vsevolod Stakhov vsevolod at highsecure.ru
Wed Jun 26 17:42:05 UTC 2019


Author: Vsevolod Stakhov
Date: 2019-06-26 18:09:23 +0100
URL: https://github.com/rspamd/rspamd/commit/d3a9edd737c7cb1bcfac16a7c9eef1bc7af9a353

[Minor] Another asan fix for rdrand

---
 contrib/libottery/ottery_entropy_rdrand.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/contrib/libottery/ottery_entropy_rdrand.c b/contrib/libottery/ottery_entropy_rdrand.c
index 36ae60b83..4f227d31f 100644
--- a/contrib/libottery/ottery_entropy_rdrand.c
+++ b/contrib/libottery/ottery_entropy_rdrand.c
@@ -22,7 +22,7 @@
 extern int ottery_valgrind_;
 /** Helper: invoke the RDRAND instruction to get 4 random bytes in the output
  * value. Return 1 on success, and 0 on failure. */
-#define rdrand32(x) ({ unsigned char err = 0; asm volatile(".byte 0x0f; .byte 0xc7; .byte 0xf0; setc %1":"=a"(*x), "=qm"(err)); err; })
+#define rdrand32(x) ({ unsigned char err = 0; __asm volatile(".byte 0x0f; .byte 0xc7; .byte 0xf0; setc %1":"=a"(x), "=qm"(err) :"a"(0) :"cc"); err; })
 
 /** Generate bytes using the Intel RDRAND instruction. */
 static int
@@ -36,7 +36,7 @@ ottery_get_entropy_rdrand(const struct ottery_entropy_config *cfg,
   if (! (ottery_get_cpu_capabilities_() & OTTERY_CPUCAP_RAND) || ottery_valgrind_)
     return OTTERY_ERR_INIT_STRONG_RNG;
   while (outlen >= 4) {
-    if (rdrand32(&up) != 1)
+    if (rdrand32(up) != 1)
       return OTTERY_ERR_INIT_STRONG_RNG;
     memcpy (out, &up, sizeof (up));
     out += sizeof (up);
@@ -44,7 +44,7 @@ ottery_get_entropy_rdrand(const struct ottery_entropy_config *cfg,
   }
 
   if (outlen) {
-    if (rdrand32(&up) != 1)
+    if (rdrand32(up) != 1)
       return OTTERY_ERR_INIT_STRONG_RNG;
     memcpy(out, &up, outlen);
   }


More information about the Commits mailing list