Fix nullptr dereference in service catalog

Fix nullptr dereference in service catalog

ProcessManifest can return null in a few failure modes.
This result gets passed into Reader::OnReadManifest which
was doing no null checking. This fixes that.

BUG=632916
TBR=ben@chromium.org

Committed: https://crrev.com/3d1a0f65aa799880b017881b5b5e0f14e958cb74
Cr-Commit-Position: refs/heads/master@{#414769}

[Ken Rockot(use gerrit already), 2016-08-26 17:55:55]

The CQ bit was checked by rockot@chromium.org

[commit-bot: I haz the power, 2016-08-26 17:56:35]

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-26 18:56:58]

Committed patchset #1 (id:1)

[commit-bot: I haz the power, 2016-08-26 19:16:37]

Description was changed from

==========
Fix nullptr dereference in service catalog

ProcessManifest can return null in a few failure modes.
This result gets passed into Reader::OnReadManifest which
was doing no null checking. This fixes that.

BUG=632916
TBR=ben@chromium.org
==========

to

==========
Fix nullptr dereference in service catalog

ProcessManifest can return null in a few failure modes.
This result gets passed into Reader::OnReadManifest which
was doing no null checking. This fixes that.

BUG=632916
TBR=ben@chromium.org

Committed: https://crrev.com/3d1a0f65aa799880b017881b5b5e0f14e958cb74
Cr-Commit-Position: refs/heads/master@{#414769}
==========

[commit-bot: I haz the power, 2016-08-26 19:16:37]

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