Mercurial > dovecot > core-2.2
changeset 18114:e9ac32255ef1
lib: test-array - fix off-by-one in test_reverse
i is not used as an index, but as a count. Therefore, it can attain the
array size in the loop without causing any buffer overflows.
Signed-off-by: Phil Carmody <phil@dovecot.fi>
| author | Phil Carmody <phil@dovecot.fi> |
|---|---|
| date | Tue, 25 Nov 2014 03:45:35 +0200 |
| parents | 9141a16aff4e |
| children | b7074256cc65 |
| files | src/lib/test-array.c |
| diffstat | 1 files changed, 1 insertions(+), 1 deletions(-) [+] |
line wrap: on
line diff
--- a/src/lib/test-array.c Tue Nov 25 03:44:55 2014 +0200 +++ b/src/lib/test-array.c Tue Nov 25 03:45:35 2014 +0200 @@ -41,7 +41,7 @@ test_begin("array reverse"); t_array_init(&intarr, 5); - for (i = 0; i < N_ELEMENTS(input); i++) { + for (i = 0; i <= N_ELEMENTS(input); i++) { array_clear(&intarr); array_append(&intarr, input, i); array_reverse(&intarr);