glassfish
  1. glassfish
  2. GLASSFISH-3618

static weaving of "large" classes can fail

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: 9.0pe
    • Fix Version/s: not determined
    • Component/s: entity-persistence
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      3,618
    • Status Whiteboard:
      Hide

      as91ur1-na as911-na

      Show
      as91ur1-na as911-na

      Description

      When performing static weaving the class bytes are read from a stream
      incorrectly allocating an input array using the "available()" method on the stream.

      Per the InputStream javadoc:
      "It is never correct to use the return value of this method to allocate a buffer
      intended to hold all data in this stream."

      Sometimes when loading large classes, some of the bytes aren't read causing
      weaving to fail.

      [TopLink Severe]: 2007.09.17
      07:51:15.015-ServerSession(914691)-java.lang.ArrayIndexOutOfBoundsException: 101
      at oracle.toplink.libraries.asm.ClassReader.readUTF8(ClassReader.java:906)
      at oracle.toplink.libraries.asm.ClassReader.accept(ClassReader.java:291)
      at
      oracle.toplink.essentials.internal.weaving.TopLinkWeaver.transform(TopLinkWeaver.java:107)
      at
      oracle.toplink.essentials.internal.weaving.StaticWeaveClassTransformer.transform(StaticWeaveClassTransformer.java:130)
      at
      oracle.toplink.essentials.weaving.StaticWeaveProcessor.process(StaticWeaveProcessor.java:299)
      at
      oracle.toplink.essentials.weaving.StaticWeaveProcessor.performWeaving(StaticWeaveProcessor.java:181)
      at oracle.toplink.essentials.weaving.StaticWeave.start(StaticWeave.java:135)
      at oracle.toplink.essentials.weaving.StaticWeave.main(StaticWeave.java:107)

        Activity

        Hide
        epricer added a comment -

        Created an attachment (id=1151)
        proposed patch

        Show
        epricer added a comment - Created an attachment (id=1151) proposed patch
        Hide
        gfbugbridge added a comment -

        <BT6606316>

        Show
        gfbugbridge added a comment - <BT6606316>
        Hide
        sirius2k added a comment -
            • Issue 3618 has been confirmed by votes. ***
        Show
        sirius2k added a comment - Issue 3618 has been confirmed by votes. ***
        Hide
        tware added a comment -

        Updating target milestone

        Show
        tware added a comment - Updating target milestone
        Hide
        Mitesh Meswani added a comment -

        Will not be fixed for V2.1

        Show
        Mitesh Meswani added a comment - Will not be fixed for V2.1
        Hide
        sanandal added a comment -

        "Reclassifying as P4 because this issue is not deemed "must fix" for this v2.1
        release whose primary release driver is SailFin.
        This issue will be scrubbed after this release and will be given the right
        priority for the next release."

        Show
        sanandal added a comment - "Reclassifying as P4 because this issue is not deemed "must fix" for this v2.1 release whose primary release driver is SailFin. This issue will be scrubbed after this release and will be given the right priority for the next release."
        Hide
        Tom Mueller added a comment -

        Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

        Show
        Tom Mueller added a comment - Bulk change to set fix version to "not determined" where the issue is open but the value is for a released version.

          People

          • Assignee:
            tware
            Reporter:
            epricer
          • Votes:
            3 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: