RFR: 8345757: [ASAN] clang17 report dprintf macro redefined
SendaoYan
syan at openjdk.org
Sun Dec 8 11:55:17 UTC 2024
Hi all,
File src/java.smartcardio/share/native/libj2pcsc/pcsc.c and src/jdk.crypto.cryptoki/share/native/libj2pkcs11/j2secmod.h generate compile warning `'dprintf' macro redefined` by clang17/llvm17 with fastdebug configure. This PR undefine `dprintf` before define dprintf macro, to avoid deplication macro definition to linux header `/usr/include/bits/stdio2.h`, risk is low.
Additional testing:
- [ ] build with fastdebug and release configure by gcc14 on linux-x64
- [ ] build with fastdebug and release configure by gcc14 on linux-aarch64
- [ ] jtreg tests(include tier1/2/3) on linux-x64 with release build
- [ ] jtreg tests(include tier1/2/3) on linux-x64 with fastdebug build
- [ ] jtreg tests(include tier1/2/3) on linux-aarch64 with release build
- [ ] jtreg tests(include tier1/2/3) on linux-aarch64 with fastdebug build
-------------
Commit messages:
- 8345757: [ASAN] clang17 report dprintf macro redefined
Changes: https://git.openjdk.org/jdk/pull/22630/files
Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22630&range=00
Issue: https://bugs.openjdk.org/browse/JDK-8345757
Stats: 2 lines in 2 files changed: 2 ins; 0 del; 0 mod
Patch: https://git.openjdk.org/jdk/pull/22630.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22630/head:pull/22630
PR: https://git.openjdk.org/jdk/pull/22630
More information about the security-dev
mailing list