crnlib: attempt to silence false positive CodeQL cpp/static-buffer-ov…

…erflow On the default case it wrongly reports as critical: > Potential buffer-overflow: 'm_buf' has size 2 but 'm_buf[3]' may be accessed here. Because it fails to understand that default only happens with m_buf size being 4.
DaemonEngine · Jul 6, 2024 · c83c921 · c83c921
1 parent a4f81a6
commit c83c921
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/crnlib/crn_packed_uint.h b/crnlib/crn_packed_uint.h
@@ -39,7 +39,7 @@ struct packed_uint {
   }
 
   inline operator unsigned int() const {
-    switch (N) {
+    switch (sizeof(m_buf)) {
       case 1:
         return m_buf[0];
       case 2:

diff --git a/inc/crn_defs.h b/inc/crn_defs.h
@@ -225,7 +225,7 @@ struct crn_packed_uint {
   }
 
   inline operator unsigned int() const {
-    switch (N) {
+    switch (sizeof(m_buf)) {
       case 1:
         return m_buf[0];
       case 2: