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

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を引き起こしていました。これが原因で起動できなくなっているようです。

	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;
	}

	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;
	}

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

Uh oh!

接続ホストのヒストリ個数 #382

Description

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions