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 PAM package #21

Merged
merged 7 commits into from
Jul 18, 2017
Merged

Add PAM package #21

merged 7 commits into from
Jul 18, 2017

Conversation

josephlr
Copy link
Member

This PR significantly updates the PAM package so we will be able to build our PAM module (see #4).
Specifically, this includes:

  • Changing the Key struct to support creation from/to C strings
  • Changing the util package to support more array conversions
  • Changing the pam package to have significantly more features (using this package)
  • Changing cmd/fscrypt to support the new API
  • Refactoring how unknown UIDs are handled.

Code pulled from github.com/msteinert/pam
This commit adds in a C implementation for the pam_conv we will use in
login.go as well as adding three CleanupFuncs that will be used with
pam_set_data(). It also adds copyInfoSecret() which should be paired
with freeSecret().
This commit provides a Go interface to the PAM functions.
The commit changes how we get the username representation, and uses the
new pam API for checking the proposed login token.
@josephlr josephlr merged commit 3d08d9f into master Jul 18, 2017
@josephlr josephlr mentioned this pull request Jul 19, 2017
5 tasks
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.

1 participant