From ee6e65a5bf16c4ea6069d217d3356cfe041439e3 Mon Sep 17 00:00:00 2001 From: Alex T Date: Sun, 12 Oct 2025 21:14:34 +0200 Subject: [PATCH] mock: use MRAA_PIN_NAME_SIZE for strncpy to avoid truncation MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit This fixes the following compilation warnings: mraa/src/src/mock/mock_board.c: In function ‘mraa_mock_board’: mraa/src/src/mock/mock_board.c:156:5: warning: ‘strncpy’ output truncated before terminating nul copying 8 bytes from a string of the same length [-Wstringop-truncation] 156 | strncpy(b->pins[pos].name, "SPI0MOSI", 8); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ mraa/src/src/mock/mock_board.c:162:5: warning: ‘strncpy’ output truncated before terminating nul copying 8 bytes from a string of the same length [-Wstringop-truncation] 162 | strncpy(b->pins[pos].name, "SPI0MISO", 8); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ mraa/src/src/mock/mock_board.c:168:5: warning: ‘strncpy’ output truncated before terminating nul copying 8 bytes from a string of the same length [-Wstringop-truncation] 168 | strncpy(b->pins[pos].name, "SPI0SCLK", 8); | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ Signed-off-by: Alex T --- src/mock/mock_board.c | 20 ++++++++++---------- 1 file changed, 10 insertions(+), 10 deletions(-) diff --git a/src/mock/mock_board.c b/src/mock/mock_board.c index d626cf1..d83f8de 100644 --- a/src/mock/mock_board.c +++ b/src/mock/mock_board.c @@ -123,62 +123,62 @@ mraa_mock_board() // Pin definitions int pos = 0; - strncpy(b->pins[pos].name, "GPIO0", 8); + strncpy(b->pins[pos].name, "GPIO0", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 1, 0, 0, 0, 0, 0, 0 }; b->pins[pos].gpio.pinmap = 0; b->pins[pos].gpio.mux_total = 0; pos++; - strncpy(b->pins[pos].name, "ADC0", 8); + strncpy(b->pins[pos].name, "ADC0", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 0, 0, 1, 0 }; b->pins[pos].aio.pinmap = 0; b->pins[pos].aio.mux_total = 0; pos++; - strncpy(b->pins[pos].name, "I2C0SDA", 8); + strncpy(b->pins[pos].name, "I2C0SDA", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 0, 1, 0, 0 }; b->pins[pos].i2c.mux_total = 0; b->pins[pos].i2c.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "I2C0SCL", 8); + strncpy(b->pins[pos].name, "I2C0SCL", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 0, 1, 0, 0 }; b->pins[pos].i2c.mux_total = 0; b->pins[pos].i2c.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "SPI0CS", 8); + strncpy(b->pins[pos].name, "SPI0CS", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 1, 0, 0, 0 }; b->pins[pos].spi.mux_total = 0; b->pins[pos].spi.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "SPI0MOSI", 8); + strncpy(b->pins[pos].name, "SPI0MOSI", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 1, 0, 0, 0 }; b->pins[pos].spi.mux_total = 0; b->pins[pos].spi.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "SPI0MISO", 8); + strncpy(b->pins[pos].name, "SPI0MISO", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 1, 0, 0, 0 }; b->pins[pos].spi.mux_total = 0; b->pins[pos].spi.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "SPI0SCLK", 8); + strncpy(b->pins[pos].name, "SPI0SCLK", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 1, 0, 0, 0 }; b->pins[pos].spi.mux_total = 0; b->pins[pos].spi.pinmap = 0; pos++; - strncpy(b->pins[pos].name, "UART0RX", 8); + strncpy(b->pins[pos].name, "UART0RX", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 0, 0, 0, 1 }; b->pins[pos].uart.pinmap = 0; b->pins[pos].uart.parent_id = 0; b->pins[pos].uart.mux_total = 0; pos++; - strncpy(b->pins[pos].name, "UART0TX", 8); + strncpy(b->pins[pos].name, "UART0TX", MRAA_PIN_NAME_SIZE); b->pins[pos].capabilities = (mraa_pincapabilities_t){ 1, 0, 0, 0, 0, 0, 0, 1 }; b->pins[pos].uart.pinmap = 0; b->pins[pos].uart.parent_id = 0;