Bug Tracker

Ticket #14492 (closed bug: fixed)

Opened 9 months ago

Last modified 9 months ago

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:
Blocking: Blocked by:

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

comment:1 Changed 9 months ago by gibson042

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

comment:2 Changed 9 months ago by timmywil

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

comment:3 Changed 9 months ago by Richard Gibson

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

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.