「Linux カーネルの zero-day exploit コード、リリースされる」記事へのコメント

バグ？仕様？

[Anonymous Coward]

少し詳しい解説 [blogspot.com]によると、GCCの最適化によりnullチェックのif文が外されてしまうそうで。面白い

Re:

[Anonymous Coward]

良く分からないなぁ。

struct sock *sk = tun->sk;

tun が NULL なら tun->sk を参照した時点で落ちるんじゃないの？
その時点で後ろの if ブロックが削られようが削られまいが何も問題ないと思うんだけど。
それとも sk のオフセットはメモリアクセスできちゃうエリアに到達するほど大きい値？

struct sock *sk = &tun->sk;

なら分からないでもないけど、それだとGCCの問題な気が・・・

Re:バグ？仕様？

[Anonymous Coward]

すいません、自己解決。カーネルなら tun が NULL でも tun->sk を参照しても落ちるわけありませんね。
NULL 参照が許されるような環境でそんな最適化を行ってしまう GCC にも問題がある気がします。

どこが問題か考えるとちょっと複雑

[Anonymous Coward]

シロウトの印象だと、必要な処理をすっ飛ばしちゃうGCCの最適化手法が
おかしいんだろうとは思うんですが、ソースコードベースで
ソフトウェアを公開するとなるとけっこうめんどくさい問題ですね、これ。

GCCの挙動がもし「仕様」（速いコードを吐く代わりに妙なところで命令を
すっ飛ばすことがある）だとしたら、コンパイラやコンパイルオプションを選んだ人にも
問題がないわけでもなく、でもそうすると「選んだ人」って誰ということにもなり、
あとはリリースエンジニアリング的なトコにもツッコミどころがあるのかなとか、
いろいろ考えられますよね。

でも少なくとも心情的には「コードが間違ってる」とは言いがたいなあ。

Re:どこが問題か考えるとちょっと複雑

[Kazsa (25846)]

通常のユーザープロセスで動かす分には、最近の環境ならNULLアクセスしたら大抵トラップ食らってプログラム終了ですから、そういう前提 (NULLアクセスしたらカーネルが止めてくれる) で最適化している可能性はあるかも。

カーネルコードとか組込み機器とかでは、そもそもNULLアクセスで止まる保証がないわけで、そんなところでGCCの最適化を盲目的に適用するのは危険という教訓になるんですかね。

Re:

[Anonymous Coward]

> カーネルコードとか組込み機器とかでは、そもそもNULLアクセスで止まる保証がないわけで、そんなところでGCCの最適化を盲目的に適用するのは危険という教訓になるんですかね。

ヌル参照はANSI Cでは未定義になっているわけです。今回の話も仕様に従った正しい動作です。
gccの最適化により潜在的なバグが顕在化したということですから、gccを責めるのはお門違いです。

Re:バグ？仕様？

[rti (659)]

NULLでもカーネルなら落ちないから大丈夫です(キリッ
って感じのコードを書くのそもそもどうかと思うなー。
gccも悪いかもしれないけど、カーネルハカーももう少し注意してコードを書いてほしいと思う。

よくわからんから勘違いしているかもしれんけど、、、
なんかエラーでたけど @つけたら大丈夫でしたっていうphpプログラマや、
try{}catch(...){} で余裕でしたとかいう C++ プログラマーの香りがする。
#間違っていたら突っ込みください。あやまります。ごめんご。

--
by rti.

Re:

[Anonymous Coward]

> NULL 参照が許されるような環境でそんな最適化を行ってしまう GCC にも問題がある気がします。

ANSI CではNULL参照の動作は未定義だから(たぶん)、脆弱性を引き起こすのも仕様に合致している。
gccは悪くない、悪くないよ。