UF.JNDI

UF (Use Freed) 問題は、リソースが解放された後にリソースを使用する試みがある場合に報告されます。警告 UF.JNDI は、JNDI コンテキストが閉じられた後にこのコンテキストの使用を試みていることを示します。

例 1

25     public void scan(Name name, String... filters) {
26         try {
27             DirContext ctx = initContext();
28             try {
29                 for (final String filter : filters) {
30                     search(name, filter, ctx);
31                 }
32             } finally {
33                 ctx.close();
34             }
35         } catch (NamingException e) {
36             e.printStackTrace();
37         }
38     }
39 
40     private DirContext initContext() throws NamingException {
41         Hashtable env = new Hashtable(11);
42         env.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.ldap.LdapCtxFactory");
43         env.put(Context.PROVIDER_URL, "ldap:///o=JndiSample,dc=example,dc=com");
44 
45         DirContext ctx = new InitialDirContext(env);
46         return ctx;
47     }
48 
49     private void search(Name name, String filter, DirContext ctx) throws NamingException {
50         try {
51             final SearchControls searchControls = new SearchControls();
52             searchControls.setReturningAttributes(necessaryItemAttributes);
53             searchControls.setSearchScope(SearchControls.SUBTREE_SCOPE);
54             final NamingEnumeration<SearchResult> resultNamingEnumeration = ctx.search(name, filter, searchControls);
55         } catch (NamingException e) {
56             ctx.close();
57         }
58     }

UF.JNDI が 30 行目のスニペットについて報告されています。JNDI コンテキスト 'ctx' がメソッド 'scan' の for サイクルでアクセスされています (54 行目を参照)。ところが、56 行目のメソッド 'search' で例外が発生した場合、繰り返しのいずれかで 'ctx' が先に閉じられる可能性があります。

例 2

25     public void scan(Name name, String... filters) {
26         try {
27             DirContext ctx = initContext();
28             try {
29                 for (final String filter : filters) {
30                     search(name, filter, ctx);
31                 }
32             } finally {
33                 ctx.close();
34             }
35         } catch (NamingException e) {
36             e.printStackTrace();
37         }
38     }
39 
40     private DirContext initContext() throws NamingException {
41         Hashtable env = new Hashtable(11);
42         env.put(Context.INITIAL_CONTEXT_FACTORY, "com.sun.jndi.ldap.LdapCtxFactory");
43         env.put(Context.PROVIDER_URL, "ldap:///o=JndiSample,dc=example,dc=com");
44 
45         DirContext ctx = new InitialDirContext(env);
46         return ctx;
47     }
48 
49     private void search(Name name, String filter, DirContext ctx) throws NamingException {
50         final SearchControls searchControls = new SearchControls();
51         searchControls.setReturningAttributes(necessaryItemAttributes);
52         searchControls.setSearchScope(SearchControls.SUBTREE_SCOPE);
53         final NamingEnumeration<SearchResult> resultNamingEnumeration = ctx.search(name, filter, searchControls);
54     }

前節のスニペットが修正されています。メソッド 'search' ではストリームを閉じないようになっています。'scan' メソッドで閉じられるようになりました。このメソッドは、このストリームを開いたメソッドです。一般に、リソースを最初に割り当てたメソッドと同じメソッドでリソースを解放するのが好ましいコーディング手法です。