[JS2-472] FragmentImpl.getFragments() can be null but is assumed not null

• Type: Bug
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 2.1.3, 2.2.0
• Fix Version/s: 2.1.3, 2.2.0
• Assignee: W. Randall Watler
• Reporter: David Jencks
• Created: 19/Jan/06 17:56
• Updated: 04/Oct/11 21:08
• Resolved: 17/Apr/09 06:52

Description

org.apache.jetspeed.om.page.psml.FragmentImpl.getFragments() can return null if all fragments fail the security check: see line 560.

However, ContentFragmentImpl assumes that a non-null value is returned: for instance in ContentFragmentList.duplicateList().

I assume the best fix is to always return a list, even if empty.

I will supply a patch in the next day or two.