From 48a3b80996d0dce7e472318c2090e5f4df901567 Mon Sep 17 00:00:00 2001 From: Tomi Valkeinen Date: Sat, 13 Sep 2025 18:40:03 +0300 Subject: fix: Correct printf format specifiers for unsigned integers This commit addresses cppcheck warnings about mismatched printf format specifiers (invalidPrintfArgType_sint). The format specifiers %i and %d expect signed integers (int) but the variables being passed are unsigned integers (uint32_t, unsigned). This mismatch can lead to undefined behavior and incorrect output formatting. Changes made: - utils/kmstouch.cpp: Changed %i to %u for event code formatting - utils/kmstouch.cpp: Changed %d to %u for event type and property type formatting - utils/kmsview.cpp: Changed %d to %u for frame number formatting These fixes ensure proper type safety in printf formatting and prevent potential undefined behavior when printing unsigned integer values. --- utils/kmsview.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'utils/kmsview.cpp') diff --git a/utils/kmsview.cpp b/utils/kmsview.cpp index 4b60bb8..7cee024 100644 --- a/utils/kmsview.cpp +++ b/utils/kmsview.cpp @@ -99,7 +99,7 @@ int main(int argc, char** argv) printf("file size %u, frame size %u, frames %u\n", fsize, frame_size, num_frames); for (unsigned i = 0; i < num_frames; ++i) { - printf("frame %d", i); + printf("frame %u", i); fflush(stdout); read_frame(is, fb, crtc, plane); if (!time) { -- cgit v1.2.3