Kaydet (Commit) f86c07af authored tarafından Caolán McNamara's avatar Caolán McNamara

coverity#1326381 Dereference null return value

Change-Id: I86f5fee9bd77b741536816b6b4d1b6b520acba9e
üst c4e1857d
......@@ -285,23 +285,25 @@ public class ButtonList implements XItemEventBroadcaster, XActionListener
}
for (int i = 0; i < m_aButtons.length; i++)
{
Object[] oResources = renderer.getImageUrls(getObjectFor(i));
if (oResources != null)
Object oObj = getObjectFor(i);
if (oObj == null)
continue;
Object[] oResources = renderer.getImageUrls(oObj);
if (oResources == null)
continue;
if (oResources.length == 1)
{
if (oResources.length == 1)
{
Helper.setUnoPropertyValue(m_aButtons[i].getModel(), PropertyNames.PROPERTY_IMAGEURL, oResources[0]);
}
else if (oResources.length == 2)
{
oUnoDialog.getPeerConfiguration().setImageUrl(m_aButtons[i].getModel(), oResources[0], oResources[1]);
}
boolean bTabStop = Boolean.TRUE; // focusable ? Boolean.TRUE : Boolean.FALSE;
Helper.setUnoPropertyValue(m_aButtons[i].getModel(), "Tabstop", bTabStop);
if (refreshOverNull)
{
setVisible(m_aButtons[i], true);
}
Helper.setUnoPropertyValue(m_aButtons[i].getModel(), PropertyNames.PROPERTY_IMAGEURL, oResources[0]);
}
else if (oResources.length == 2)
{
oUnoDialog.getPeerConfiguration().setImageUrl(m_aButtons[i].getModel(), oResources[0], oResources[1]);
}
boolean bTabStop = Boolean.TRUE; // focusable ? Boolean.TRUE : Boolean.FALSE;
Helper.setUnoPropertyValue(m_aButtons[i].getModel(), "Tabstop", bTabStop);
if (refreshOverNull)
{
setVisible(m_aButtons[i], true);
}
}
}
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment