c - 重新分配() : invalid pointer error when using argc and argv

Question

作为练习的一部分，我正在用 C 语言开发一个类似 grep 的小程序，该程序应该接收一个字符串和一些文件名，以便在引用的文件中查找该字符串。

我通过对文件名和字符串进行硬编码来使其工作:

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int searchString(char *str1, char *str2) {
    if (strstr(str1, str2) == NULL) {
        return 0;
    } else {
        return 1;
    }
}

void searchInFile(char *filename, char *str) {
    FILE *f;
    int lineno =0;
    size_t len;
    char *buffer;
    f = fopen(filename, "r");
    if (f != NULL) {
        while (getline(&buffer, &len, f) != -1) {
            lineno++;
            if (searchString(buffer, str)) {
                printf("[%s:%d] %s", filename, lineno, buffer);
            } 
        }
    }
    fclose(f);
}

int main() {
    searchInFile("loremipsum.txt", "dolor");
    return 0;
}

但是一旦我修改代码以更改 int main() 行以添加 argc 和 argv，编译它然后运行它，脚本就会返回一个错误信息。修改为:

int main(int argc, char *argv[]) {
    searchInFile("loremipsum.txt", "dolor");
    return 0;
}

收到的错误:

* Error in `./a.out': realloc(): invalid pointer: 0x000056424999394d *

有人知道为什么我会收到此错误吗？

Answer 1

您使用 getline() 的主要问题是在传递 buffer 之前未能初始化 len = 0; 和 buffer = NULL; 作为 lineptr 参数，len 作为 n 参数。 man 3 getline，解释:

ssize_t getline(char **lineptr, size_t *n, FILE *stream); 
...
If *lineptr is set to NULL and *n is set 0 before the call, then 
getline() will allocate a buffer for storing the line."

除非您初始化 lineptr 的指针和 n 的值，否则它们将包含不确定(垃圾)值，从而阻止 getline 自动执行正确处理分配。

修复这些问题，提供一些小的清理，并允许将文件名和搜索词作为第一个和第二个参数传递给程序，您可以执行类似于以下操作的操作:

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

/* consider adding parameter for case-insensitive search */
int searchstring (const char *str1, const char *str2)
{
    if (strstr (str1, str2) == NULL)
        return 0;
    else
        return 1;
}

int searchinfile (const char *filename, char *str)
{
    int lineno = 0;
    size_t len = 0;
    char *buffer = NULL;
    FILE *f = fopen (filename, "r");

    if (!f) {   /* validate file open for reading */
        fprintf (stderr, "error: file open failed '%s'.\n", filename);
        return 1;
    }

    /* read each line into buffer, search for searchstring */
    while (getline (&buffer, &len, f) != -1) {
        lineno++;
        if (searchstring (buffer, str))
            printf("[%s:%d] %s", filename, lineno, buffer);
    }
    fclose (f);     /* close file */

    return 0;
}

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

    if (argc < 3) { /* validate at least 2 arguments given */
        fprintf (stderr, "error: insufficient input.\n"
                "usage: %s <file> <searchstring>\n", argv[0]);
        return 1;
    }

    /* search file argv[1] for string argv[2] */
    if (searchinfile (argv[1], argv[2]))
        fprintf (stderr, "unable to search '%s'.\n", argv[1]);

    return 0;
}

输入文件示例

$ cat dat/captnjack.txt
This is a tale
Of Captain Jack Sparrow
A Pirate So Brave
On the Seven Seas.

示例使用/输出

$ /tmp/tmp-david/greplite dat/captnjack.txt Sea
[dat/captnjack.txt:4] On the Seven Seas.

请注意代码中的注释 - 传递附加参数来指示是否应执行不区分大小写的搜索(然后转换行和搜索)将是一个巧妙的技巧 -搜索前将术语转为小写)。

仔细检查一下，如果您还有其他问题，请告诉我。