From tatenaga ＠ gmail.com Wed May 18 17:41:46 2011 From: tatenaga ＠ gmail.com (yosuke takadate) Date: Wed, 18 May 2011 17:41:46 +0900 Subject: [Ultramonkey-l7-users 403] Re: =?iso-2022-jp?b?bDd2c2QbJEIlVyVtJTslOSQsTW4kQSRrTGRCahsoQg==?= Message-ID: 立石様 高舘です。 パッチ提供ありがとうございました。 適用後1か月以上経過しましたが、l7vsdが落ちる現象は再発しておりません。 頂いたパッチでこの問題は解決したようです。 ご対応頂き、ありがとうございました。 2011年3月24日19:22 TATEISHI Katsuyuki : > 立石です。 > > さきほどのパッチですが、後半のcldataの初期化が漏れてました。 > > TATEISHI Katsuyuki -san wrote: > >>> 以下のようにコードを変更してみましたが、 >>> やはり同じ箇所で落ちてしまうことがわかりました。 >>> バッファは追加で確保した領域以上に格納されていないように見えるのですが・・ >> >> 添付のパッチでいかがでしょうか？ >> >> 1. 問題の本質ではないですが、 X_FORWARDED_FOR_LENGTH は >> include/l7vs_conn.h で定義したほうがよさそうなので、そっ >> ちで定義しています。名前も >> L7VS_X_FORWARDED_FOR_HEADER_LEN としています。 >> >> 2. conn->cldata を malloc() しているところは2個所あるので、 >> 両方で確保するようにしました。 >> >> 3. sizeof(conn->cldata) だと、ポインタのサイズになってしまう >> と思うので、memset()の引数はいじらず、cldata_bufsize その >> ものに l7vs_conn_read_bufsize + L7VS_X_FORWARDED_FOR_HEADER_LEN >> を代入しています。 > > 4. malloc()したバッファの初期化(memset()の)サイズに > l7vs_conn_read_bufsize を用いていたので、 > conn->cldata_bufsize (つまり、l7vs_conn_read_bufsize + > L7VS_CONN_READ_BUFSIZE)で初期化するように変更 > > -- > TATEISHI Katsuyuki >