From: <sc...@hy...> - 2010-01-21 23:55:49
|
Author: scottmf Date: 2010-01-21 15:55:40 -0800 (Thu, 21 Jan 2010) New Revision: 14215 URL: http://svn.hyperic.org/?view=rev&root=Hyperic+HQ&revision=14215 Modified: branches/HQ_4_2_0_PATCH/src/org/hyperic/hq/appdef/server/session/ApplicationManagerEJBImpl.java Log: [HHQ-3676] fix potential NPE Modified: branches/HQ_4_2_0_PATCH/src/org/hyperic/hq/appdef/server/session/ApplicationManagerEJBImpl.java =================================================================== --- branches/HQ_4_2_0_PATCH/src/org/hyperic/hq/appdef/server/session/ApplicationManagerEJBImpl.java 2010-01-21 23:55:14 UTC (rev 14214) +++ branches/HQ_4_2_0_PATCH/src/org/hyperic/hq/appdef/server/session/ApplicationManagerEJBImpl.java 2010-01-21 23:55:40 UTC (rev 14215) @@ -411,6 +411,11 @@ List rtn = new ArrayList(services.size()); for (Iterator it=services.iterator(); it.hasNext(); ) { AppServiceValue val = (AppServiceValue) it.next(); + if (val == null || val.getService() == null || + val.getService().getResource() == null || + val.getService().getResource().isInAsyncDeleteState()) { + continue; + } rtn.add(val.getService().getResource()); } return rtn; |