glassfish
  1. glassfish
  2. GLASSFISH-452

infinite recursive loop reported by findbugs

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 9.0pe
    • Fix Version/s: 9.0pe
    • Component/s: entity-persistence
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      452

      Description

      Jerome observed this in findbugs report and sent me an email. I am filing after
      after confirming with Marina that this is a new issue and not captured by issue
      #268. This should be fixed by 3/27, hence a p3.
      ----------
      There is an apparent infinite recursive loop in
      oracle.toplink.essentials.internal.sessions.AbstractSession.getClassDescriptorForAlias(String).

      Bug type IL_INFINITE_RECURSIVE_LOOP (click for details)
      <http://javaweb.sfbay/java/re/glassfish/9.0/nightly/findbugs/snapshot/Mar212006fb.html#IL_INFINITE_RECURSIVE_LOOP>
      In class oracle.toplink.essentials.internal.sessions.AbstractSession
      In method
      oracle.toplink.essentials.internal.sessions.AbstractSession.getClassDescriptorForAlias(String)
      At AbstractSession.java:[line 1221]

      IL There is an apparent infinite recursive loop in
      oracle.toplink.essentials.sessions.Project.getClassDescriptorForAlias(String).

      Bug type IL_INFINITE_RECURSIVE_LOOP (click for details)
      <http://javaweb.sfbay/java/re/glassfish/9.0/nightly/findbugs/snapshot/Mar212006fb.html#IL_INFINITE_RECURSIVE_LOOP>
      In class oracle.toplink.essentials.sessions.Project
      In method
      oracle.toplink.essentials.sessions.Project.getClassDescriptorForAlias(String)
      At Project.java:[line 778]
      ------------

        Activity

        Hide
        marina vatkina added a comment -

        This bug is a typo duplicated by copy-and-paste excercise:
        public ClassDescriptor getClassDescriptorForAlias(String alias) {
        ClassDescriptor desc = getClassDescriptorForAlias(alias);

        should be
        ClassDescriptor desc = getDescriptorForAlias(alias);

        Show
        marina vatkina added a comment - This bug is a typo duplicated by copy-and-paste excercise: public ClassDescriptor getClassDescriptorForAlias(String alias) { ClassDescriptor desc = getClassDescriptorForAlias(alias); should be ClassDescriptor desc = getDescriptorForAlias(alias);
        Hide
        guypelletier added a comment -

        I'll correct this ...

        Show
        guypelletier added a comment - I'll correct this ...
        Hide
        guypelletier added a comment -

        Fix available in next persistence drop.

        Show
        guypelletier added a comment - Fix available in next persistence drop.
        Hide
        pkrogh added a comment -

        This was accidentally closed. It is not currently fixed in CVS. Fixed locally.

        Show
        pkrogh added a comment - This was accidentally closed. It is not currently fixed in CVS. Fixed locally.
        Hide
        tware added a comment -

        Fixed in latest drop

        Show
        tware added a comment - Fixed in latest drop
        Hide
        marina vatkina added a comment -

        Added build #

        Show
        marina vatkina added a comment - Added build #

          People

          • Assignee:
            tware
            Reporter:
            Sanjeeb Sahoo
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: