fix handling of odd lengths in swab function
authorRich Felker <dalias@aerifal.cx>
Sat, 4 Oct 2014 15:14:01 +0000 (11:14 -0400)
committerRich Felker <dalias@aerifal.cx>
Sat, 4 Oct 2014 15:14:01 +0000 (11:14 -0400)
this function is specified to leave the last byte with "unspecified
disposition" when the length is odd, so for the most part correct
programs should not be calling swab with odd lengths. however, doing
so is permitted, and should not write past the end of the destination
buffer.

src/string/swab.c

index 9ed6fcd..ace0f46 100644 (file)
@@ -4,7 +4,7 @@ void swab(const void *restrict _src, void *restrict _dest, ssize_t n)
 {
        const char *src = _src;
        char *dest = _dest;
-       for (; n>0; n-=2) {
+       for (; n>1; n-=2) {
                dest[0] = src[1];
                dest[1] = src[0];
                dest += 2;