Hideaki Kondo
kondo****@oss*****
2008年 12月 9日 (火) 14:44:57 JST
竹林さん、各位 近藤です お疲れ様です。 先に送付のパッチは、安定版Ver.2.0.0-1リリース ソースに対するパッチファイルになっておりましたので、 unstable/sachielに対するパッチを作成しましたので 再送付させていただきます。 ご確認およびご対応の程よろしくお願い致します。 On Tue, 09 Dec 2008 13:34:58 +0900 Hideaki Kondo <kondo****@oss*****> wrote: > > 竹林さま、各位 > > 近藤です。 > お疲れ様です。 > > ご対応有難うございます。 > ログレベルチェックを考慮した最終版パッチを > 作成しましたので、送付させていただきます。 > > カレントログレベルチェックを行い、INFOレベル以下( > つまり、INFO または DEBUGレベルが対象)であれば、 > アクセスログを出力する実装となっております。 > > ご確認の程よろしくお願い致します。 > > お手数をおかけしますが、branchへのマージを > よろしくお願い致します。>竹林さん > > > On Tue, 09 Dec 2008 13:05:59 +0900 > Shinya TAKEBAYASHI <makot****@kanon*****> wrote: > > > 近藤 様 > > > > > > 竹林です. > > お疲れ様です. > > > > 下記承知しました. > > 調査ありがとうございます. > > > > 先ほど口頭でもお話ししましたが,最終版のパッチを頂けますか. > > 竹林が branch へマージします. > > > > ---------------------------------------------------------------- > > Shinya TAKEBAYASHI > > > > E-mail : makot****@kanon***** > > GPG ID : FFD20D1F > > GPG FP : 7B5B E0FC B785 7457 683C 47D6 5564 DDDD FFD2 0D1F > > ---------------------------------------------------------------- > > > > Hideaki Kondo <kondo****@oss*****> wrote in message <20081****@oss***** > > > > > *** Subject: [Ultramonkey-l7-develop 227] Re: UM-L7 アクセスログ出力処理追加につ > > いて > > *** Date: 2008/12/09 11:28:59 > > > > > > 竹林さん、各位 > > > > > > 近藤です。 > > > お疲れ様です。 > > > > > > コメント有難うございます。 > > > > > > > glibc 内の inet_ntoa() の実装ではバッファサイズは 18 バイトになりますが, > > > > この点の考慮は要らないでしょうか. > > > > > > > > xxx.xxx.xxx.xxx なので,16 バイトあれば充分と言えば充分ですが・・・ > > > (snip) > > > > ・・・何故 inet_ntoa のバッファサイズが 18 バイトなんでしょうね. > > > > > > ★ > > > 少し調べてみましたが、例えばFreeBSDにおける実装では、 > > > 「sizeof("aaa.bbb.ccc.ddd")」(つまり16)となっていたり、 > > > > > > ------------------------------------------------------------------ > > > http://fxr.watson.org/fxr/source/libkern/inet_ntoa.c?v=DFBSD > > > > > > static char buf[sizeof "aaa.bbb.ccc.ddd"]; > > > ------------------------------------------------------------------ > > > > > > WINAPIの実装では、以下のようになっていたりしますね。 > > > > > > ------------------------------------------------------------------ > > > http://research.microsoft.com/projects/invisible/src/winshim/inet_ntoa.c.htm > > > > > > static char b[18]; /* perfectly MP safe Berkeley junk, tzk tzk */ > > > ------------------------------------------------------------------ > > > > > > 上記の"perfectly MP safe Berkeley junk"というコメントから > > > 特別な理由があったということではなく、そのまま過去の実装を > > > 踏襲した?結果のように思われます。 > > > > > > Linuxにおけるinet_ntoa()の実装に変更がない限り、(静的に割当 > > > られた領域から逸脱してデータを読み出すようなことをすれば確かに > > > 問題ですが)この領域の範囲内で必要となるIPアドレス文字列数分( > > > sizeof("aaa.bbb.ccc.ddd"))を指定して読み出すことになるだけ > > > ですので、18バイトまでの考慮は不要で16バイトのままで良いと考えます。 > > > > > > もちろん、今後IPv6対応化等において別途考慮は必要になりますが。。。 > > > > > > 特に異論等なければ、このままとさせていただきます。 > > > > > -- > Hideaki Kondo -- Hideaki Kondo -------------- next part -------------- テキスト形式以外の添付ファイルを保管しました... ファイル名: accesslog_addition_svn_diff.patch.gz 型: application/octet-stream サイズ: 733 バイト 説明: 無し ダウンロード