From 69ecbc75b8a09ff9a738fa95df0b34596fb998b0 Mon Sep 17 00:00:00 2001 From: verwaest Date: Wed, 18 Feb 2015 07:48:30 -0800 Subject: [PATCH] The expected key is a valid identifier, which is already free of \\, <0x20, ". BUG= Review URL: https://codereview.chromium.org/936613006 Cr-Commit-Position: refs/heads/master@{#26725} --- src/json-parser.h | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/json-parser.h b/src/json-parser.h index 61043db307..9a22738e98 100644 --- a/src/json-parser.h +++ b/src/json-parser.h @@ -108,10 +108,8 @@ class JsonParser BASE_EMBEDDED { const uint8_t* expected_chars = content.ToOneByteVector().start(); for (int i = 0; i < length; i++) { uint8_t c0 = input_chars[i]; - if (c0 != expected_chars[i] || - c0 == '"' || c0 < 0x20 || c0 == '\\') { - return false; - } + // The expected string has to be free of \, " and characters < 0x20. + if (c0 != expected_chars[i]) return false; } if (input_chars[length] == '"') { position_ = position_ + length + 1;