Changes in common/stdc/uchar.c [0600976:696b405] in mainline

File:

• common/stdc/uchar.c (modified) (4 diffs)

common/stdc/uchar.c

@@ -84 +84 @@
 }
 
-static bool _is_non_shortest(unsigned short cont, uint8_t b)
-{
-        return (cont == 0b1111110000000000 && !(b & 0b00100000)) ||
-            (cont == 0b1111111111110000 && !(b & 0b00110000));
-}
-
 size_t mbrtoc32(char32_t *c, const char *s, size_t n, mbstate_t *mb)
 {
@@ -145 +139 @@
 
                 if (_is_2_byte(b)) {
-                        /* Reject non-shortest form. */
-                        if (!(b & 0b00011110)) {
-                                _set_ilseq();
-                                return UCHAR_ILSEQ;
-                        }
-
                         /* 2 byte encoding               110xxxxx */
                         mb->continuation = b ^ 0b0000000011000000;
@@ -164 +152 @@
         }
 
-        for (; i < n; i++) {
+        while (i < n) {
                 /* Read continuation bytes. */
-                uint8_t b = s[i];
-
-                if (!_is_continuation(b) || _is_non_shortest(mb->continuation, b)) {
+
+                if (!_is_continuation(s[i])) {
                         _set_ilseq();
                         return UCHAR_ILSEQ;
@@ -175 +162 @@
                 /* Top bit becomes zero just before the last byte is shifted in. */
                 if (!(mb->continuation & 0x8000)) {
-                        *c = ((char32_t) mb->continuation) << 6 | (b & 0x3f);
+                        *c = ((char32_t) mb->continuation) << 6 | (s[i++] & 0x3f);
                         mb->continuation = 0;
-                        return ++i;
-                }
-
-                mb->continuation = mb->continuation << 6 | (b & 0x3f);
+                        return i;
+                }
+
+                mb->continuation = mb->continuation << 6 | (s[i++] & 0x3f);
         }