diff options
| author | Alexander Egorenkov <egorenar@linux.ibm.com> | 2021-01-15 13:25:43 +0100 |
|---|---|---|
| committer | Petr Vorel <pvorel@suse.cz> | 2021-01-20 12:10:31 +0100 |
| commit | 95720230e358682bd32f0da45c111ac5d058e084 (patch) | |
| tree | 823109087246809d2b8ec5efde098919216f3cab | |
| parent | 7fe2ad11df192dd757bd3cb241f017d9c4f744a8 (diff) | |
swapping01: fix parameter truncation in abs
Parameters passed to abs(int) are of type long. Use labs(long) instead
to avoid value truncation.
Fixes the following warning:
swapping01.c: In function ‘check_swapping’:
swapping01.c:133:7: warning: absolute value function ‘abs’ given an argument of type ‘long int’ but has parameter of type ‘int’ which may cause truncation of value [-Wabsolute-value]
133 | if (abs(swap_free_now - SAFE_READ_MEMINFO("SwapFree:")) < 512)
| ^~~
Signed-off-by: Alexander Egorenkov <egorenar@linux.ibm.com>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
| -rw-r--r-- | testcases/kernel/mem/swapping/swapping01.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/testcases/kernel/mem/swapping/swapping01.c b/testcases/kernel/mem/swapping/swapping01.c index ff40c85c0..24b8313f3 100644 --- a/testcases/kernel/mem/swapping/swapping01.c +++ b/testcases/kernel/mem/swapping/swapping01.c @@ -130,7 +130,7 @@ static void check_swapping(void) while (i < 10) { swap_free_now = SAFE_READ_MEMINFO("SwapFree:"); sleep(1); - if (abs(swap_free_now - SAFE_READ_MEMINFO("SwapFree:")) < 512) + if (labs(swap_free_now - SAFE_READ_MEMINFO("SwapFree:")) < 512) break; i++; |