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.

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

Add functions to get password/group file entry #342

Merged
merged 2 commits into from
Jul 31, 2016

Conversation

knight42
Copy link
Contributor

@knight42 knight42 commented Jul 31, 2016

  • Add struct group
  • Add getgrnam(),getgrgid(),getpwuid(), getpwnam(),setpwent(),getpwent(),endpwent(),

@rust-highfive
Copy link

r? @alexcrichton

(rust_highfive has picked a reviewer for you, use r? to override)

pub fn getpwuid_r(uid: ::uid_t,
pwd: *mut passwd,
buf: *mut ::c_char,
buflen: ::size_t) -> *const passwd;
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks like the signature of getpwuid_r above is different than this one for solaris, was that intentional?

Unfortunately we don't have CI for the solaris bindings yet, so I'm not sure which one is right :(

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the catch! I have updated my patch according to the man pages.

@alexcrichton alexcrichton merged commit 932f22b into rust-lang:master Jul 31, 2016
@alexcrichton
Copy link
Member

Thanks!

@knight42 knight42 deleted the getpw branch August 6, 2016 02:12
Susurrus pushed a commit to Susurrus/libc that referenced this pull request Mar 26, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants