Network entry API#5049
Conversation
89888d3 to
a7783fd
Compare
This comment has been minimized.
This comment has been minimized.
| pub fn getnetbyname(name: *const c_char) -> *mut netent; | ||
| pub fn getnetbyaddr(net: u32, type_: c_int) -> *mut netent; | ||
| pub fn setnetent(stayopen: c_int); | ||
| pub fn endnetent(); |
There was a problem hiding this comment.
I wonder if this had the same situation as setservent/endservent and we might need a test tweak.
cc @jclulow, @pfmooney for confirmation
Line 1173 in a3e451c
https://github.com/illumos/illumos-gate/blob/5425aa59e8fd3834b86390ed6d8a13586c918df9/usr/src/head/netdb.h#L288-L305
|
Reminder, once the PR becomes ready for a review, use |
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
The tests fails for Android because the symbol How should i handle this situation? Should i move these new symbols to Linux specifically ( I have tried to run the docker tests locally but can't get it to run. It fails on missing Local log |
|
I got the docker test running locally now. The problem was a I will take a closer look at what fails and what works. |
Description
Add functions and struct for the "get network entry" API, including the reentrant GNU extension.
Sources
glic/resolv/netdb.h
Checklist
libc-test/semverhave been updated*LASTor*MAXare included (see #3131)cd libc-test && cargo test --target mytarget);especially relevant for platforms that may not be checked in CI
@rustbot label +stable-nominated