Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

std: fix readdir errors for solaris #34776

Merged
merged 4 commits into from
Jul 14, 2016
Merged

std: fix readdir errors for solaris #34776

merged 4 commits into from
Jul 14, 2016

Commits on Jul 12, 2016

  1. std: Fix Thread::set_name() for newlib and solaris 

    The `use ffi::CStr` in `unix/thread.rs` was previously guarded, but now
all platforms need it for `Thread::set_name()`.  Newlib and Solaris do
nothing here, as they have no way to set a thread name, but they still
define the same method signature.
    @cuviper
    cuviper committed Jul 12, 2016
    Configuration menu
    Copy the full SHA
    ef1bd08 View commit details
    Browse the repository at this point in the history

  2. std: Fix IPV6 imports for solaris 

    Like BSDs, Solaris maps `IPV6_ADD_MEMBERSHIP` and `IPV6_DROP_MEMBERSHIP`
from `IPV6_JOIN_GROUP` and `IPV6_LEAVE_GROUP` respectively.
    @cuviper
    cuviper committed Jul 12, 2016
    Configuration menu
    Copy the full SHA
    9503b13 View commit details
    Browse the repository at this point in the history

  3. libc: fast-forward for solaris RTLD_DEFAULT

    @cuviper
    cuviper committed Jul 12, 2016
    Configuration menu
    Copy the full SHA
    726fa9f View commit details
    Browse the repository at this point in the history

  4. std: clear errno before readdir, then check it (solaris) 

    A `NULL` from `readdir` could be the end of stream or an error.  The
only way to know is to check `errno`, so it must be set to a known value
first, like a 0 that POSIX will never use.

This patch adds `set_errno`, uses it to clear the value before calling
`readdir`, then checks it again after to see the reason for a `NULL`.
    @cuviper
    cuviper committed Jul 12, 2016
    Configuration menu
    Copy the full SHA
    79fb552 View commit details
    Browse the repository at this point in the history