[ruby-trunk - Bug #6147][Open] should we check a result returned by fcntl?

Issue #6147 has been reported by Yusuke E…


Bug #6147: should we check a result returned by fcntl?

Author: Yusuke E.
Status: Open
Priority: Normal
Assignee: Motohiro KOSAKI
Category:
Target version:
ruby -v: ruby 2.0.0dev (2012-03-14 trunk 35017) [i686-linux]

たぶん kosaki さん (それともささださん?)

thread_pthread.c の rb_thread_create_timer_thread で

fcntl(timer_thread_pipe[1], F_SETFL, oflags);

の返り値チェックしなくていいの?ほかの fcntl の呼び出しでは
全部チェックしてるよ?って Coverity Scan さんが言ってます。

ユーザランドの開発者的にはここで fcntl で失敗するとかどうしろ
というの。。。と思いそうですが、システムコールの返り値は全部
チェックしろよ派のカーネル開発者的にはどうでしょうか。


Yusuke E. [email protected]

Issue #6147 has been updated by kosaki (Motohiro KOSAKI).

単に失敗したら死ねばいいんじゃないですかね。

  • どうせ失敗しないだろう
  • 失敗しても動くようなコードかけないだろう
    の2つの意味で

時間見つけてやっときます

Bug #6147: should we check a result returned by fcntl?

Author: mame (Yusuke E.)
Status: Assigned
Priority: Normal
Assignee: kosaki (Motohiro KOSAKI)
Category:
Target version:
ruby -v: ruby 2.0.0dev (2012-03-14 trunk 35017) [i686-linux]

たぶん kosaki さん (それともささださん?)

thread_pthread.c の rb_thread_create_timer_thread で

fcntl(timer_thread_pipe[1], F_SETFL, oflags);

の返り値チェックしなくていいの?ほかの fcntl の呼び出しでは
全部チェックしてるよ?って Coverity Scan さんが言ってます。

ユーザランドの開発者的にはここで fcntl で失敗するとかどうしろ
というの。。。と思いそうですが、システムコールの返り値は全部
チェックしろよ派のカーネル開発者的にはどうでしょうか。


Yusuke E. [email protected]