Skip to content

Commit ffbda7c

Browse files
tpambortpambor
committed
tests: posix: fix -Wformat errors
Use %lld format specifier for variables of type eventfd_t. Cast variables of type time_t to long long and use %lld format specifier to ensure compatibility with 32-bit and 64-bit time_t. Signed-off-by: Tim Pambor <[email protected]>
1 parent 9ae10ea commit ffbda7c

File tree

6 files changed

+21
-21
lines changed

6 files changed

+21
-21
lines changed

‎tests/posix/common/src/pthread_attr.c‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -527,9 +527,9 @@ ZTEST(pthread_attr, test_pthread_attr_policy_and_priority_limits)
527527
*
528528
* Note: we relax this requirement
529529
*/
530-
zassert_true(pmax > pmin, "pmax (%d) <= pmin (%d)", pmax, pmin,
531-
"%s min/max inconsistency: pmin: %d pmax: %d",
532-
policy_names[policy], pmin, pmax);
530+
zassert_true(pmax > pmin,
531+
"%s min/max inconsistency, pmax (%d) <= pmin (%d)",
532+
policy_names[policy], pmax, pmin);
533533

534534
/*
535535
* Getting into the weeds a bit (i.e. whitebox testing), Zephyr

‎tests/posix/eventfd/src/blocking.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@ ZTEST_F(eventfd, test_write_then_read)
1919

2020
ret = eventfd_read(fixture->fd, &val);
2121
zassert_true(ret == 0, "read ret %d", ret);
22-
zassert_true(val == 5, "val == %d", val);
22+
zassert_true(val == 5, "val == %lld", val);
2323

2424
/* Test EFD_SEMAPHORE */
2525
reopen(&fixture->fd, 0, EFD_SEMAPHORE);
@@ -32,7 +32,7 @@ ZTEST_F(eventfd, test_write_then_read)
3232

3333
ret = eventfd_read(fixture->fd, &val);
3434
zassert_true(ret == 0, "read ret %d", ret);
35-
zassert_true(val == 1, "val == %d", val);
35+
zassert_true(val == 1, "val == %lld", val);
3636
}
3737

3838
ZTEST_F(eventfd, test_zero_shall_not_unblock)

‎tests/posix/eventfd/src/ioctl.c‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ ZTEST_F(eventfd, test_set_flags)
3232

3333
ret = eventfd_read(fixture->fd, &val);
3434
zassert_ok(ret);
35-
zassert_equal(val, 3, "val == %d", val);
35+
zassert_equal(val, 3, "val == %lld", val);
3636

3737

3838
/* Set nonblocking without reopening. */
@@ -53,7 +53,7 @@ ZTEST_F(eventfd, test_set_flags)
5353

5454
ret = eventfd_read(fixture->fd, &val);
5555
zassert_ok(ret);
56-
zassert_equal(val, 19, "val == %d", val);
56+
zassert_equal(val, 19, "val == %lld", val);
5757

5858

5959
/* Set back to blocking. */
@@ -74,7 +74,7 @@ ZTEST_F(eventfd, test_set_flags)
7474

7575
ret = eventfd_read(fixture->fd, &val);
7676
zassert_ok(ret);
77-
zassert_equal(val, 10, "val == %d", val);
77+
zassert_equal(val, 10, "val == %lld", val);
7878

7979

8080
/* Test setting internal in-use-flag. Should fail. */
@@ -88,5 +88,5 @@ ZTEST_F(eventfd, test_set_flags)
8888

8989
ret = eventfd_read(fixture->fd, &val);
9090
zassert_ok(ret);
91-
zassert_equal(val, 97, "val == %d", val);
91+
zassert_equal(val, 97, "val == %lld", val);
9292
}

‎tests/posix/eventfd/src/nonblocking.c‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,10 +22,10 @@ ZTEST_F(eventfd, test_read_nonblock)
2222

2323
ret = eventfd_read(fixture->fd, &val);
2424
zassert_true(ret == 0, "read set ret %d", ret);
25-
zassert_true(val == TESTVAL, "red set val %d", val);
25+
zassert_true(val == TESTVAL, "red set val %lld", val);
2626

2727
ret = eventfd_read(fixture->fd, &val);
28-
zassert_true(ret == -1, "read subsequent ret %d val %d", ret, val);
28+
zassert_true(ret == -1, "read subsequent ret %d val %lld", ret, val);
2929
zassert_true(errno == EAGAIN, "errno %d", errno);
3030
}
3131

