Pluto
  1. Pluto
  2. PLUTO-391

DefaultPortletPreferenceService#clonePreferences can return a null instead of an empty array and cause a NullPointerException

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 1.1.3
    • Fix Version/s: None
    • Component/s: portlet container
    • Labels:
      None
    • Environment:
      All

      Description

      The DefaultPortletPreferenceService class has a private method clonePreferences() that can return 'null' whereas it should instead return an empty array. The value returned is used in a for-loop in another methd without any check for the array object being null.

        Activity

        Hide
        Ate Douma added a comment -

        Target version 1.2.0 no longer exists do changing to unknown fix version

        Show
        Ate Douma added a comment - Target version 1.2.0 no longer exists do changing to unknown fix version
        Hide
        Marc Veary added a comment -

        Actually, it won't be an issue for an implementation (no default) of the PortletPreferencesService - end-of-the-week

        Show
        Marc Veary added a comment - Actually, it won't be an issue for an implementation (no default) of the PortletPreferencesService - end-of-the-week
        Hide
        Marc Veary added a comment -

        If null is returned from a call to getStoredPreferences() then a NullPointerException will be thrown the constructor of the PortletPreferencesImpl class. It could also present issues to any implementation of the PortletPreferencesService optional container service.

        Show
        Marc Veary added a comment - If null is returned from a call to getStoredPreferences() then a NullPointerException will be thrown the constructor of the PortletPreferencesImpl class. It could also present issues to any implementation of the PortletPreferencesService optional container service.

          People

          • Assignee:
            Unassigned
            Reporter:
            Marc Veary
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:

              Development