Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

StackOverflowError when creating a new JSONArray #372

Closed
agouz opened this issue Oct 30, 2017 · 2 comments · Fixed by #373
Closed

StackOverflowError when creating a new JSONArray #372

agouz opened this issue Oct 30, 2017 · 2 comments · Fixed by #373

Comments

@agouz
Copy link

agouz commented Oct 30, 2017

Hello Sean,
Thanks for this great library.
I have one issue when trying to initialise a JSONArray object and the input is simply "[". The jvm crashes with StackOverflowError.

simply try to run this: new JSONArray("[");

version used: 20171018

Thanks again :)
@johnjaylward
Copy link
Contributor

I can confirm this behavior

@johnjaylward johnjaylward mentioned this issue Oct 30, 2017
Merged
johnjaylward added a commit to johnjaylward/JSON-java that referenced this issue Oct 30, 2017
@johnjaylward
fixes stleary#372.
ee9e706 
Corrects behavior of unclosed arrays
@johnjaylward johnjaylward mentioned this issue Oct 30, 2017
Merged
johnjaylward added a commit to johnjaylward/JSON-java that referenced this issue Oct 30, 2017
@johnjaylward
fixes stleary#372.
ed8745c 
Corrects behavior of unclosed arrays
@stleary
Copy link
Owner

stleary commented Oct 31, 2017

Continued discussion in #373

@stleary stleary closed this as completed Oct 31, 2017
@Ichbinjoe Ichbinjoe mentioned this issue Mar 30, 2018
Closed
@miguelfreitas93 miguelfreitas93 mentioned this issue Jun 16, 2020
Closed
miguelfreitas93 referenced this issue in checkmarx-ts/checkmarx-github-action Jun 16, 2020
@miguelfreitas93
add default exclusions to OSA Scans, improve gitignore, start to supp…
837e6d5 
…ort OSA Github issues
@srcdev888 srcdev888 mentioned this issue Mar 28, 2022
Open
@TaynaCT TaynaCT mentioned this issue Apr 13, 2022
Open
@EylamCX EylamCX mentioned this issue May 1, 2022
Open
This was referenced May 27, 2022
Merged
Merged
@mridilla mridilla mentioned this issue Jul 1, 2022
Open
@scott-cx scott-cx mentioned this issue Aug 1, 2022
Open
hewholived added a commit to hewholived/shift-java that referenced this issue Aug 17, 2022
@hewholived
fix: Updating org.json version
2b55d34 
Updating org.json dependency version to to mitigate stleary/JSON-java#372
@hewholived hewholived mentioned this issue Aug 17, 2022
Closed
YellowfinBI pushed a commit to YellowfinBI/JSON-java that referenced this issue Dec 8, 2022
@johnjaylward @YellowfinBI
fixes stleary#372.
9313524 
Corrects behavior of unclosed arrays
@ksalman-Cx ksalman-Cx mentioned this issue Nov 8, 2022
Closed
@SamHeadrickCx SamHeadrickCx mentioned this issue Mar 14, 2023
Open
@cyates-checkmarx cyates-checkmarx mentioned this issue Jun 23, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes Unclosed json array stack overflow johnjaylward/JSON-java
3 participants
@johnjaylward @stleary @agouz