Blame SOURCES/0035-v4l2-avoid-shifting-left-a-signed-number-by-31-bit.patch
|
 |
8e169c |
From 91281fec7823f1cd3df3374fbcbd14af52a3fa1b Mon Sep 17 00:00:00 2001
|
|
|
8e169c |
From: Eugene Syromyatnikov <evgsyr@gmail.com>
|
|
|
8e169c |
Date: Wed, 14 Aug 2019 17:15:47 +0200
|
|
|
8e169c |
Subject: [PATCH] v4l2: avoid shifting left a signed number by 31 bit
|
|
|
8e169c |
|
|
|
8e169c |
cppcheck warns about it with the following diagnostics:
|
|
|
8e169c |
|
|
|
8e169c |
error[shiftTooManyBitsSigned]: Shifting signed 32-bit value by 31 bits is
|
|
|
8e169c |
undefined behaviour
|
|
|
8e169c |
|
|
|
8e169c |
* v4l2.c [!v4l2_fourcc_be] (v4l2_fourcc_be): Shift left 1U and not 1 in
|
|
|
8e169c |
order to get 0x80000000.
|
|
|
8e169c |
---
|
|
|
8e169c |
v4l2.c | 2 +-
|
|
|
8e169c |
1 file changed, 1 insertion(+), 1 deletion(-)
|
|
|
8e169c |
|
|
|
8e169c |
diff --git a/v4l2.c b/v4l2.c
|
|
|
8e169c |
index 5da457c..505e7b8 100644
|
|
|
8e169c |
--- a/v4l2.c
|
|
|
8e169c |
+++ b/v4l2.c
|
|
|
8e169c |
@@ -47,7 +47,7 @@ typedef struct v4l2_standard struct_v4l2_standard;
|
|
|
8e169c |
|
|
|
8e169c |
/* v4l2_fourcc_be was added by Linux commit v3.18-rc1~101^2^2~127 */
|
|
|
8e169c |
#ifndef v4l2_fourcc_be
|
|
|
8e169c |
-# define v4l2_fourcc_be(a, b, c, d) (v4l2_fourcc(a, b, c, d) | (1 << 31))
|
|
|
8e169c |
+# define v4l2_fourcc_be(a, b, c, d) (v4l2_fourcc(a, b, c, d) | (1U << 31))
|
|
|
8e169c |
#endif
|
|
|
8e169c |
|
|
|
8e169c |
#define FMT_FRACT "%u/%u"
|
|
|
8e169c |
--
|
|
|
8e169c |
2.1.4
|
|
|
8e169c |
|