界: Code Quality

コードの質が低いと、予測できない動作につながります。ユーザーの視点には、それがしばしば使い勝手の悪さとなって現れます。攻撃者にとっては、予期せぬ方法でシステムにストレスを与える機会となります。

Poor Style: Redundant Initialization

Abstract

変数の値が割り当てられているにもかかわらず、一度も使用されていない場合、それは無駄な格納領域になります。

Explanation

この変数の最初の値は使用されません。初期化後は、変数には別の値が割り当てられるか、スコープ外に移ります。

例 1: 次のコード引用では、変数 r に割り当ててから、それを使用することなく、値を上書きしています。


  int r = getNum();
  r = getNewNum(buf);

References

[1] Standards Mapping - Common Weakness Enumeration CWE ID 398

[2] Standards Mapping - Security Technical Implementation Guide Version 3.1 APP3050 CAT II

[3] Standards Mapping - Security Technical Implementation Guide Version 3.4 APP3050 CAT II

[4] Standards Mapping - Security Technical Implementation Guide Version 3.5 APP3050 CAT II

[5] Standards Mapping - Security Technical Implementation Guide Version 3.6 APP3050 CAT II

[6] Standards Mapping - Security Technical Implementation Guide Version 3.7 APP3050 CAT II

[7] Standards Mapping - Security Technical Implementation Guide Version 3.9 APP3050 CAT II

[8] Standards Mapping - Security Technical Implementation Guide Version 3.10 APP3050 CAT II

desc.structural.cpp.poor_style_redundant_initialization

Abstract

変数の値が割り当てられているにもかかわらず、一度も使用されていない場合、それは無駄な格納領域になります。

Explanation


  int r = getNum();
  r = getNewNum(buf);