接続ホストのヒストリ個数 about ffftp

[sayurin]

OSS化当初からヒストリ個数 0 を想定していないコードになっていました。

ffftp/history.c

Lines 103 to 120 in 2bf85c0

	void AddHistoryToHistory(HISTORYDATA *Hist)
	{
	HISTORYDATA *New;
	CheckHistoryNum(1);
	if(FileHist > HistoryNum)
	{
	New = malloc(sizeof(HISTORYDATA));
	if(New != NULL)
	{
	memcpy(New, Hist, sizeof(HISTORYDATA));
	New->Next = HistoryBase;
	HistoryBase = New;
	HistoryNum++;
	}
	}
	return;
	}

ヒストリを追加する前にCheckHistoryNum(1)でスペースを１個分空けます。

ffftp/history.c

Lines 147 to 180 in 2bf85c0

	void CheckHistoryNum(int Space)
	{
	int i;
	HISTORYDATA *Prev;
	HISTORYDATA *Pos;
	HISTORYDATA *Next;
	if(HistoryNum > FileHist-Space)
	{
	/* 残すべきヒストリを探す */
	Pos = HistoryBase;
	Prev = NULL;
	for(i = 0; i < FileHist-Space; i++)
	{
	Prev = Pos;
	Pos = Pos->Next;
	}
	/* いらないヒストリを消す */
	if(Prev == NULL)
	HistoryBase = NULL;
	else
	Prev->Next = NULL;
	while(Pos != NULL)
	{
	Next = Pos->Next;
	free(Pos);
	Pos = Next;
	HistoryNum--;
	}
	}
	return;
	}

内部では FileHist-Space とやっていてunderflowを引き起こしていました。

---

v4.4 で行った #157 リファクタリング後の現在の実装でも

ffftp/history.cpp

Lines 48 to 52 in f2de70b

	void AddHistoryToHistory(HISTORYDATA const& history) {
	if (size_as<int>(histories) == FileHist)
	histories.resize((size_t)FileHist - 1);
	histories.insert(begin(histories), history);
	}

スペースを１個分空ける際にunderflowを引き起こしていました。これが原因で起動できなくなっているようです。