RLK.SQLCON

RLK (リソースリーク) 指摘は、割り当てられたリソースが使用後に適切に破棄されなかった場合に報告されます。リソースが適切に破棄されないと、次のような問題につながる可能性があります。

• 開いているファイルが多すぎます
• 必要なときにアプリケーションが一時ファイルにアクセスできない

RLK.SQLCON は、SQL 接続が終了時に閉じられていないことを示します。

脆弱性とリスク

ストリーム、接続、グラフィックオブジェクトなどのリソースは、明示的に閉じられる必要があります。閉じる操作により、トランザクションをアンブロックすることや、ファイルシステム内のファイルの変更をフラッシュすることができます。リソースは、最終的にはガーベッジコレクタにより閉じられますが、ガーベッジコレクションが開始される前にリソースの枯渇が発生する可能性があります。リソースの性質によっては、別のリソース割り当ての失敗時に、さまざまな例外がスローされます。次のような例外があります：java.io.FileNotFoundException：開かれたファイルが多すぎるか、データベース接続が多すぎます。

軽減と防止

重要でないと考えられるリソースであっても、メソッドを持つリソースはすべて明示的に閉じてください。こうすることで、将来的なコード変更時にもこのようなエラーから安全が保たれます。

例 1

13
14
15
16
17
18
19
20
21
22
23
24
25
     public List<Integer> getActiveIDs() throws SQLException {
         Connection con = DriverManager.getConnection("http://");
         Statement statement = con.createStatement();
         ResultSet rs = statement.executeQuery("SELECT * FROM table");
 
         final List<Integer> result = new ArrayList<Integer>();
         while (rs.next()) {
             if (rs.getBoolean("active")) {
                 result.add(rs.getInt("id"));
             }
         }
         return result;
     }

RLK.SQLCON が 14 行目のスニペットに対して報告されています。 接続 'con' が終了時に閉じられていません。

関連チェッカー

• RLK.SQLOBJ

外部参考資料

• CERT FIO04-J: リソースが不要になったら解放する
• CWE-404: 不適切なリソースのシャットダウンまたはリリース
• CWE-772: 有効な生存期間後のリソースの解放の欠落

拡張機能

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