gn: Don't read unitialized memory on files containing just a comment.

gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856
Committed: https://crrev.com/c3b455922b7ddb748e9a7b351cac7a4586c7e374
Cr-Commit-Position: refs/heads/master@{#414522}

[Nico, 2016-08-25 19:32:24]

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

[Nico, 2016-08-25 19:32:35]

Description was changed from

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch by scottmg.

BUG=640856
==========

to

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856
==========

[Nico, 2016-08-25 19:32:46]

thakis@chromium.org changed reviewers:
+ scottmg@chromium.org

[Nico, 2016-08-25 19:32:47]

Maybe like so?

[commit-bot: I haz the power, 2016-08-25 19:33:05]

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

[scottmg, 2016-08-25 19:46:31]

poing! lgtm

[Nico, 2016-08-25 19:47:49]

The CQ bit was unchecked by thakis@chromium.org

[Nico, 2016-08-25 19:47:55]

The CQ bit was checked by thakis@chromium.org

[commit-bot: I haz the power, 2016-08-25 19:48:34]

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, 2016-08-25 20:22:01]

Description was changed from

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856
==========

to

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856
==========

[commit-bot: I haz the power, 2016-08-25 20:22:02]

Committed patchset #3 (id:40001)

[commit-bot: I haz the power, 2016-08-25 20:24:13]

Description was changed from

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856
==========

to

==========
gn: Don't read unitialized memory on files containing just a comment.

The BlockNode::GetRange() of a block containing nothing doesn't have
an initialized bytes_ member.  Skip empty nodes, they'll be assinged
to the file block further down.

Patch sketch by scottmg.

BUG=640856

Committed: https://crrev.com/c3b455922b7ddb748e9a7b351cac7a4586c7e374
Cr-Commit-Position: refs/heads/master@{#414522}
==========

[commit-bot: I haz the power, 2016-08-25 20:24:13]

Patchset 3 (id:??) landed as
https://crrev.com/c3b455922b7ddb748e9a7b351cac7a4586c7e374
Cr-Commit-Position: refs/heads/master@{#414522}