This is the mail archive of the libc-hacker@sourceware.org mailing list for the glibc project.
Note that libc-hacker is a closed list. You may look at the archives of this list, but subscription and posting are not open.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi! As the attached testcase shows (found when testing on i486 in sort-test, but it didn't show up on any other arch), I messed up the test when to terminate the string. We store at needed - 1, so even if --needed == n needed - 1 is < n and therefore the returned string is valid. Sorry. 2006-11-10 Jakub Jelinek <jakub@redhat.com> * string/strxfrm_l.c (STRXFRM): Fix trailing \1 optimization if N is one bigger than return value. * string/tst-strxfrm2.c (do_test): Also test strxfrm with l1 + 1 and l1 last arguments, if buf is defined, verify the return value equals to strlen (buf) and verify no byte beyond passed length is modified. --- libc/string/strxfrm_l.c 9 Nov 2006 20:17:42 -0000 1.6 +++ libc/string/strxfrm_l.c 10 Nov 2006 08:45:07 -0000 @@ -432,7 +432,7 @@ STRXFRM (STRING_TYPE *dest, const STRING if (needed > 2 && needed == last_needed + 1) { /* Remove the \1 byte. */ - if (--needed < n) + if (--needed <= n) dest[needed - 1] = L('\0'); } --- libc/string/tst-strxfrm2.c 9 Nov 2006 20:18:54 -0000 1.1 +++ libc/string/tst-strxfrm2.c 10 Nov 2006 08:45:07 -0000 @@ -7,14 +7,34 @@ do_test (void) { int res = 0; - char buf[10]; + char buf[20]; size_t l1 = strxfrm (NULL, "ab", 0); size_t l2 = strxfrm (buf, "ab", 1); size_t l3 = strxfrm (buf, "ab", sizeof (buf)); + if (l3 < sizeof (buf) && strlen (buf) != l3) + { + puts ("C locale l3 test failed"); + res = 1; + } + + size_t l4 = strxfrm (buf, "ab", l1 + 1); + if (l4 < l1 + 1 && strlen (buf) != l4) + { + puts ("C locale l4 test failed"); + res = 1; + } + + buf[l1] = 'Z'; + size_t l5 = strxfrm (buf, "ab", l1); + if (buf[l1] != 'Z') + { + puts ("C locale l5 test failed"); + res = 1; + } - if (l1 != l2 || l1 != l3) + if (l1 != l2 || l1 != l3 || l1 != l4 || l1 != l5) { - puts ("C locale test failed"); + puts ("C locale retval test failed"); res = 1; } @@ -28,10 +48,30 @@ do_test (void) l1 = strxfrm (NULL, "ab", 0); l2 = strxfrm (buf, "ab", 1); l3 = strxfrm (buf, "ab", sizeof (buf)); + if (l3 < sizeof (buf) && strlen (buf) != l3) + { + puts ("UTF-8 locale l3 test failed"); + res = 1; + } + + l4 = strxfrm (buf, "ab", l1 + 1); + if (l4 < l1 + 1 && strlen (buf) != l4) + { + puts ("UTF-8 locale l4 test failed"); + res = 1; + } + + buf[l1] = 'Z'; + l5 = strxfrm (buf, "ab", l1); + if (buf[l1] != 'Z') + { + puts ("UTF-8 locale l5 test failed"); + res = 1; + } - if (l1 != l2 || l1 != l3) + if (l1 != l2 || l1 != l3 || l1 != l4 || l1 != l5) { - puts ("UTF-8 locale test failed"); + puts ("UTF-8 locale retval test failed"); res = 1; } } Jakub
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |