Non-SFI mode: Quick workaround of unexpected CHECK failure.
commitb16b57b17667f75ee80b3abe5dd59529bc48a8e5
authorhidehiko <hidehiko@chromium.org>
Thu, 11 Sep 2014 15:58:13 +0000 (11 08:58 -0700)
committerCommit bot <commit-bot@chromium.org>
Thu, 11 Sep 2014 16:01:49 +0000 (11 16:01 +0000)
tree6dfedff936f21c2f4c1e6f62e89f8d493d9c8f22
parent90e7f20d43ce56ce0470871e86c03abbbb8d9c9c
Non-SFI mode: Quick workaround of unexpected CHECK failure.

Recently, crrev.com/418423002 is landed, but it has a bug in Non-SFI mode.
It introduces CHECK for the file token in ManifestService::OpenResource().
However, in Non-SFI mode, there is no NaClIPCAdapter, so the token is passed
from the renderer directly. (Actually the IPC channel is connected directly
to the renderer).
As a result, if the renderer fills the file token properly, it crashes.
As far as I investigated, it happens, at least, when the fast-path is triggered
(i.e. OpenNaClExecutable works in DownloadFile in ppb_nacl_private_impl.cc).
Anyway, we can ignore file tokens in Non-SFI mode, because it is for
SFI NaCl's validation cache.

BUG=394130
TEST=Ran trybots. Patched locally and run our Non-SFI NaCl app.
CQ_EXTRA_TRYBOTS=tryserver.chromium.linux:linux_rel_precise32

Review URL: https://codereview.chromium.org/560983003

Cr-Commit-Position: refs/heads/master@{#294396}
ppapi/nacl_irt/manifest_service.cc