チャンク分割処理の想定漏れにより、データが細切れに受信すると、セグメントサーバーに2回以上同時に接続したりするかもしれない

[DaisukeDaisuke]

セグメントサーバー回りについては2重受信ガードを備えてるので、影響はありませんが、メッセージサーバーについては2重以上でメッセージを受信してもおかしくないことに気が付きました。
つまり、セグメントサーバーに異常なまでに多重接続する可能性があるってことです。
データを受信した後に、処理したチャンクを消さない為発生します。
といっても多くの人に使ってもらってる以上、これ以上のバージョン更新は避けた方がいいだろうし、どうしかものか...

ここで常に0が使用されているのが最大の原因です。

MultiCommentViewer/NicoSitePlugin2/Client/BinaryStream.cs

Lines 70 to 77 in f2b562c

	int offset = 0;
	while (true)
	{
	var result = DecodeVarint(ref offset);
	if (result == null)
	{
	break;

ここでオフセットはメモしていますが、使われていません。もっと早く気がつくべきでした...

MultiCommentViewer/NicoSitePlugin2/Client/BinaryStream.cs

Line 91 in f2b562c

_offset = end;

[DaisukeDaisuke]

といっても、セグメントサーバーの2重受信ガードを実装する前でも2重受信はネットワーク遮断時以外発生してなかったので、影響はめちゃくちゃ遅い回線を使ってる人に限られると思われます。

[DaisukeDaisuke]

対策として、

• セグメントサーバーurlに2重受信ガードを設ける
• 上で触れたオフセットの問題を直す

を行います。

[DaisukeDaisuke]

下記のコミットで仮修正済み
2583be8