about summary refs log tree commit diff
path: root/src/libstd/sys/unix/stack_overflow.rs
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2016-07-13 19:32:17 -0700
committerGitHub <noreply@github.com>2016-07-13 19:32:17 -0700
commit935bd763674ea3e5b79928bed08016291cc52901 (patch)
treebe4885c9792ca6abe3f8a12b7a33533b0d9980d9 /src/libstd/sys/unix/stack_overflow.rs
parent3dbbe2f716a09246939ad371125617c5526fd3fd (diff)
parent79fb5522bd5bf79c7dad7adfd516ad364dff3e9e (diff)
downloadrust-935bd763674ea3e5b79928bed08016291cc52901.tar.gz
rust-935bd763674ea3e5b79928bed08016291cc52901.zip
Auto merge of #34776 - cuviper:solaris-readdir, r=alexcrichton
std: fix `readdir` errors for 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 currently only matters for solaris targets, as the other unix platforms
are using `readdir_r` with a direct error return indication.  However, this is
getting deprecated (#34668) so they should all eventually switch to `readdir`.

This PR adds `set_errno`, uses it to clear the value before calling `readdir`,
then checks it again after to see the reason for a `NULL`.  A few other small
fixes are included just to get solaris compiling at all.

I couldn't get cross-compilation completely going, so I don't have a good way
to test this beyond a smoke-test cargo build of std.  I'd appreciate input from
someone more familiar with solaris -- cc @nbaksalyar?
Diffstat (limited to 'src/libstd/sys/unix/stack_overflow.rs')
0 files changed, 0 insertions, 0 deletions