CERT C: Rec.FIO24-C

既に開いているファイルを開かない

説明

ルール定義

既に開いているファイルを開かないようにします。¹

Polyspace 実装

ルール チェッカーは、"以前に開いたリソースを開く" をチェックします。

例

以前に開いたリソースを開く

問題

以前に開いたリソースを開くでは、既に開いているファイルを開こうとしている関数がチェックされます。

リスク

リソースを複数回開くと、以下が発生する可能性があります。

• ファイル アクセス時の競合状態。

• そのファイルに対する未定義の動作または予期しない動作。

• 異なるターゲットでプログラムを実行するときの移植性の問題。

修正方法

リソースを開いたら、再度開く前にそのリソースを閉じます。

例 - 新しいアクセス許可でファイルを再度開く

#include <stdio.h>
const char* logfile = "my_file.log";

void doubleresourceopen()
{
    FILE* fpa = fopen(logfile, "w");
    if (fpa == NULL) {
        return;
    }
    (void)fprintf(fpa, "Writing");
    FILE* fpb = fopen(logfile, "r"); //Noncompliant
    (void)fclose(fpa);
    (void)fclose(fpb);
}

この例では、この関数の最初の行で書き込み権限を指定して logfile を開きます。この関数の途中で、読み取り権限を指定して logfile を再び開きます。

修正 — 再度開く前に閉じる

考えられる 1 つの修正方法として、異なる権限を指定してファイルを再度開く前に、そのファイルを閉じます。

#include <stdio.h>
const char* logfile = "my_file.log";

void doubleresourceopen()
{
    FILE* fpa = fopen(logfile, "w");
    if (fpa == NULL) {
        return;
    }
    (void)fprintf(fpa, "Writing");
    (void)fclose(fpa);
    FILE* fpb = fopen(logfile, "r");
    (void)fclose(fpb);
}

チェック情報

グループ: Rec.09.入出力 (FIO)

バージョン履歴

R2019a で導入

参考

SEI CERT-C をチェック (-cert-c)

トピック

• コーディング規約違反のチェックおよびレビュー

外部の Web サイト

• FIO24-C

---

¹ This software has been created by MathWorks incorporating portions of: the “SEI CERT-C Website,” © 2017 Carnegie Mellon University, the SEI CERT-C++ Web site © 2017 Carnegie Mellon University, ”SEI CERT C Coding Standard – Rules for Developing safe, Reliable and Secure systems – 2016 Edition,” © 2016 Carnegie Mellon University, and “SEI CERT C++ Coding Standard – Rules for Developing safe, Reliable and Secure systems in C++ – 2016 Edition” © 2016 Carnegie Mellon University, with special permission from its Software Engineering Institute.

ANY MATERIAL OF CARNEGIE MELLON UNIVERSITY AND/OR ITS SOFTWARE ENGINEERING INSTITUTE CONTAINED HEREIN IS FURNISHED ON AN "AS-IS" BASIS. CARNEGIE MELLON UNIVERSITY MAKES NO WARRANTIES OF ANY KIND, EITHER EXPRESSED OR IMPLIED, AS TO ANY MATTER INCLUDING, BUT NOT LIMITED TO, WARRANTY OF FITNESS FOR PURPOSE OR MERCHANTABILITY, EXCLUSIVITY, OR RESULTS OBTAINED FROM USE OF THE MATERIAL. CARNEGIE MELLON UNIVERSITY DOES NOT MAKE ANY WARRANTY OF ANY KIND WITH RESPECT TO FREEDOM FROM PATENT, TRADEMARK, OR COPYRIGHT INFRINGEMENT.

This software and associated documentation has not been reviewed nor is it endorsed by Carnegie Mellon University or its Software Engineering Institute.