diff options
author | Chris Lattner <sabre@nondot.org> | 2005-02-13 22:46:37 +0000 |
---|---|---|
committer | Chris Lattner <sabre@nondot.org> | 2005-02-13 22:46:37 +0000 |
commit | 946af52687fea113a6596385bfbdb26994709a37 (patch) | |
tree | bd1830b39bcb205b95f317f0fb1f7c21cc173051 /lib/System | |
parent | 2e12c745eae4be1f0c5c5f4840c58918c11c6b1d (diff) |
If errno is zero strerror_r does not modify the buffer, leaving it unterminated.
This causes garbage to be printed out after error messages.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@20165 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/System')
-rw-r--r-- | lib/System/Unix/MappedFile.inc | 1 | ||||
-rw-r--r-- | lib/System/Unix/Unix.h | 7 |
2 files changed, 6 insertions, 2 deletions
diff --git a/lib/System/Unix/MappedFile.inc b/lib/System/Unix/MappedFile.inc index 07683c14392..409cea7e822 100644 --- a/lib/System/Unix/MappedFile.inc +++ b/lib/System/Unix/MappedFile.inc @@ -51,6 +51,7 @@ void MappedFile::initialize() { else if (options_&WRITE_ACCESS) mode = O_WRONLY; info_->fd_ = ::open(path_.c_str(),mode); + if (info_->fd_ < 0) { delete info_; info_ = 0; diff --git a/lib/System/Unix/Unix.h b/lib/System/Unix/Unix.h index 6dc75545b66..75009d0746e 100644 --- a/lib/System/Unix/Unix.h +++ b/lib/System/Unix/Unix.h @@ -68,14 +68,17 @@ inline void ThrowErrno(const std::string& prefix) { char buffer[MAXPATHLEN]; + buffer[0] = 0; #ifdef HAVE_STRERROR_R // strerror_r is thread-safe. - strerror_r(errno,buffer,MAXPATHLEN-1); + if (errno) + strerror_r(errno,buffer,MAXPATHLEN-1); #elif HAVE_STRERROR // Copy the thread un-safe result of strerror into // the buffer as fast as possible to minimize impact // of collision of strerror in multiple threads. - strncpy(buffer,strerror(errno),MAXPATHLEN-1); + if (Errno) + strncpy(buffer,strerror(errno),MAXPATHLEN-1); buffer[MAXPATHLEN-1] = 0; #else // Strange that this system doesn't even have strerror |