‎tests/posix/signals/src/main.c‎

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ ZTEST(posix_signals, test_sigaddset)
7474
for (int i = 0; i < SIGSET_NLONGS; i++) {
7575
zassert_equal(_set[i], _target[i],
7676
"set.sig[%d of %d] has content: %lx, expected %lx", i,
77-
SIGSET_NLONGS - 1, _set[i], _target[i]);
77+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
7878
}
7979

8080
signo = SIGSYS;
@@ -83,7 +83,7 @@ ZTEST(posix_signals, test_sigaddset)
8383
for (int i = 0; i < SIGSET_NLONGS; i++) {
8484
zassert_equal(_set[i], _target[i],
8585
"set.sig[%d of %d] has content: %lx, expected %lx", i,
86-
SIGSET_NLONGS - 1, _set[i], _target[i]);
86+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
8787
}
8888

8989
/* TODO: move rt signal tests to realtime_signals testsuite */
@@ -108,7 +108,7 @@ ZTEST(posix_signals, test_sigaddset)
108108
for (int i = 0; i < SIGSET_NLONGS; i++) {
109109
zassert_equal(_set[i], _target[i],
110110
"set.sig[%d of %d] has content: %lx, expected %lx", i,
111-
SIGSET_NLONGS - 1, _set[i], _target[i]);
111+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
112112
}
113113
}
114114
}
@@ -144,7 +144,7 @@ ZTEST(posix_signals, test_sigdelset)
144144
for (int i = 0; i < SIGSET_NLONGS; i++) {
145145
zassert_equal(_set[i], _target[i],
146146
"set.sig[%d of %d] has content: %lx, expected %lx", i,
147-
SIGSET_NLONGS - 1, _set[i], _target[i]);
147+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
148148
}
149149

150150
signo = SIGSYS;
@@ -153,7 +153,7 @@ ZTEST(posix_signals, test_sigdelset)
153153
for (int i = 0; i < SIGSET_NLONGS; i++) {
154154
zassert_equal(_set[i], _target[i],
155155
"set.sig[%d of %d] has content: %lx, expected %lx", i,
156-
SIGSET_NLONGS - 1, _set[i], _target[i]);
156+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
157157
}
158158

159159
/* TODO: move rt signal tests to realtime_signals testsuite */
@@ -178,7 +178,7 @@ ZTEST(posix_signals, test_sigdelset)
178178
for (int i = 0; i < SIGSET_NLONGS; i++) {
179179
zassert_equal(_set[i], _target[i],
180180
"set.sig[%d of %d] has content: %lx, expected %lx", i,
181-
SIGSET_NLONGS - 1, _set[i], _target[i]);
181+
(int)(SIGSET_NLONGS - 1), _set[i], _target[i]);
182182
}
183183
}
184184
}

‎tests/posix/timers/src/nanosleep_common.c‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,10 +66,10 @@ void common_lower_bound_check(int selection, clockid_t clock_id, int flags, cons
6666

6767
zassert_equal(r, 0, "actual: %d expected: %d", r, 0);
6868
zassert_equal(errno, 0, "actual: %d expected: %d", errno, 0);
69-
zassert_equal(req.tv_sec, s, "actual: %d expected: %d", req.tv_sec, s);
70-
zassert_equal(req.tv_nsec, ns, "actual: %d expected: %d", req.tv_nsec, ns);
71-
zassert_equal(rem.tv_sec, 0, "actual: %d expected: %d", rem.tv_sec, 0);
72-
zassert_equal(rem.tv_nsec, 0, "actual: %d expected: %d", rem.tv_nsec, 0);
69+
zassert_equal(req.tv_sec, s, "actual: %lld expected: %d", (long long)req.tv_sec, s);
70+
zassert_equal(req.tv_nsec, ns, "actual: %lld expected: %d", (long long)req.tv_nsec, ns);
71+
zassert_equal(rem.tv_sec, 0, "actual: %lld expected: %d", (long long)rem.tv_sec, 0);
72+
zassert_equal(rem.tv_nsec, 0, "actual: %lld expected: %d", (long long)rem.tv_nsec, 0);
7373

7474
switch (selection) {
7575
case SELECT_NANOSLEEP:

0 commit comments

Comments
 (0)