Skip to content

Check for maximum scale in IOBuffer before array copy#2239

Merged
Jeffery-Wasty merged 16 commits intomainfrom
arrayIndexOutOfBounds
Nov 3, 2023
Merged

Check for maximum scale in IOBuffer before array copy#2239
Jeffery-Wasty merged 16 commits intomainfrom
arrayIndexOutOfBounds

Conversation

@Jeffery-Wasty
Copy link
Copy Markdown
Contributor

Input was not validated for max length before being copied to byte array.

@Jeffery-Wasty Jeffery-Wasty marked this pull request as ready for review October 20, 2023 17:41
@Jeffery-Wasty Jeffery-Wasty requested review from lilgreenbird and tkyc and removed request for lilgreenbird and tkyc October 20, 2023 17:41
@Jeffery-Wasty Jeffery-Wasty requested review from lilgreenbird and tkyc and removed request for tkyc October 23, 2023 17:16
@Jeffery-Wasty Jeffery-Wasty linked an issue Oct 25, 2023 that may be closed by this pull request
@Jeffery-Wasty Jeffery-Wasty added this to the 12.5.0 milestone Oct 25, 2023
Comment thread src/test/java/com/microsoft/sqlserver/jdbc/tvp/TVPTypesTest.java Outdated
tkyc
tkyc previously approved these changes Nov 2, 2023
Comment thread src/main/java/com/microsoft/sqlserver/jdbc/SQLServerResource.java Outdated
tkyc
tkyc previously approved these changes Nov 3, 2023
Comment thread src/test/java/com/microsoft/sqlserver/jdbc/tvp/TVPTypesTest.java Outdated
@Jeffery-Wasty Jeffery-Wasty merged commit 729520d into main Nov 3, 2023
@Jeffery-Wasty Jeffery-Wasty deleted the arrayIndexOutOfBounds branch November 3, 2023 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Closed/Merged PRs

Development

Successfully merging this pull request may close these issues.

Driver throws ArrayIndexOutOfBoundsException from internal code

3 participants