glibc 检测到错误 - id3Tag

Question

所以，我是 C 新手，我一直在尝试编写这个工具来从命令行编辑音乐文件上的 id3 标签。我一直收到这个错误：

* glibc 检测到双重释放或损坏（顶部）：0x0000000000502010 **

从我读到的内容来看，我知道它与释放内存。不过，我只是不太确定从这里该去哪里。无论如何，我的逻辑是，如果存在标签，我会读取该标签，然后根据命令行中指定的字段进行所需的任何更改。这是一直给我带来麻烦的块。感谢您提前提供任何见解！

    fopen(argv[1], "rb");
    fseek(in_file, -128, SEEK_END);
    fread(&tagTest, sizeof(struct iD3Tag), 1, in_file);
    fclose(in_file);

    for (x = 2; x < argc-1; x++)
    {            
        if (strcmp(argv[x], "-title"))
            strncpy(tagTest.title, argv[x+1], 30);
        if (strcmp(argv[x], "-artist"))
            strncpy(tagTest.artist, argv[x+1], 30);
        if (strcmp(argv[x], "-album"))
            strncpy(tagTest.album, argv[x+1], 30);
        if (strcmp(argv[x], "-year"))
            strncpy(tagTest.year, argv[x+1], 4);
        if (strcmp(argv[x], "-comment"))
            strncpy(tagTest.comment, argv[x+1], 28);
        if (strcmp(argv[x], "-track"))
            tagTest.track = atoi(argv[x+1]);
    }

    tagTest.seperator = 0;

    fopen(argv[1], "r+b");
    fseek(in_file, -128, SEEK_END);
    fwrite(&tagTest, sizeof(struct iD3Tag), 1, in_file);
    fclose(in_file);

Answer 1

这段代码中没有 free 调用，所以我想您很高兴没有双重释放任何东西？因此，我认为你这里有一些堆损坏。也就是说，您向标签写入的数据超出了空间。

首先，为了解决@H2CO3 的观点，我不认为这就是问题所在。他提到的问题可能确实存在 - 在这种情况下你会损坏你的数据文件 - 但看起来这并不是程序失败的原因。

我认为您真正的问题是您从未将任何内容分配给 in_file！我认为您需要：

in_file = fopen (argv[1], "rb");

一些其他注释：

• 当字符串匹配时，strcmp 返回零（false），当字符串匹配时，返回非零（true）...所以所有您的比较似乎已损坏 - 这不应该导致腐败。
• 该循环似乎检查参数值以及开关 - 您应该跳过这些。
• 无需打开文件两次 - 您可以在开始时打开它进行读取和写入。
• 您需要进行更多的错误检查，以防文件不存在或不是您所期望的 - 但我确信您知道这一点。