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

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 on 2014年02月24日 19時05分 (#2550908)

        そうですね。
        どっちにしろ2行以上あれば必ず括弧が必要な訳だし、if文書いたら脊髄反射で{}を書いてしまうくらいでちょうどいいです。

        親コメント
        • by Anonymous Coward

          心掛ける以前に、オートインデントのために{}は必須でしょう
          自分でTabキーを押すなんて面倒臭すぎて死んじゃう

          • by Anonymous Coward

            括弧がないくらいでオートインデントできないエディタは
            使うべきじゃないんじゃないかな

アレゲはアレゲを呼ぶ -- ある傍観者

処理中...