Skip to content

Avoid ArrayIndexOutOfBoundsException in SpEL's Indexer#23658

Closed
JoyceZhan wants to merge 1 commit into
spring-projects:masterfrom
JoyceZhan:fix-indexer
Closed

Avoid ArrayIndexOutOfBoundsException in SpEL's Indexer#23658
JoyceZhan wants to merge 1 commit into
spring-projects:masterfrom
JoyceZhan:fix-indexer

Conversation

@JoyceZhan

@JoyceZhan JoyceZhan commented Sep 18, 2019 •
edited by sbrannen
Loading

Copy link
Copy Markdown

When index == arrayLength, the array index is also out of bounds.

For this use case, a SpelEvaluationException should be thrown instead of
ArrayIndexOutOfBoundsException.

@Joyce-Zhan
Avoid ArrayIndexOutOfBoundsException from Indexer.java
4e91cf8 
When index==arrayLength, array index is also out of bound,
SpelEvaluationException should also be thrown instead of
ArrayIndexOutOfBoundsException.
@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Sep 18, 2019
@sbrannen

sbrannen commented Sep 18, 2019

Copy link
Copy Markdown
Member

Can you please provide a test case that fails before the change and passes after the change?

@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) status: waiting-for-feedback We need additional information before we can continue labels Sep 18, 2019
@JoyceZhan

JoyceZhan commented Sep 19, 2019

Copy link
Copy Markdown
Author

Can you please provide a test case that fails before the change and passes after the change?

Test case is checked in, see to IndexerTests.java

@spring-projects-issues spring-projects-issues added status: feedback-provided Feedback has been provided and removed status: waiting-for-feedback We need additional information before we can continue labels Sep 19, 2019
@sbrannen sbrannen changed the title Avoid ArrayIndexOutOfBoundsException from Indexer.java Avoid ArrayIndexOutOfBoundsException SpEL's Indexer Sep 19, 2019
@sbrannen

sbrannen commented Sep 19, 2019 •
edited
Loading

Copy link
Copy Markdown
Member

Test case is checked in, see to IndexerTests.java

Thanks for adding the test!

This PR breaks the build due to:

  • a missing copyright header
  • a missing blank link in the imports
  • a missing newline at the end of the file

For future reference, please make sure you run ./gradlew check before pushing commits.

I'll fix those minor issues before merging.

@sbrannen sbrannen self-assigned this Sep 19, 2019
@sbrannen sbrannen added the type: enhancement A general enhancement label Sep 19, 2019
@sbrannen sbrannen added this to the 5.2 GA milestone Sep 19, 2019
@sbrannen sbrannen changed the title Avoid ArrayIndexOutOfBoundsException SpEL's Indexer Avoid ArrayIndexOutOfBoundsException in SpEL's Indexer Sep 19, 2019
@sbrannen sbrannen closed this in fde7b1e Sep 19, 2019
sbrannen added a commit that referenced this pull request Sep 19, 2019
@sbrannen
Polish contribution
734ceed 
See gh-23658
@sbrannen

sbrannen commented Sep 19, 2019

Copy link
Copy Markdown
Member

This has been merged into master in fde7b1e, and I revised the test and moved it to PropertyAccessTests in 734ceed.

Thanks!

@sbrannen sbrannen removed the status: waiting-for-triage An issue we've not yet triaged or decided on label Apr 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@sbrannen sbrannen

Labels

in: core Issues in core modules (aop, beans, core, context, expression) status: feedback-provided Feedback has been provided type: enhancement A general enhancement

Projects

None yet

Milestone

5.2 GA

Development

Successfully merging this pull request may close these issues.

4 participants

@JoyceZhan @sbrannen @spring-projects-issues @Joyce-Zhan