diff options
| author | mseaborn@chromium.org <mseaborn@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-25 21:04:57 +0000 |
|---|---|---|
| committer | mseaborn@chromium.org <mseaborn@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2013-07-25 21:04:57 +0000 |
| commit | 302492be5971df01cddcca27ad892adc7ca8792f (patch) | |
| tree | cfe30b5f0f950240402078f39c5976a22ffc25be /ppapi | |
| parent | b787d6b2f124f3240db3a389a82f9a18258e512a (diff) | |
| download | chromium_src-302492be5971df01cddcca27ad892adc7ca8792f.zip chromium_src-302492be5971df01cddcca27ad892adc7ca8792f.tar.gz chromium_src-302492be5971df01cddcca27ad892adc7ca8792f.tar.bz2 | |
PNaCl: Fix IRT wrapper to return proper error value
-1 is not a valid error return value for the IRT query function, which
is supposed to return 0 if an interface is not found.
This is just a nit, because this error is not likely to occur in
practice, but the presence of "return -1" might lead people to think
this is a valid error value for this interface.
BUG=none
TEST=none
Review URL: https://codereview.chromium.org/20438002
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@213692 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'ppapi')
| -rw-r--r-- | ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_ppapi.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_ppapi.c b/ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_ppapi.c index bace426..76bc4d5 100644 --- a/ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_ppapi.c +++ b/ppapi/native_client/src/untrusted/pnacl_irt_shim/shim_ppapi.c @@ -90,7 +90,7 @@ size_t __pnacl_irt_interface_wrapper(const char *interface_ident, &real_irt_ppapi_hook, sizeof real_irt_ppapi_hook) != sizeof real_irt_ppapi_hook) { - return -1; + return 0; } /* * Copy the portion of the ppapihook interface that is not wrapped. |