Add -Werror=format-overflow compile flag

Some build environment will automatically add -Werror=format-overflow
compile flag.

Signed-off-by: Min Xu <min.m.xu@intel.com>

teeio-validator/include/ide_test.h

@@ -22,8 +22,8 @@
 #define MAX_SUPPORTED_PORTS_NUM     16
 #define MAX_SUPPORTED_SWITCHES_NUM  16
 #define MAX_SUPPORTED_SWITCH_PORTS_NUM  16
-#define PORT_NAME_LENGTH 16
-#define SWITCH_NAME_LENGTH 16
+#define PORT_NAME_LENGTH 32
+#define SWITCH_NAME_LENGTH 32
 #define BDF_LENGTH 8
 #define DF_LENGTH 5
 #define TEST_CASE_DESCRIPTION_LENGTH 32

teeio-validator/teeio_validator/CMakeLists.txt

@@ -1,6 +1,6 @@
 cmake_minimum_required(VERSION 2.8.12)
 
-ADD_COMPILE_OPTIONS(-Wno-unused-result -Werror)
+ADD_COMPILE_OPTIONS(-Wno-unused-result -Werror=format-overflow -Werror)
 
 if(CMAKE_BUILD_TYPE STREQUAL "Release")
     ADD_COMPILE_OPTIONS(-Wno-error=unused-variable -Wno-error=unused-but-set-variable)

teeio-validator/teeio_validator/scan_pci.c

@@ -41,7 +41,7 @@ bool scan_rp_switch_internal_port_at_bus(IDE_PORT* port, uint8_t bus, uint8_t* S
   int fd = -1;
 
   port->bus = bus;
-  sprintf(port->bdf, "%02x:%02x.%x", bus, port->device, port->function);
+  snprintf(port->bdf, BDF_LENGTH, "%02x:%02x.%x", bus, port->device, port->function&0xf);
 
   TEEIO_DEBUG((TEEIO_DEBUG_INFO, "Scan %s...\n", port->bdf));
 
@@ -114,7 +114,7 @@ bool scan_endpoint_at_bus(uint8_t bus, IDE_PORT* port)
   int fd = -1;
 
   port->bus = bus;
-  sprintf(port->bdf, "%02x:%02x.%x", bus, port->device, port->function);
+  snprintf(port->bdf, BDF_LENGTH, "%02x:%02x.%x", bus, port->device, port->function&0xf);
 
   TEEIO_DEBUG((TEEIO_DEBUG_INFO, "Scan endpoint(%s)...\n", port->bdf));