Forum: Ruby-dev [Backport 200 - Backport #8064][Assigned] r39687 - io.c (rb update max fd): use ATOMIC CAS のバックポート

Posted by kosaki (Motohiro KOSAKI) (Guest)
on 2013-03-10 05:36
(Received via mailing list)
Issue #8064 has been reported by kosaki (Motohiro KOSAKI).

----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Posted by nagachika (Tomoyuki Chikanaga) (Guest)
on 2013-03-20 14:37
(Received via mailing list)
Issue #8064 has been updated by nagachika (Tomoyuki Chikanaga).


r39687 と一緒に変数の型を変更している r39776 もバックポートしようと思います。
----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064#change-37762

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Posted by kosaki (Motohiro KOSAKI) (Guest)
on 2013-03-24 21:22
(Received via mailing list)
Issue #8064 has been updated by kosaki (Motohiro KOSAKI).


あ、すいません。これちょっと待ってください。
r39687書いたときはちょっと考慮できていなかったのですが、型を変更してしまうとどこが壊れるかわからないので、
素直にmutex導入して、変更最小限修正に出来ないか一度考えたい。
----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064#change-37904

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Posted by nagachika (Tomoyuki Chikanaga) (Guest)
on 2013-04-19 17:38
(Received via mailing list)
Issue #8064 has been updated by nagachika (Tomoyuki Chikanaga).


こちらどうでしょうか。急がなくてもいいと思いますが、r39687 だけ入れていてもあまり意味はないでしょうか?
----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064#change-38760

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Posted by kosaki (Motohiro KOSAKI) (Guest)
on 2013-04-24 07:18
(Received via mailing list)
Issue #8064 has been updated by kosaki (Motohiro KOSAKI).


見なおしたけど r40438 入れれば、触ってる場所一箇所になるから僕的に許せる感じ。
----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064#change-38856

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Posted by nagachika (Tomoyuki Chikanaga) (Guest)
on 2013-04-26 19:10
(Received via mailing list)
Issue #8064 has been updated by nagachika (Tomoyuki Chikanaga).


ありがとうございます。
一旦 r39687, r39776, r40438 をマージして閉じようと思います。
追加で変更があればまた backport チケットを作ってください。
----------------------------------------
Backport #8064: r39687 - io.c (rb_update_max_fd): use ATOMIC_CAS のバックポート
https://bugs.ruby-lang.org/issues/8064#change-38935

Author: kosaki (Motohiro KOSAKI)
Status: Assigned
Priority: Normal
Assignee: nagachika (Tomoyuki Chikanaga)
Category:
Target version:


現状、1.9.3と2.0において、rb_update_max_fdが排他制御されずに呼ばれているので最大FDを間違って更新し、
結果fd leakにつながる可能性があります。
これはsensitive dataのリークにつながるかどうかはスクリプト次第ですが、塞いでおいたほうが心の安全が図れそうです
Please log in before posting. Registration is free and takes only a minute.
Existing account (Switch to SSL-encrypted connection)
NEW: Do you have a Google/GoogleMail or Yahoo account? No registration required!
Log in with Google account | Log in with Yahoo account
No account? Register here.