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

RelocationHolder constructors have bugs

    Details

    • Subcomponent:
    • CPU:
      generic
    • OS:
      generic

      Description

      RelocationHolder has a _relocbuf member, which is really just storage for a Relocation object. The constructors for RelocationHolder are both problematic. The no-arg constructor is

      RelocationHolder::RelocationHolder() {
        new(*this) Relocation();
      }

      This is all very contorted and fragile. I wonder why RelocationHolder doesn't just use placement new to (default) construct the Relocation object. e.g.

      new (_relocbuf) Relocation();

      The other constructor is

      RelocationHolder::RelocationHolder(Relocation* r) {
        // wordwise copy from r (ok if it copies garbage after r)
        for (int i = 0; i < _relocbuf_size; i++) {
          _relocbuf[i] = ((void**)r)[i];
        }
      }

      and that comment is just wrong, since the actual object could have been allocated close to the end of accessible memory, with a read beyond its real end potentially resulting in some kind of memory fault.

        Attachments

          Issue Links

          duplicates

          Sub-task - The sub-task of the issue JDK-8160354 uninitialized value warning and VM crash are occurred with GCC 6

          • P3 - Major loss of function.
          • Closed
          relates to

          Bug - A problem which impairs or prevents the functions of the product. JDK-8160310 HotSpot cannot be built with GCC 6

          • P3 - Major loss of function.
          • Resolved

          Bug - A problem which impairs or prevents the functions of the product. JDK-8187676 Disable uninitialized warnings for two files until proper fix available

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

          Enhancement - null JDK-8253311 Cleanup relocInfo constructors

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

          Sub-task - The sub-task of the issue JDK-8160354 uninitialized value warning and VM crash are occurred with GCC 6

          • P3 - Major loss of function.
          • Closed

            Activity

              People

              • Assignee:
                rraghavan Rahul Raghavan
                Reporter:
                kbarrett Kim Barrett
              • Votes:
                0 Vote for this issue
                Watchers:
                8 Start watching this issue

                Dates

                • Created:
                  Updated: