0

我在这段代码中找不到错误,我看了好几个小时...... Valgrind 说:

==23114== Invalid read of size 1
==23114== Invalid write of size 1

我尝试使用一些 printfs 进行调试,我认为错误出在此函数中。

void rdm_hide(char *name, Byte* img, Byte* bits, int msg, int n, int size)
{
    FILE *fp;
    int r;/
    Byte* used;
    int i = 0, j = 0;
    int p;

    fp = fopen(name, "wb");

    used = malloc(sizeof(Byte) * msg);


    for(i = 0; i < msg; i++)
        used[i] = -1;


    while(i < 3)
    {
        if(img[j] == '\n')
            i++;
        j++;
    }

    for(i = 0; i < msg; i++)
    {
        r = genrand_int32();
        p = r % n;

        if(!search(p, used, msg))
        {
            used[i] = (Byte)p;

            if(bits[i] == (Byte)0)
                img[j + p] = img[j + p] & (~1);
            else if(bits[i] == (Byte)1)
                img[j + p] = img[j + p] | 1;
        }
        else
            i --;
    }

    for(i = 0; i < size; i++)
        fputc( (char) img[i], fp);

    fclose(fp);
    free(used);
}

感谢帮助!

4

1 回答 1

0

==23114== 大小为 1 的无效读取
==23114== 大小为 1 的无效写入

我很确定这不是valgrind 所说的全部。

你应该

  1. -g使用调试信息(很可能是标志)构建您的程序。这会让 valgrind准确地告诉你哪一行触发了无效的读写
  2. 如果问题不明显,请编辑您的问题并包含整个valgrind 输出。
  3. 重新运行valgrind --track-origins=yes your-exe可能会提供额外的有用信息。

最后,您的算法似乎完全是虚假的。据我所知,j在第一个循环之后变成 3while并且在那之后永远不会改变(在这种情况下你应该只使用const int j = 3;和取消j++)。另外,您参考img[j + p],和p之间的位置。如果确实是 的大小,那么索引超出限制并触发这两个错误也就不足为奇了。0nnimgj + pimg

于 2012-07-03T11:06:24.800 回答