Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8275301 Unify C-heap buffer overrun checks into NMT
  3. JDK-8275320

NMT should perform buffer overrun checks

    XMLWordPrintable

Details

    • b26

    Backports

      Description

        As laid out in JDK-8275301, it makes sense to let NMT do buffer overflow checking.

        - it would give us C-heap overflow checking in release builds
        - the additional costs are neglectable
        - NMT needs intact headers anyway. Faced with buffer overwrites, today it may crash or cause other errors
        - it is a preparation for future code removal (the memory guarding done in debug only in os::malloc() and friends, and possibly the guarding done with CheckJNICalls)

        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-8299799 NMT should perform buffer overrun checks

            • 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-8298003 NMT: fatal block printout does not show the correct corruption address

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

            Sub-task - The sub-task of the issue JDK-8277822 Remove debug-only heap overrun checks in os::malloc and friends

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

            Commit Commit openjdk/jdk17u-dev/6f115029

            Commit Commit openjdk/jdk/cf7adae6

            Review Review openjdk/jdk17u-dev/842

            Review Review openjdk/jdk/5952

            Activity

              People

                stuefe Thomas Stuefe
                stuefe Thomas Stuefe
                Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: