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