Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8293691

converting a defined BasicType value to a string should not crash the VM

    XMLWordPrintable

Details

    • b18

    Backports

      Description

        The fix for JDK-8287491 was a little too aggressive in that it modified a test (TestInvalidDebugInfo ) to prevent type2name being called with BasicType::T_ILLEGAL. While T_ILLEGAL models an invalid value and it should not show up in certain contexts, simply converting it to a string should not crash the VM. That is, in the context of type2name, T_ILLEGAL is a valid value.

        I would recommend adopting the fix suggested by [~kbarrett]: https://bugs.openjdk.org/browse/JDK-8287491?focusedCommentId=14498870&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14498870

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8301198 converting a defined BasicType value to a string should not crash the VM

            • P4 - Minor loss of function, or other problem where easy workaround is present.
            • Resolved
            relates to

            Bug - A problem which impairs or prevents the functions of the product. JDK-8287491 compiler/jvmci/errors/TestInvalidDebugInfo.java fails new assert: assert((uint)t < T_CONFLICT + 1) failed: invalid type #

            • P2 - Crashes, loss of data, severe memory leak.
            • Closed
            links to

            Commit Commit openjdk/jdk17u-dev/d1ad5962

            Commit Commit openjdk/jdk/f2a32d99

            Review Review openjdk/jdk17u-dev/1094

            Review Review openjdk/jdk/10447

            Activity

              People

                jsjolen Johan Sjölen
                dnsimon Douglas Simon
                Votes:
                0 Vote for this issue
                Watchers:
                7 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: