diff options
| author | Steven G. Johnson <stevenj@mit.edu> | 2016-02-04 10:57:25 -0500 |
|---|---|---|
| committer | Steven G. Johnson <stevenj@mit.edu> | 2016-02-04 10:57:25 -0500 |
| commit | 5a84e53b0cf676b53c9a8b80d42ef91311b124f9 (patch) | |
| tree | cec3e05eb7b416717edf22fa5f8c301fc8df7c77 /test/iterate.c | |
| parent | ec0daa50bbedc36a0bada4a0f713eb9dc317d444 (diff) | |
| parent | 1f17487aa952a11f2ac53481b26b5a25b9a2fccc (diff) | |
| download | libutf8proc-5a84e53b0cf676b53c9a8b80d42ef91311b124f9.tar.gz libutf8proc-5a84e53b0cf676b53c9a8b80d42ef91311b124f9.tar.bz2 | |
Merge pull request #66 from michaelnmmeyer/master
Fix overrun
Diffstat (limited to 'test/iterate.c')
| -rw-r--r-- | test/iterate.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/test/iterate.c b/test/iterate.c index 30b307d..c1674b7 100644 --- a/test/iterate.c +++ b/test/iterate.c @@ -13,11 +13,17 @@ static void testbytes(unsigned char *buf, int len, utf8proc_ssize_t retval, int utf8proc_int32_t out[16]; utf8proc_ssize_t ret; + /* Make a copy to ensure that memory is left uninitialized after "len" + * bytes. This way, Valgrind can detect overreads. + */ + unsigned char tmp[16]; + memcpy(tmp, buf, len); + tests++; - if ((ret = utf8proc_iterate(buf, len, out)) != retval) { + if ((ret = utf8proc_iterate(tmp, len, out)) != retval) { fprintf(stderr, "Failed (%d):", line); for (int i = 0; i < len ; i++) { - fprintf(stderr, " 0x%02x", buf[i]); + fprintf(stderr, " 0x%02x", tmp[i]); } fprintf(stderr, " -> %zd\n", ret); error++; |