gn: fix fuzzer bug in parser for stack local

gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=brettw@chromium.org
BUG=683454
Review-Url: https://codereview.chromium.org/2654143005
Cr-Commit-Position: refs/heads/master@{#446528}
Committed: https://chromium.googlesource.com/chromium/src/+/0fcc08c5d93f2e8128d5e583c236dd2a81e70a7b

[scottmg, 2017-01-26 22:55:26]

Description was changed from

==========
gn: fix fuzzer bug in parser for stack local

R=dpranke@chromium.org
BUG=683454
==========

to

==========
gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=dpranke@chromium.org
BUG=683454
==========

[scottmg, 2017-01-26 22:55:36]

The CQ bit was checked by scottmg@chromium.org to run a CQ dry run

[commit-bot: I haz the power, 2017-01-26 22:56:08]

Dry run: CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[scottmg, 2017-01-26 23:00:06]

Description was changed from

==========
gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=dpranke@chromium.org
BUG=683454
==========

to

==========
gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=brettw@chromium.org
BUG=683454
==========

[scottmg, 2017-01-26 23:00:07]

scottmg@chromium.org changed reviewers:
+ brettw@chromium.org
- dpranke@chromium.org

[commit-bot: I haz the power, 2017-01-26 23:13:14]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2017-01-26 23:13:14]

Dry run: This issue passed the CQ dry run.

[brettw, 2017-01-26 23:18:59]

lgtm

[scottmg, 2017-01-27 01:41:16]

The CQ bit was checked by scottmg@chromium.org

[commit-bot: I haz the power, 2017-01-27 01:42:29]

CQ is trying da patch. Follow status at
 
https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.or...

[commit-bot: I haz the power, 2017-01-27 01:49:02]

CQ is committing da patch.

Bot data: {"patchset_id": 1, "attempt_start_ts": 1485481276721240, "parent_rev":
"6c977960be5a999b8dfde01ce57836eb038233c2", "commit_rev":
"0fcc08c5d93f2e8128d5e583c236dd2a81e70a7b"}

[commit-bot: I haz the power, 2017-01-27 01:49:34]

Description was changed from

==========
gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=brettw@chromium.org
BUG=683454
==========

to

==========
gn: fix fuzzer bug in parser for stack local

ParseNode::GetRange() returns a LocationRange by value, so the reference taken
is invalid at the next ;. Copy the Location by value instead.

R=brettw@chromium.org
BUG=683454

Review-Url: https://codereview.chromium.org/2654143005
Cr-Commit-Position: refs/heads/master@{#446528}
Committed:
https://chromium.googlesource.com/chromium/src/+/0fcc08c5d93f2e8128d5e583c236...
==========

[commit-bot: I haz the power, 2017-01-27 01:49:35]

Committed patchset #1 (id:1) as
https://chromium.googlesource.com/chromium/src/+/0fcc08c5d93f2e8128d5e583c236...