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 SSPI context flag validation functions #7

Merged
merged 1 commit into from
Jun 14, 2018

Conversation

enj
Copy link
Contributor

@enj enj commented Jun 6, 2018

This change adds two verification functions for SSPI context: VerifyFlags and VerifySelectiveFlags. The former is used to confirm that all flags requested for the context were honored. The latter is used to confirm if some subset of the requested flags were honored.

Signed-off-by: Monis Khan mkhan@redhat.com

@enj
Copy link
Contributor Author

enj commented Jun 6, 2018

@alexbrainman PTAL

@alexbrainman
Copy link
Owner

This looks nice, thank you very much. But it would be useful for me and other users, if you could add calls to your new function in any of our existing tests. Is that possible?

Alex

@enj
Copy link
Contributor Author

enj commented Jun 12, 2018

add calls to your new function in any of our existing tests

I will look into this sometime this week.

@enj
Copy link
Contributor Author

enj commented Jun 12, 2018

@alexbrainman PTAL at 9b3b824

@alexbrainman
Copy link
Owner

Nice, thank you very much. Please, squash all commits into one, and I will merge.

Alex

This change adds two verification functions for SSPI context:
VerifyFlags and VerifySelectiveFlags.  The former is used to confirm
that all flags requested for the context were honored.  The latter
is used to confirm if some subset of the requested flags were
honored.

Signed-off-by: Monis Khan <mkhan@redhat.com>
@enj
Copy link
Contributor Author

enj commented Jun 13, 2018

@alexbrainman squashed into one commit and rebased on master.

@alexbrainman
Copy link
Owner

Thank you very much. Submitting now.

Alex

@alexbrainman alexbrainman merged commit e580b90 into alexbrainman:master Jun 14, 2018
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.

2 participants