Skip to content

Conversation

@spodgurskiy
Copy link

MethodBasedEvaluationContext.lazyLoadArguments fails for an empty variable arguments method call
java.lang.ArrayIndexOutOfBoundsException at MethodBasedEvaluationContextTests.java:70

No similar issue is found in project JIRA
ICLA signed.

…iable arguments method call

java.lang.ArrayIndexOutOfBoundsException at MethodBasedEvaluationContextTests.java:70
@snicoll
Copy link
Member

snicoll commented Jun 9, 2016

No similar issue is found in project JIRA

Could you please create one and link your PR? thanks.

@snicoll
Copy link
Member

snicoll commented Jul 12, 2016

ping @spodgurskiy

@spodgurskiy
Copy link
Author

Here is the issue https://jira.spring.io/browse/SPR-14453

@snicoll snicoll changed the title MethodBasedEvaluationContext.lazyLoadArguments() fails for variable arguments method call SPR-14453 - MethodBasedEvaluationContext.lazyLoadArguments() fails for variable arguments method call Jul 12, 2016
@pivotal-issuemaster
Copy link

@spodgurskiy Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@spodgurskiy Thank you for signing the Contributor License Agreement!

snicoll pushed a commit that referenced this pull request Jul 14, 2016
This commit fix a potential `ArrayIndexOutOfBoundsException` if
`lazyLoadArguments` is called with an empty variable argument.

See gh-1070
@snicoll snicoll closed this in ff738e7 Jul 14, 2016
snicoll pushed a commit that referenced this pull request Jul 14, 2016
This commit fix a potential `ArrayIndexOutOfBoundsException` if
`lazyLoadArguments` is called with an empty variable argument.

See gh-1070
snicoll added a commit that referenced this pull request Jul 14, 2016
@snicoll
Copy link
Member

snicoll commented Jul 14, 2016

Thanks for the PR!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants