パスワードを忘れた? アカウント作成
この議論は賞味期限が切れたので、アーカイブ化されています。 新たにコメントを付けることはできません。

iOS7.0.6で修正された「最悪のセキュリティバグ」はありがちなコーディングミスで発生していた」記事へのコメント

  • 警告を無視しちゃいかんという教訓を得たと。

    こんな感じに、if分を一行にまとめてしまえば、こんな不具合出なかった気がします。

    if ((err = SSLHashSHA1.update(&hashCtx, &serverRandom)) != 0) goto fail;
    if ((err = SSLHashSHA1.update(&hashCtx, &signedParams)) != 0) goto fail;
    if ((err = SSLHashSHA1.final(&hashCtx, &hashOut)) != 0) goto fail;

    一行が長くなるのを嫌ったのかな?

    • Re: (スコア:3, すばらしい洞察)

      by Anonymous Coward

      一行にまとめるコーディングは嫌いですし、職業プログラマーはあまり使うべきではないとも思っています。
      なので対処はif分の処理が1行でも必ず括弧で括る事です。

      • by Anonymous Coward

        スタイルの問題ではないでしょこれは。
        こういう重要な箇所くらいテストで100%のカバレッジを確保しろよと。
        どーでもいいとこに心血注ぐ必要はないけどさ。

身近な人の偉大さは半減する -- あるアレゲ人

処理中...