Bug Tracker

Opened 3 years ago

Closed 2 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 3 years ago by gibson042

  • Component changed from unfiled to core
  • Priority changed from undecided to high
  • Status changed from new to open

comment:2 Changed 3 years ago by timmywil

  • Milestone changed from None to 1.11/2.1
  • Owner set to gibson042
  • Status changed from open to assigned

comment:3 Changed 2 years ago by Richard Gibson

  • Resolution set to fixed
  • Status changed from assigned to closed

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.