diff options
| author | Paul Eggert <eggert@cs.ucla.edu> | 2021-03-21 14:52:08 -0700 |
|---|---|---|
| committer | Paul Eggert <eggert@cs.ucla.edu> | 2021-03-21 14:52:34 -0700 |
| commit | af3745fb175b382b31ce2c343972165fe4a6ca64 (patch) | |
| tree | bc25ed9bce0c879abe3b31b7d8ca6a7962ccc8ce /tests | |
| parent | 0b06bc0c0f0617ee6a1be9b67944e34bedb23f63 (diff) | |
| download | gnulib-af3745fb175b382b31ce2c343972165fe4a6ca64.tar.gz | |
tests: fix signed overflow issues
* tests/test-dynarray.c (value_at): Avoid undefined behavior
in signed integer multiplication.
* tests/test-scratch-buffer.c (byte_at): Likewise, for the
theoretically-possible case where size_t is narrower than int.
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/test-dynarray.c | 2 | ||||
| -rw-r--r-- | tests/test-scratch-buffer.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/tests/test-dynarray.c b/tests/test-dynarray.c index 4308148f89..c3f9a29b54 100644 --- a/tests/test-dynarray.c +++ b/tests/test-dynarray.c @@ -28,7 +28,7 @@ #define N 100000 static int -value_at (int i) +value_at (long long int i) { return (i % 13) + ((i * i) % 251); } diff --git a/tests/test-scratch-buffer.c b/tests/test-scratch-buffer.c index f05c84fa46..517d73fd76 100644 --- a/tests/test-scratch-buffer.c +++ b/tests/test-scratch-buffer.c @@ -24,7 +24,7 @@ #include "macros.h" static int -byte_at (size_t i) +byte_at (unsigned long long int i) { return ((i % 13) + ((i * i) % 251)) & 0xff; } |