Kuali Rice Development
  1. Kuali Rice Development
  2. KULRICE-12260

Infinite loop in permission lookupable helper service

    Details

    • Type: Bug Fix Bug Fix
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Complete
    • Affects Version/s: None
    • Fix Version/s: 2.4
    • Component/s: Development
    • Security Level: Public (Public: Anyone can view)
    • Similar issues:
      KULRICE-8211Potential Issue in BusClientFailoverProxy causing infinite loop
      KULRICE-6708BaseInvocationHandler toString method infinite loop
      KULRICE-3806JPA - Deal with infinite flush loop issue
      KULRICE-4152KIM allows you to create infinite loops of role memberships
      KULRICE-13349Investigate infinite loop with anchors in HtmlUnit tests
      KULRICE-3630infinite sizing loop in firefox caused by bug in page.tag
      KULRICE-2601BusinessObjectDictionaryServiceImpl.performForceUppercase susceptible to infinite recursion if cycles present in BO relationships
      KULRICE-4597remove the lookup permission for pessimistic lock since the lookup helper has built-in permission checking.
      KULRICE-12272Inifnite Loop in KNS EditablePropertiesHistoryHolder
      KULRICE-1149improve flexibility of link configuration on lookups, so folks aren't needing to add custom lookupable helper services do frequently
    • Rice Module:
      KIM
    • KAI Review Status:
      Not Required
    • KTI Review Status:
      Not Required
    • Code Review Status:
      Not Required
    • Contributing Institution:
      Indiana Univ
    • Include in Release Notes?:
      Yes

      Description

      When the permission service looks up permissions by principal it is using some logic to search for the parent roles of the roles the user is in. That code has a logic error which is using a "roleId" property to search for roles, but that gets removed since it isn't a valid search criteria. This causes the search to return every role in the database and the iterate through them for processing and this leads to an infinite loop. The fix is to change the "roleId" criteria to "id". This can bring down a Rice server after a few minutes of processing.

        Issue Links

          Activity

          Hide
          Shravya Saripella (Inactive) added a comment -

          Contribution complete.

          Show
          Shravya Saripella (Inactive) added a comment - Contribution complete.

            People

            • Assignee:
              Shravya Saripella (Inactive)
              Reporter:
              James Bennett
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Structure Helper Panel