diff options
| author | matthewturk@gmail.com <matthewturk@gmail.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-10 19:14:53 +0000 |
|---|---|---|
| committer | matthewturk@gmail.com <matthewturk@gmail.com@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-01-10 19:14:53 +0000 |
| commit | 06b98a9afdf6b5e17b5c368684a8f618ee472831 (patch) | |
| tree | b0f3066af54d079afe80be8fd2e73e6ed8442ae0 /native_client_sdk | |
| parent | fed63e98ed3ec83c3133c06f02c661e8e40ab008 (diff) | |
| download | chromium_src-06b98a9afdf6b5e17b5c368684a8f618ee472831.zip chromium_src-06b98a9afdf6b5e17b5c368684a8f618ee472831.tar.gz chromium_src-06b98a9afdf6b5e17b5c368684a8f618ee472831.tar.bz2 | |
[NaCl SDK] Change KernelProxy::GetCWD() to allocate path.
When supplying a zero size to getcwd() typical behavior is to allocate
internally and return a pointer. This behavior was already implemented, but
disabled by checking if size <= 0 and erroring with EINVAL. As size_t is
positive definite, this entire check can be removed.
R=noelallen@chromium.org, sbc@chromium.org, binji@chromium.org
BUG=
Review URL: https://codereview.chromium.org/122943005
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@244213 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'native_client_sdk')
| -rw-r--r-- | native_client_sdk/src/AUTHORS | 1 | ||||
| -rw-r--r-- | native_client_sdk/src/libraries/nacl_io/kernel_proxy.cc | 5 |
2 files changed, 1 insertions, 5 deletions
diff --git a/native_client_sdk/src/AUTHORS b/native_client_sdk/src/AUTHORS index 24df0cd..4f96a60a 100644 --- a/native_client_sdk/src/AUTHORS +++ b/native_client_sdk/src/AUTHORS @@ -5,3 +5,4 @@ # Name/Organization <email address> Google Inc. native-client-discuss@google.com +Matthew Turk <matthewturk@gmail.com> diff --git a/native_client_sdk/src/libraries/nacl_io/kernel_proxy.cc b/native_client_sdk/src/libraries/nacl_io/kernel_proxy.cc index bbb3896..3ddca20 100644 --- a/native_client_sdk/src/libraries/nacl_io/kernel_proxy.cc +++ b/native_client_sdk/src/libraries/nacl_io/kernel_proxy.cc @@ -270,11 +270,6 @@ int KernelProxy::chdir(const char* path) { char* KernelProxy::getcwd(char* buf, size_t size) { std::string cwd = GetCWD(); - if (size <= 0) { - errno = EINVAL; - return NULL; - } - // If size is 0, allocate as much as we need. if (size == 0) { size = cwd.size() + 1; |