c - 为什么我的可变参数 vsscanf 函数有未定义的行为？

Question

我创建了一个可变参数函数来解析可能具有不同参数的字符串。 在当前情况下，传入字符串将为“38400,8,N,1\n”。

static int parse_response(const char *buff, char *format, ...){
    
    uint16_t retval=0;
    va_list arg_ptr;
    va_start(arg_ptr, format);

    retval = vsscanf((const char*)buff, format, arg_ptr);
    va_end(arg_ptr);
    return retval;
}

这是我调用 vsscanf 函数的函数。

 satel_return_typedef Satel_GetBaudrate(satel_typedef* Satel, uint32_t* Baudrate, uint32_t timeout_ms){
    
    int Len;
    Len = printf_SLCommandString(Satel->tx_data_buffer, Satel->max_size, "SL%%B?");
    uint8_t DataBits; char Parity; uint8_t StopBit;
    if(Len < 0 || Len > (int)Satel->max_size) {
        return R_API_ERROR;
    }
    if(Satel->Platform->Comm_Send(Satel->tx_data_buffer, Len, NULL)<0)
    return R_PLATFORM_ERROR;
    if(Satel_WaitForCommResponse(Satel,timeout_ms)==R_COMM_RESPONSE_VALUE){
        parse_response((char*)Satel->rx_data_buffer,"%d,%d,%c,%d\n", Baudrate, &DataBits, &Parity, &StopBit);
        return R_OP_SUCCESSED;
    }     
    return R_OP_ERROR;
}

我知道，有一些规则可以避免陷入未定义的行为，但是，我很困惑这里违反了哪些规则。

typedef void (*Write_Enamod_Pin)(satel_power_typedef Config);
typedef void (*Write_Service_Pin)(satel_service_typedef Config);
typedef int32_t (*Send_Message)(const uint8_t *data_to_send, uint32_t bytes_to_send, void *custom); /* -1 IO Error*/
typedef int32_t (*Read_Message)(uint8_t *data_to_read, uint32_t bytes_to_read, void *custom); /* -1 IO Error*/
typedef uint32_t (*Get_TickCount)(void);
typedef void (*Delay_ms)(uint32_t milisec);
typedef struct{
    Write_Enamod_Pin    Power;
    Write_Service_Pin   Service;
    Send_Message        Comm_Send;
    Read_Message        Comm_Read;
    Get_TickCount       Get_TickCount;
    Delay_ms            Delay_Ms;
}satel_hw_typedef;
typedef struct{
    satel_state_typedef State;
    uint8_t*            tx_data_buffer;
    uint8_t*            rx_data_buffer;
    uint32_t            max_size;
    satel_hw_typedef*   Platform;
}satel_typedef;

Answer 1

格式说明符不匹配。

%d 格式说明符需要一个 int *。在您使用它的 3 个地方，您传递了一个 uint32_t *、一个 uint8_t * 和一个 uint8_t *。这些不匹配会触发未定义的行为。

您需要使用 u 作为所有 3 个无符号整数参数，并且您希望使用 hh 修饰符作为 uint8_t (即 unsigned char) 参数。另外，您希望从格式字符串中删除 \n，因为这样可以防止读取停止，直到您按 ENTER 后按其他非空白字符。

"%u,%hhu,%c,%hhu"