Fix pnacl-sz to not accept files containing multiple modules.

Fix pnacl-sz to not accept files containing multiple modules.

Previous code checked, after the fact, that there weren't multiple
modules in the block. However, the code for TopLevelParser assumes
that there is only on module block, allowing it to more efficiently
test internal state. To fix the problem, modified
TopLevelParser::ParseBlock to fail, and not attempt to parse the second
block if the file contains more than one block.

BUG= https://code.google.com/p/nativeclient/issues/detail?id=4260
R=stichnot@chromium.org

Committed: https://gerrit.chromium.org/gerrit/gitweb?p=native_client/pnacl-subzero.git;a=commit;h=0b8763eae21d5d8952d401cf8e4dd7e4fe9b401d

[Karl, 2015-09-22 17:23:31]

kschimpf@google.com changed reviewers:
+ jpp@chromium.org, stichnot@chromium.org

[Jim Stichnoth, 2015-09-22 17:33:43]

lgtm

https://codereview.chromium.org/1362643002/diff/1/src/PNaClTranslator.cpp
File src/PNaClTranslator.cpp (right):

https://codereview.chromium.org/1362643002/diff/1/src/PNaClTranslator.cpp#new...
src/PNaClTranslator.cpp:3231: bool results = Parser.ParseThisBlock();
capitalize Results.

Better yet, rename to something like ParseSuccess ?

[Karl, 2015-09-22 17:41:16]

Committed patchset #2 (id:20001) manually as
0b8763eae21d5d8952d401cf8e4dd7e4fe9b401d (presubmit successful).

[Karl, 2015-09-22 17:41:31]

https://codereview.chromium.org/1362643002/diff/1/src/PNaClTranslator.cpp
File src/PNaClTranslator.cpp (right):

https://codereview.chromium.org/1362643002/diff/1/src/PNaClTranslator.cpp#new...
src/PNaClTranslator.cpp:3231: bool results = Parser.ParseThisBlock();
On 2015/09/22 17:33:43, stichnot wrote:
> capitalize Results.
> 
> Better yet, rename to something like ParseSuccess ?

Used ParseFailed, since that is the context of the boolen.