Fix #2118, Endian macro mask before shift to avoid shift overflow war…

…ning
nasa · Jun 14, 2022 · 41897e1 · 41897e1
1 parent 10898c9
commit 41897e1
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/modules/core_api/fsw/inc/cfe_endian.h b/modules/core_api/fsw/inc/cfe_endian.h
@@ -61,9 +61,9 @@
 #define CFE_MAKE_BIG16(n) (n)
 #define CFE_MAKE_BIG32(n) (n)
 #else
-#define CFE_MAKE_BIG16(n) ((((n) << 8) & 0xFF00) | (((n) >> 8) & 0x00FF))
+#define CFE_MAKE_BIG16(n) ((((n) & 0x00FF) << 8) | (((n) & 0xFF00) >> 8))
 #define CFE_MAKE_BIG32(n) \
-    ((((n) << 24) & 0xFF000000) | (((n) << 8) & 0x00FF0000) | (((n) >> 8) & 0x0000FF00) | (((n) >> 24) & 0x000000FF))
+    ((((n) & 0x000000FF) << 24) | (((n) & 0x0000FF00) << 8) | (((n) & 0x00FF0000) >> 8) | (((n) & 0xFF000000) >> 24))
 #endif
 
 #endif /* CFE_ENDIAN_H */