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

Class.getReflectionFactory() is not thread-safe

    XMLWordPrintable

Details

    • b03
    • generic
    • generic
    • Verified

    Backports

      Description

        A DESCRIPTION OF THE PROBLEM :
        The private method `java.lang.Class.getReflectionFactory()` appears to not be thread-safe.
        It lazily initializes the `reflectionFactory` field but there is no happens-before relationship. Therefore it is allowed by the Java Memory Model that the first read of the `reflectionFactory` reads a non-null value and the second read (as part of the `return`) statement returns null, causing a NullPointerException for the caller.

        See also https://shipilev.net/blog/2016/close-encounters-of-jmm-kind/#wishful-benign-is-resilient


        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-8324850 Class.getReflectionFactory() is not thread-safe

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

            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-8327792 Class.getReflectionFactory() is not thread-safe

            • 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-8183178 Review thread safety of java.lang.reflect.TypeVariable methods

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

            Bug - A problem which impairs or prevents the functions of the product. JDK-8261407 ReflectionFactory.checkInitted() is not thread-safe

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

            Commit Commit openjdk/jdk11u-dev/a26445db

            Commit Commit openjdk/jdk17u-dev/d7c67fc3

            Commit Commit openjdk/jdk/905b7639

            Review Review openjdk/jdk11u-dev/2554

            Review Review openjdk/jdk17u-dev/2153

            Review Review openjdk/jdk/6870

            Activity

              People

                darcy Joe Darcy
                webbuggrp Webbug Group
                Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: