getline выдаёт мусор, плюс утечка

Сначала мусор, утечка потом.

name, role и port — это указатели внутрь line. strtok просто расставляет NUL в этом буфере и возвращает адреса внутри него. Потом getline переиспользует тот же буфер для следующей строки. В итоге все хосты указывают на одну и ту же память, и вы видите то, что туда записала последняя строка. Если line перевыделится через realloc, будет ещё хуже, потому что старые указатели повиснут.

Каждой строке нужна своя копия:

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

Проверяйте на NULL и освобождайте обе, когда освобождаете inventory. Это граница времени жизни: line живёт одну итерацию, строки хостов живут столько же, сколько inv.

Утечка: ранние ветки return -1 (плохая строка, неудачный grow) пропускают free(line). Освобождается он только на «счастливом» пути. Либо освобождайте перед каждым return, либо делайте 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 при ошибке, их тоже надо освобождать, нет?