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

compile fixes for !HAVE_GSS_USEROK and !HAVE_GSS_INDICATE_MECHS_BY_ATTRS #1

Closed
wants to merge 1 commit into from

Conversation

jbasney
Copy link

@jbasney jbasney commented Aug 13, 2015

Fixes for compiling against GSI GSSAPI which doesn't (yet) have gss_userok() or gss_indicate_mechs_by_attrs(). Needed so I can include support for generic GSS-API mechanisms in the GSI-OpenSSH code base (https://github.com/ncsa/gsi-openssh).

Compiler warnings/errors fixed are:

gss-serv.c:81:1: warning: unused function 'ssh_gssapi_generic_localname'
gss-serv.c:118:30: error: initializing 'const ssh_gssapi_mech' with an expression of incompatible type 'void *'
gss-serv.c:251:22: error: use of undeclared identifier 'GSS_C_MA_MECH_NEGO'
gss-serv.c:252:22: error: use of undeclared identifier 'GSS_C_MA_NOT_MECH'
gss-serv.c:253:22: error: use of undeclared identifier 'GSS_C_MA_DEPRECATED'
gss-serv.c:258:19: warning: implicit declaration of function 'gss_indicate_mechs_by_attrs'

@hartmans
Copy link

Any chance I could get you to submit this over at
https://github.com/gss-openssh?
Painless security is getting out of the openssh business and we're
trying to get an open-source group together to do that work.

@jbasney
Copy link
Author

jbasney commented Aug 14, 2015

Yes, I'll try to submit it over at https://github.com/gss-openssh today.

@jbasney
Copy link
Author

jbasney commented Aug 14, 2015

gss-openssh/openssh-portable#1 submitted. Since https://github.com/gss-openssh didn't already have SimonWilkinson#2 I submitted a pull request combining both.

@jbasney jbasney closed this Aug 14, 2015
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