Fix build warnings for gcc9

- Ignore implicit-fallthrough warning for intended code
- Fix buffer size for strncat

Change-Id: I004639407efa462291e20357bd47fb221ced3ae0
Signed-off-by: Semun Lee <semun.lee@samsung.com>

diff --git a/CMakeLists.txt b/CMakeLists.txt
index 154792c..962a3bd 100644 (file)
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -19,7 +19,7 @@ ENDFOREACH(flag)
 
 INCLUDE_DIRECTORIES(${CMAKE_SOURCE_DIR}/include/)
 
-SET(ADDITIONAL_CFLAGS "-Wall -Wno-array-bounds -Wno-empty-body -Wno-ignored-qualifiers -Wshadow -Wwrite-strings -Wswitch-default -Wno-unused-but-set-parameter -Wno-unused-but-set-variable -Wcast-qual")
+SET(ADDITIONAL_CFLAGS "-Wall -Wno-array-bounds -Wno-empty-body -Wno-ignored-qualifiers -Wshadow -Wwrite-strings -Wswitch-default -Wno-unused-but-set-parameter -Wno-unused-but-set-variable -Wcast-qual -Wno-implicit-fallthrough")
 SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${EXTRA_CFLAGS}")
 SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Werror -Wextra -fvisibility=hidden -fPIC")
 SET(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -Wno-unused-parameter -Wno-missing-field-initializers -Wdeclaration-after-statement -Wmissing-declarations -Wredundant-decls -Wcast-align")

diff --git a/src/vdpram_dump.c b/src/vdpram_dump.c
index 09c0e1a..8e0fd11 100644 (file)
--- a/src/vdpram_dump.c
+++ b/src/vdpram_dump.c
@@ -41,10 +41,10 @@ static void hex_dump(const char *pad, int size, const void *data)
 
        p = (unsigned const char *)data;
 
-       snprintf(buf, 255, "%s%04X: ", pad, 0);
+       snprintf(buf, sizeof(buf), "%s%04X: ", pad, 0);
        for (i = 0; i < size; i++) {
                snprintf(hex, 4, "%02X ", p[i]);
-               strncat(buf, hex, strlen(hex));
+               strncat(buf, hex, sizeof(buf) - strlen(buf) - 1);
 
                if ((i + 1) % 8 == 0) {
                        if ((i + 1) % 16 == 0) {