From tanuma.kouhei @ nttcom.co.jp Tue Aug 3 15:08:42 2010 From: tanuma.kouhei @ nttcom.co.jp (Kohei TANUMA) Date: Tue, 03 Aug 2010 15:08:42 +0900 Subject: [Ultramonkey-l7-develop 630] Re: =?iso-2022-jp?b?ZGVidWcgGyRCJW0lMCRHGyhCIHNlZ3Y=?= In-Reply-To: <20100802.195739.724810403336758117.tateishi.katsuyuki@oss.ntt.co.jp> References: <4C5672C5.5060303@nttcom.co.jp> <20100802.195739.724810403336758117.tateishi.katsuyuki@oss.ntt.co.jp> Message-ID: <4C57B26A.2010801@nttcom.co.jp> 田沼です。 > だとしたら、他の l7vs_conn_is_rs_connected() 呼び出し箇所は > 大丈夫でしょうか?(同様に未初期化のポインタが渡ることはない > ですか?) l7vs_conn_is_rs_connected が呼ばれるのは全体で 2 箇所で 今回の場所と、l7vs_conn_client_receiving 関数から呼ばれる箇所があり そちらの方は変数宣言時に初期化が行われていることを確認しました。 全てのカテゴリを debug にして簡単な確認をしましたが、 ダウンするなどの不具合は見られませんでした。 (いつも確認が甘いのは否めませんが…) 以上、宜しくお願い致します。 From tateishi.katsuyuki @ oss.ntt.co.jp Tue Aug 3 15:28:57 2010 From: tateishi.katsuyuki @ oss.ntt.co.jp (TATEISHI Katsuyuki) Date: Tue, 03 Aug 2010 15:28:57 +0900 (JST) Subject: [Ultramonkey-l7-develop 631] Re: =?iso-2022-jp?b?ZGVidWcgGyRCJW0lMCRHGyhCIHNlZ3Y=?= In-Reply-To: <4C57B26A.2010801@nttcom.co.jp> References: <4C5672C5.5060303@nttcom.co.jp> <20100802.195739.724810403336758117.tateishi.katsuyuki@oss.ntt.co.jp> <4C57B26A.2010801@nttcom.co.jp> Message-ID: <20100803.152857.643680897113221665.tateishi.katsuyuki@oss.ntt.co.jp> 立石です。 Kohei TANUMA -san wrote: >> だとしたら、他の l7vs_conn_is_rs_connected() 呼び出し箇所は >> 大丈夫でしょうか?(同様に未初期化のポインタが渡ることはない >> ですか?) > > l7vs_conn_is_rs_connected が呼ばれるのは全体で 2 箇所で > 今回の場所と、l7vs_conn_client_receiving 関数から呼ばれる箇所があり > そちらの方は変数宣言時に初期化が行われていることを確認しました。 > > 全てのカテゴリを debug にして簡単な確認をしましたが、 > ダウンするなどの不具合は見られませんでした。 > (いつも確認が甘いのは否めませんが…) なるほど。了解しました。 OKです。 一応、2.1.3-1にこれだけあててリリースする可能性も考慮し、 2.1.3-0 からブランチを切ってもらえますか? 現在、 [2.1.3-0]-*-*-*-[master/maint] \ *-[2.1.3-1] という状況であり、master, maint からブランチ切ると2.1.3-1にマー ジしたとき余計なコミットまで含んでしまうことになるので。 2.1.3-1の後ろに伸ばしてもいいですが、2.1.3-1のRAは暫定対処的 な色合いが強く、もしかしたら ours で master/maint にマージす る可能性もあります。なので、今回のパッチは master/maint にも 2.1.3-1 にもマージできるよう、共通祖先である 2.1.3-0 からブラ ンチしてもらうとマージがスムーズにいくはずです。 -- TATEISHI Katsuyuki > > 以上、宜しくお願い致します。 > > >