0

我正在尝试格式化字符串，然后在ISR期间将其打印到LCD上。注释掉的ISR功能正常；但是当它被引入时，程序在ISR期间崩溃。sprintf()

void __ISR(_CHANGE_NOTICE_VECTOR, IPL1) CNIntHandler(void) {
    //more of my code later
    LATBINV = LEDC; //invert LEDC to a 1
    hw_msDelay(20);

    int buttons = read_buttons();
    decode_buttons(buttons, &g_step_mode, &g_step_dir, &g_motor_delay);

    LATBINV = LEDC; //reinvert LEDC to 0

    mCNClearIntFlag(); //clear the CN flag
}

sprintf()发生在)decode_buttons(函数中：

void decode_buttons(int f_buttons, int *f_step_mode, int *f_step_dir, int *f_motor_delay) {
    char f_mode_str;
    char f_dir_str;
    int f_RPM;
    char f_LCD_str;

    switch(f_buttons) {
        case 0x0000: //none pressed
            *f_step_mode = HS;
            f_mode_str = "HALF";
            *f_step_dir = CW;
            f_dir_str = "CW";
            *f_motor_delay = 20; // semipermanent value
            f_RPM = 15;
            break;

        case 0x0040: //BTN 1 pressed
            *f_step_mode = FS;
            f_mode_str = "FULL";
            *f_step_dir = CW;
            f_dir_str = "CW";
            *f_motor_delay = 40;
            f_RPM = 15;
            break;

        case 0x0080: // BTN 2 pressed
            *f_step_mode = HS;
            f_mode_str = "HALF";
            *f_step_dir = CCW;
            f_dir_str = "CCW";
            *f_motor_delay = 30;
            f_RPM = 10;
            break;

        case 0x00C0: //Both pressed
            *f_step_mode = FS;
            f_mode_str = "FULL";
            *f_step_dir = CCW;
            f_dir_str = "CCW";
            *f_motor_delay = 24;
            f_RPM = 25;
            break;

        default:
            break;
    }

    sprintf(f_LCD_str, "%s %s %d", f_dir_str, f_mode_str, &f_RPM);
    LCD_puts(f_LCD_str); //output string to LCD

    return;
}

我正在开发PIC32 Cerebot板，目标是使用步进电机的当前模式、方向和 RPM更新LCD 。我会澄清我能做什么并发布任何需要的东西。

4

1 回答 1

0

sprintf 组成一个字符串并将其存储在一个必须足够大的数组中：http ://www.cplusplus.com/reference/cstdio/sprintf/

在您的代码中，您传递给 sprintf 的输出缓冲区声明为

char f_LCD_str;

这不是一个数组，而是一个字符。sprintf 将不可避免地溢出。一个好的声明将是

char f_LCD_str[30];

（对您的其他 char 数组也有同样的评论：f_mode_str 和 f_dir_str）

为了避免溢出风险，使用 snprintf 总是更安全。在这种情况下，调用将是：

sprintf(f_LCD_str, 30, "%s %s %d", f_dir_str, f_mode_str, f_RPM);

注意： &f_RPM 将打印变量的地址，其中 f_RPM 将根据您的需要打印出变量值。

于 2015-03-27T21:37:15.113 回答