fortify-headers

standalone fortify-source implementation
git clone git://git.2f30.org/fortify-headers
Log | Files | Refs | README | LICENSE

commit 7392bcaef8dfdfba67253f3b84f1f1a611e7bab3
parent 3d237352de2832b07b2f61349cf618f5880a1e7b
Author: sin <sin@2f30.org>
Date:   Wed,  4 Mar 2015 17:10:16 +0000

Handle early termination in wcsnrtombs() and mbsnrtowcs()

Diffstat:
Minclude/wchar.h | 16++++++++++++++--
1 file changed, 14 insertions(+), 2 deletions(-)

diff --git a/include/wchar.h b/include/wchar.h @@ -23,8 +23,14 @@ __fortify_mbsnrtowcs(wchar_t *d, const char **s, size_t n, size_t wn, mbstate_t *st) { size_t bos = __builtin_object_size(d, 0); + size_t actual; - if (wn > bos / sizeof(wchar_t)) + if (wn > (size_t)-1 / sizeof(wchar_t)) + __builtin_trap(); + actual = wn * sizeof(wchar_t); + if (actual > n) + actual = n; + if (actual > bos) __builtin_trap(); return mbsnrtowcs(d, s, n, wn, st); } @@ -120,8 +126,14 @@ __fortify_wcsnrtombs(char *d, const wchar_t **s, size_t wn, size_t n, mbstate_t *st) { size_t bos = __builtin_object_size(d, 0); + size_t actual; - if (n > bos) + if (wn > (size_t)-1 / sizeof(wchar_t)) + __builtin_trap(); + actual = wn * sizeof(wchar_t); + if (actual > n) + actual = n; + if (actual > bos) __builtin_trap(); return wcsnrtombs(d, s, wn, n, st); }