getline 读出来的行是乱码，另外还有泄漏

先说乱码，再说泄漏。

name、role 和 port 都是指向 line 内部 的指针。strtok 只是往那块缓冲区里塞 NUL，然后返回缓冲区内的地址。getline 读下一行时又会复用同一块缓冲区。所以每个 host 最后指向的都是同一块内存，你看到的就是最后一行留在那里的内容。如果 line 被 realloc 了就更糟，旧指针会悬空。

每一行都需要自己的一份拷贝，写法如下。

inv->hosts[inv->count].name = strdup(name);
inv->hosts[inv->count].role = strdup(role);

记得检查 NULL，释放 inventory 的时候两个都要 free。这就是生命周期的边界，line 只活一次迭代，host 字符串则和 inv 活得一样久。

再说泄漏，提前 return -1 的路径（坏行、grow 失败）都跳过了 free(line)，只有正常路径会释放它。要么每次 return 前都 free，要么用 goto done 统一到一个清理出口。

getline 负责分配缓冲区，所以调用方拥有它。POSIX 的页面写得很清楚 https://pubs.opengroup.org/onlinepubs/9699919799/functions/getline.html

先说乱码。strtok 给的是指向 line 内部的指针，所以要用 strdup 拷贝出来。getline 的页面说了每次调用都会重新分配缓冲区，见 https://pubs.opengroup.org/onlinepubs/9699919799/functions/getline.html

谢谢，解决了。提前 return 也用同样的清理方式吗？

done:
	free(line);
	fclose(fp);
	return rc;

同意，用 strdup 拷贝是对的修法，getline 那个页面也确认了缓冲区会被复用。基本和 #2 说的一样。

对，这样可行。每个提前退出的地方都设 rc = -1 然后 goto done。一开始让 line = NULL。

	if (!name || !role) {
		fprintf(stderr, "%s:%d: expected 'name role [port]'\n",
			path, lineno);
		rc = -1;
		goto done;
	}

在每个提前退出的地方设 rc = -1 然后 goto done。

小心，失败时已经拷贝的那些行还留在 inv 里，所以那些也得释放，不是吗？