Bug Tracker

Opened 10 years ago

Closed 10 years ago

#14492 closed bug (fixed)

parseJSON incorrectly accepts comma expressions

Reported by: gibson042 Owned by: gibson042
Priority: high Milestone: 1.11/2.1
Component: core Version: 1.10.2
Keywords: Cc:
Blocked by: Blocking:

Description

The value returned from parseJSON can be overridden by appending a comma and some additional JSON to the input. Native JSON.parse implementations correctly throw exceptions on such input.

This bug is shared with JSON2.js, from which our implementation is derived.

http://jsfiddle.net/ZPYhF/

Change History (3)

comment:1 Changed 10 years ago by gibson042

Component: unfiledcore
Priority: undecidedhigh
Status: newopen

comment:2 Changed 10 years ago by Timmy Willison

Milestone: None1.11/2.1
Owner: set to gibson042
Status: openassigned

comment:3 Changed 10 years ago by Richard Gibson

Resolution: fixed
Status: assignedclosed

Fix #14492: More correct jQuery.parseJSON. Close gh-1419. (cherry picked from commit 60a6178131afec97b68c9a45bc24459f7b8bd905)

Conflicts:

src/ajax/parseJSON.js

Changeset: 71b2ac524e49c0d1cc4e5d7bd8fd9d5a1317a20b

Note: See TracTickets for help on using tickets.