From 72f0f535d322850bc1b8d44dd1dc71ebe313470b Mon Sep 17 00:00:00 2001 From: Caolán McNamara Date: Thu, 4 Feb 2016 11:39:46 +0000 Subject: coverity#1326400 Dereference null return value Change-Id: Ie28d817d7be24b2de19f51e11786c0396a2e016f --- javaunohelper/com/sun/star/lib/uno/helper/PropertySet.java | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/javaunohelper/com/sun/star/lib/uno/helper/PropertySet.java b/javaunohelper/com/sun/star/lib/uno/helper/PropertySet.java index 327995824c42..38f704c9a710 100644 --- a/javaunohelper/com/sun/star/lib/uno/helper/PropertySet.java +++ b/javaunohelper/com/sun/star/lib/uno/helper/PropertySet.java @@ -896,9 +896,9 @@ XMultiPropertySet lc= aVetoableLC.getContainer(arEvts[i].PropertyName); else lc= aBoundLC.getContainer(arEvts[i].PropertyName); - if (lc != null) + Iterator it = lc != null ? lc.iterator() : null; + if (it != null) { - Iterator it= lc.iterator(); while( it.hasNext()) { Object listener= it.next(); @@ -913,9 +913,9 @@ XMultiPropertySet lc= listenerContainer.getContainer(XVetoableChangeListener.class); else lc= listenerContainer.getContainer(XPropertyChangeListener.class); - if(lc != null) + it = lc != null ? lc.iterator() : null; + if (it != null) { - Iterator it= lc.iterator(); while(it.hasNext() ) { Object listener= it.next(); @@ -934,9 +934,9 @@ XMultiPropertySet PropertyChangeEvent[] arReduced= new PropertyChangeEvent[nAffectedProps]; System.arraycopy(arEvts, 0, arReduced, 0, nAffectedProps); InterfaceContainer lc= listenerContainer.getContainer(XPropertiesChangeListener.class); - if (lc != null) + Iterator it = lc != null ? lc.iterator() : null; + if (it != null) { - Iterator it= lc.iterator(); while (it.hasNext()) { XPropertiesChangeListener listener = (XPropertiesChangeListener) it.next(); -- cgit