matsumotory / ngx_mruby

ngx_mruby - A Fast and Memory-Efficient Web Server Extension Mechanism Using Scripting Language mruby for nginx
https://ngx.mruby.org/
988 stars 112 forks source link

FIX some memory errors by valgrind #516

Closed pyama86 closed 1 year ago

pyama86 commented 1 year ago

@matsumotory Please review. refs: https://github.com/matsumotory/ngx_mruby/pull/515

valgrindで検定されているメモリのエラーを修正します。 Nginx.redirectのループについてはコンフィグエラーなので、finalizeでケアしなくてもいいかなと思いました。

Pull-Request Check List

pyama86 commented 1 year ago

原因はよくわからないけど、opensslのbuildがうまくいかなくなってるので、明日また見ます。

pyama86 commented 1 year ago

Actionsのインスタンスが大きくなったっぽくて、いままで1スレッドで動いてたmakeが2で動くようになって、ビルドが正常に終了しなくなっていたので、1に固定した。

matsumotory commented 1 year ago

Thanks! finalizeは他のredirectモジュールが複数実行されたときの兼ね合いでやった気がするけど、今あるテストで問題でてないなら大丈夫そう。