t. Kad
pute.****@gmail*****
2018年 2月 17日 (土) 18:13:07 JST
puteです。 ありがとうございます。報告しておきました。 ただ、以前から報告はあったものの 明確な結論が出ないまま放置されていたようで、 今回の報告で直してもらえるかは未知数だと思っています。 NUM2Uxxxの問題に関しては、今回の報告で何かが決まれば改めて報告しますので、 よろしくお願いいたします。 2018年2月16日 22:28 Kouhei Sutou <kou****@clear*****>: > 須藤です。 > > In <CADz=syPGU****@mail*****> > "[groonga-dev,04594] Re: rroonga における drilldown について" on Fri, 16 Feb 2018 21:14:20 +0900, > "t. Kad" <pute.****@gmail*****> wrote: > >> ちなみに、max_n_sub_records が負の数だと >> StackOverflow で ruby インタプリタが死んだり、処理が戻ってこなくなったりしました。 >> ライブラリ側で引数をチェックして Groonga::InvalidArgument を >> 投げるようにしていただけると嬉しいです。 >> ご一考いただければ幸いです。 > > たしかにチェックしたほうがいいですね。 > > で、コードを見てみるとRubyのCのAPIのNUM2UINT()を使っていまし > た。 > > https://github.com/ranguba/rroonga/blob/master/ext/groonga/rb-grn-table.c#L1493 > >> max_n_sub_records = NUM2UINT(rb_max_n_sub_records); > > -1を指定するとオーバーフローして4294967295になっていたんです > が、NUM2UINT()という名前なので、負の値だったら例外、という > チェックはこの中でやった方がいいんじゃないかなぁと思いました。 > > ということで、Ruby本体に、NUM2UINT()に-1を指定すると例外じゃ > なくてオーバーフローするんだけど例外の方がいいんじゃない？と > フィードバックしておいてもらえませんか？ > > https://bugs.ruby-lang.org/projects/ruby-trunk/issues > > 日本語で大丈夫です。 > > フィードバックするときのガイドラインみたいなものは↓にまとまっ > ています。 > > https://bugs.ruby-lang.org/projects/ruby/wiki/HowToReportJa > > > -- > 須藤 功平 <kou****@clear*****> > 株式会社クリアコード <http://www.clear-code.com/> > > Groongaベースの全文検索システムを総合サポート： > http://groonga.org/ja/support/ > パッチ採用 - プログラミングが楽しい人向けの採用プロセス： > http://www.clear-code.com/recruitment/ > OSS開発支援サービス: > http://www.clear-code.com/blog/2016/6/27.html > > _______________________________________________ > groonga-dev mailing list > groon****@lists***** > https://lists.osdn.me/mailman/listinfo/groonga-dev
