smallkirby / hackmd2hatena Goto Github PK

まぁ全体的にバギーなプログラムではある。lengthをいじることでsecure_hash()でスタックが溢れるうえに、oopsがpanicではないから敢えてoopsさせてleakさせるのもできる。
->
Well, it's a buggy program overall. lengthBy playing secure_hash()with, the stack overflows, and since oops are not panic, you can dare to oops and leak. 

一番の問題は、struct sockのロックを取っていないこと。本来の実装であるtcp_recvmsg()では、内部関数を呼ぶ前にちゃんと　ソケットのロックを取っている。
->
The biggest problem is struct sockthat I haven't locked it. In the original implementation, the tcp_recvmsg()socket is properly locked before calling the internal function.