RFR: 8175874: Security.insertProviderAt should throw IllegalArgumentException when index is illegal [v2]

Ben Perez duke at openjdk.org
Fri Aug 11 17:31:58 UTC 2023


> Fixing the bug as stated would cause compatibility issues. E.g. the `addProvider` function always adds new providers at position 0 and therefore would always throw an error. Instead of changing how the function handles indices, we instead have left an `@implNote`

Ben Perez has updated the pull request incrementally with one additional commit since the last revision:

  Removed implnote and added to function specification instead

-------------

Changes:
  - all: https://git.openjdk.org/jdk/pull/14925/files
  - new: https://git.openjdk.org/jdk/pull/14925/files/274ef511..b8c4bf7c

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=14925&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=14925&range=00-01

  Stats: 8 lines in 1 file changed: 3 ins; 4 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/14925.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/14925/head:pull/14925

PR: https://git.openjdk.org/jdk/pull/14925



More information about the security-dev mailing list