Re: set.c:entry_new() のメモリ不足チェック漏れ? (Senna-dev 566) - senna

omoさん、始めまして。

パッチ投稿ありがとうございます！！

以前「failmalloc (相当) のおもいで」のブログ記事を
拝見させていただきました。

> なんとなくコードを眺めていたところ、
> set.c で entry_new() 関数の戻り値をチェックしていない場所がありました。
> entry_new() 関数は NULL を返すパスがあるので、
> クラッシュする可能性があるように見えます。
> (実際に calloc() の失敗する環境で senna を使われているケースがあるのかは 
> わかりません。)

> 1.0.4 向けに直してみた diff を添付します。参考まで。

添付のdiffファイルがMLサーバによってカットされてしまいました…

恐らく、entry_newを呼び出ししている3つの部分とその前後が
修正されているものと考えます。

先日のかずひこさんの場合もそうですが、
diffが投げられないと困りますね。

よって、MLの設定を
添付ファイルをWeb上に保存し、
そのURLを添付するような設定と変えました。
（実、設定項目に気づいていませんでした…申し訳ない）

もう一度diffを投稿いただけるとうれしいです！

> ただこれを直すと、芋蔓的に sen_set_get() が NULL を返すことになり、
> そうすると sen_set_get() の呼び出しはほぼ全滅に見えます。
> 何か見落しているかもしれません。指摘ください。

ですね…
パフォーマンス的に問題がなければ全て修正したいな、
と考えております。

---
Tasuku SUENAGA <a****@razil*****>

senna

[Senna-dev 566] Re: set.c:entry_new() のメモリ不足チェック漏れ?