http://www.ipa.go.jp/security/awareness/vendor/programming/b06_03.html

このコード、argcをチェックしないでargvにアクセスしてるとか。

悪いコードの例だけど、筆者がとくにそれに言及してないってことは、
その部分は悪いとは思ってないのか?
(malloc()エラー処理を省くって断ってるのに)

あと、文字列の0埋めとか、コメントもダメなコメントの例にでそうな書き方だよね。