EN JP CN

NPD.GEN.MUST

NPD.GEN.MUST

割り当てられた null ポインターの定数値が逆参照されます

null ポインターを使用してデータにアクセスしようとすると、ランタイムエラーが発生します。有効だと思っていたのに実際は null だと判明したポインターをプログラムが逆参照すると、null ポインター逆参照が起こります。null ポインター逆参照欠陥は、エラー処理または競合状態が無効なためにしばしば起こります。通常、プログラム終了の異常が発生します。ポインターが C/C++ コードにおいて逆参照される前に、null に等しくないことを確認するためにチェックする必要があります。

NPD チェッカーは、null または null ポインターが逆参照されるインスタンスを探します。

NPD.GEN.MUST チェッカーは、null 定数値を割り当てられたポインターが null についてチェックしないでそれを逆参照する関数に後で渡されるかまたは明示的に逆参照される状況にフラグを立てます。

脆弱性とリスク

通常、null ポインターの逆参照は、プロセスに失敗する結果となります。これらの指摘は通常、無効な例外処理が原因で発生します。

軽減と防止

この脆弱性を回避するには、次の操作を実行します。

  • 値を返すすべての関数の結果において null 値についてチェックします
  • すべての外部入力が検証されていることを確認します
  • 変数を明示的に初期化します
  • 異常な例外が正確に処理されていることを確認します

脆弱コード例

1  void xstrcpy(char *dst, char *src) {
2    if (!src) return;
3    dst[0] = src[0];
4  }
5  
6  char global_buf[256];
7  
8  void npd_gen_must(int flag) {
9    char *p = global_buf;
10   if (flag) p = 0; // NULL is assigned to p
11   xstrcpy(p, "Hello");
12 }

条件 'flag' が true の場合に定数 null は 'p' に代入され、関数 xstrcpy への呼び出しにより逆参照されるため、Klocwork はこの例の欠陥にフラグを立てます。この脆弱性が原因で、予期しない結果や意図しない結果となる可能性があります。

修正コード例

1  void xstrcpy(char *dst, char *src) {
2    if (!src) return;
3    if (!dst) return;
4    dst[0] = src[0];
5  }
6  
7  char global_buf[256];
8  
9  void npd_gen_must(int flag) {
10    char *p = global_buf;
11   if (flag) p = 0; 
12   xstrcpy(p, "Hello");
13 }

修正されたコードでは、*dst は 3 行目で null についてチェックされます。

拡張機能

このチェッカーは、Klocwork knowledge base (ナレッジベース) を利用して拡張できます。詳細については、C/C++ 解析のチューニングを参照してください。