From 9d4c902c42b3fda368d7ea64bb9575c46228fa7f Mon Sep 17 00:00:00 2001 From: Rich Felker Date: Tue, 29 Aug 2017 19:53:50 -0400 Subject: [PATCH] fix undefined behavior in memset due to missing sequence points patch by Pascal Cuoq. --- src/string/memset.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/string/memset.c b/src/string/memset.c index f438b073..5613a148 100644 --- a/src/string/memset.c +++ b/src/string/memset.c @@ -11,12 +11,16 @@ void *memset(void *dest, int c, size_t n) * offsets are well-defined and in the dest region. */ if (!n) return dest; - s[0] = s[n-1] = c; + s[0] = c; + s[n-1] = c; if (n <= 2) return dest; - s[1] = s[n-2] = c; - s[2] = s[n-3] = c; + s[1] = c; + s[2] = c; + s[n-2] = c; + s[n-3] = c; if (n <= 6) return dest; - s[3] = s[n-4] = c; + s[3] = c; + s[n-4] = c; if (n <= 8) return dest; /* Advance pointer to align it at a 4-byte boundary, -- 2.25.1