On Mon, 24 Mar 2025 13:57:31 GMT, Zdenek Zambersky <zzamb...@openjdk.org> wrote:
>> This fixes stack smashing issue in awt library on linux s390x. (jdk compiled >> with `-fstack-protector-strong`) >> >> Fix is based on patch >> [submitted](https://mail.openjdk.org/pipermail/awt-dev/2019-July/015337.html) >> in JDK-8227919 review thread, rebased to master. They decided to go for >> Solaris only fix there, with [expected >> follow-up](https://mail.openjdk.org/pipermail/awt-dev/2019-July/015347.html) >> for linux/s390x. But that never happened. >> >> I was not able to get response from original author for this issue, so I am >> creating PR myself. >> >> **Testing:** >> I tested jdk with and without this fix on linux/s390x (using Xvfb). It fixes >> the issue (reproducer for this bug no longer crashes the JVM). I have also >> tried to run `jdk_awt` tests, where lot of tests was also affected by this. >> With this patch amount of failures dropped by ~100. >> >> Without fix: >> >> TEST TOTAL PASS FAIL ERROR >> SKIP >>>> jtreg:test/jdk:jdk_awt 2171 1063 415 6 >>>> 687 << >> >> >> With fix >> >> TEST TOTAL PASS FAIL ERROR >> SKIP >>>> jtreg:test/jdk:jdk_awt 2171 1162 316 6 >>>> 687 << >> >> >> Many tests are no longer crashing the VM: >> >> -java/awt/Choice/ChoiceKeyEventReaction/ChoiceKeyEventReaction.java >> Failed. Unexpected exit from >> test [exit code: 134] >> +java/awt/Choice/ChoiceKeyEventReaction/ChoiceKeyEventReaction.java >> Passed. Execution successful >> >> There seem to be few unstable tests in jdk_awt. (Ones that fail in one or >> another run, but seem unrelated to this fix) >> >> I have not done any additional manual testing of XIM as did original author >> (in review thread mentioned higher), as I am not at all familiar with input >> method for Japanese (or other Asian characters), or how that supposed to >> work. (So I cannot verify that input of Asian characters works as expected >> on linux/s390x, but at least JVM no longer crashes.) > > Attaching jdk_awt jtreg results (with and without fix). > [jdk-awt-results.tgz](https://github.com/user-attachments/files/19430946/jdk-awt-results.tgz) > @zzambers Sorry for my late reaction. Is it possible to add test result by > using `XMODIFIERS=@im=local` ? Thank you for you reaction. I can do some additional testing. You mean to just run the tests with `XMODIFIERS` env. variable set to given value? Are there any other requirements about test system? Note that I only have access to headless linux/s390x machines, so I used Xvfb as X11 server. ------------- PR Comment: https://git.openjdk.org/jdk/pull/24197#issuecomment-2824641158