Question

我会尝试仅发布重要部分，因为整个程序有1000多行。它通常不会崩溃，运行好几天。今天我一直在尝试实现一些Redis函数来与程序进行通信，以便从MySQL数据库中获取新数据。

int RedisCheckIfUpdate(redisContext *redis_c) {
    redisReply *redis_reply;
    redis_reply = redisCommand(redis_c, "GETSET update_status 0");

    if (redis_reply->type == REDIS_REPLY_STRING) {
        if (strcmp(redis_reply->str, "newsticker") == 0) {
            freeReplyObject(redis_reply);
            return 1;
        } else if (strcmp(redis_reply->str, "slides") == 0) {
            freeReplyObject(redis_reply);
            return 2;
        }
    }

    freeReplyObject(redis_reply);
    return 0;
}

int main(int argc, char **argv) {
    int redis_update_status;

    redisContext *redis_c;
    redis_c = redisConnect((char*)"127.0.0.1", 6379);

    if (redis_c->err) {
        printf("Connection error: %s\n", redis_c->errstr);
        redisFree(redis_c);
        exit(1);
    }

    while (1) {
        redis_update_status = RedisCheckIfUpdate(redis_c); // THIS SINGLE LINE CAUSES INSTANT SEGFAULT!
        RedisCheckIfUpdate(redis_c); // THIS WORKS 

        if (RedisCheckIfUpdate(redis_c) == 1) {
            MYSQLNewsticker(con, &nt);
        } else if (RedisCheckIfUpdate(redis_c) == 2) {
            MYSQLSlides(con, &pl);
        }             
    }
}

这一行...

redis_update_status = RedisCheckIfUpdate(redis_c);

...导致程序在第一次循环迭代时立即崩溃。一旦我发表评论，即使调用了相同的函数，它也不会崩溃并继续运行 - 只是没有将返回值分配给任何东西。

我可能有一些解决方法，但我只是不明白为什么会这么错？它不是那么复杂的功能，我想要实现的是将返回值赋给变量。变量没有其他任何事情发生。如果我注释掉所说的那行，我甚至会收到一条警告，说在编译期间redis_update_status未被使用...

编辑：我已经删除了除了函数体“返回0”之外的一切，它仍然崩溃了！

Answer 1

就像有些人在评论中指出的那样，分段错误是由我发布的代码之外的东西引起的：

char time_formatted[30], date_formatted[30];

// Redis connect stuff

while (1) {
    // redis_update_status = RedisCheckIfUpdate(); part goes here
    (...)

    // BEGIN header
    TextMid(1760, 970, time_formatted, SegoeUISB, 60);
    RGBA(41, 39, 39, 1, temp_fill_color);
    vgSetParameterfv(temp_fill, VG_PAINT_COLOR, 4, temp_fill_color);
    vgSetPaint(temp_fill, VG_FILL_PATH);
    WTextMid(1760, 935, date_formatted, SegoeUI, 19);
    // END header

    (...)

    // BEGIN update datetime values
    time(&rawtime);
    timeinfo = localtime(&rawtime);
    strftime(time_formatted, 30, "%H:%M", timeinfo);
    wcsftime(date_formatted, 30, L"%A, %d.%m.%Y", timeinfo);
    // END update datetime values
}

正如您所看到的，我试图从字符串变量中绘制时钟和日期文本，这些变量在第一次迭代时没有设置为任何东西。在循环开始时移动更新日期时间值部分后，我能够正确运行整个程序，包括Redis部分。

这是一个非常愚蠢的错误，我必须在编程程序的过程中很早就做出这个错误，而且几个星期以来一直没有引起注意。对我来说唯一令人难以置信的事情是，为什么直到现在这几周都没有崩溃，当时我添加了一个简单的Redis功能，与崩溃的部分完全无关......

C - 尝试将函数的返回值赋给变量会导致分段错误

1 个答案: