RFR: 8349241: Fix the concurrent execution JVM crash of StringBuilder::append(int/long) [v9]
Shaojin Wen
swen at openjdk.org
Tue Feb 4 23:02:24 UTC 2025
> The following code can reproduce the problem, writing out of bounds causes JVM Crash
>
>
> StringBuilder buf = new StringBuilder();
> buf.append('中');
>
> Thread[] threads = new Thread[40];
> final CountDownLatch latch = new CountDownLatch(threads.length);
> Runnable r = () -> {
> for (int i = 0; i < 1000000; i++) {
> buf.setLength(0);
> buf.trimToSize();
> buf.append(123456789123456789L);
> }
> latch.countDown();
> };
>
> for (int i = 0; i < threads.length; i++) {
> threads[i] = new Thread(r);
> }
> for (Thread t : threads) {
> t.start();
> }
> latch.await();
>
>
> This problem can be avoided by using the value of ensureCapacityInternal directly.
Shaojin Wen has updated the pull request incrementally with one additional commit since the last revision:
Complete the javadoc
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/23427/files
- new: https://git.openjdk.org/jdk/pull/23427/files/9bf1330d..729a29d7
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=23427&range=08
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=23427&range=07-08
Stats: 4 lines in 1 file changed: 4 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/23427.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/23427/head:pull/23427
PR: https://git.openjdk.org/jdk/pull/23427
More information about the core-libs-dev
mailing list