Skip to content

8360941: [ubsan] MemRegion::end() shows runtime error: applying non-zero offset 8388608 to null pointer #26216

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 9 commits into from
Closed

8360941: [ubsan] MemRegion::end() shows runtime error: applying non-zero offset 8388608 to null pointer #26216

Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
e2407fd
JDK-8360941
MBaesken Jul 9, 2025
d31a4f3
Change the test, avoid using address 0
MBaesken Jul 16, 2025
850e92d
blank
MBaesken Jul 16, 2025
f325d87
blank
MBaesken Jul 16, 2025
842077a
Avoid assertions, align address in test
MBaesken Jul 17, 2025
f0888ce
Simplify test coding
MBaesken Jul 18, 2025
0a9adaa
Suggestion by Thomas Stuefe
MBaesken Jul 21, 2025
53d20a2
Alignment
MBaesken Jul 22, 2025
bf78338
Adjust alignment and freeing
MBaesken Jul 22, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions src/hotspot/share/memory/memRegion.hpp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
/*
* Copyright (c) 2000, 2024, Oracle and/or its affiliates. All rights reserved.
* Copyright (c) 2000, 2025, Oracle and/or its affiliates. All rights reserved.
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
*
* This code is free software; you can redistribute it and/or modify it
Expand Down Expand Up @@ -63,7 +63,8 @@ class MemRegion {
MemRegion minus(const MemRegion mr2) const;

HeapWord* start() const { return _start; }
HeapWord* end() const { return _start + _word_size; }
// in the gtests we call end() with a _start == nullptr so adjust the addition to avoid ub
HeapWord* end() const { return reinterpret_cast<HeapWord*>(reinterpret_cast<uintptr_t>(_start) + (_word_size * sizeof(HeapWord))); }
HeapWord* last() const { return _start + _word_size - 1; }

void set_start(HeapWord* start) { _start = start; }
Expand Down