-
Notifications
You must be signed in to change notification settings - Fork 99
fix error: implicit declaration of function 'wc_SSH_KDF' #848
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
Conversation
hopefully resolves wolfSSL#781 when compiling with ./configure --enable-static --disable-shared --enable-scp --enable-sftp --enable-shell --enable-cryptonly --with-wolfssl=wolfssl/install/ I got: $ make -j20 make -j17 all-am make[1]: warning: -j17 forced in submake: resetting jobserver mode. make[1]: Entering directory '/home/hans/projects/wolfssh' CC src/libwolfssh_la-internal.lo src/internal.c: In function 'GenerateKey': src/internal.c:2348:15: error: implicit declaration of function 'wc_SSH_KDF'; did you mean 'wc_HKDF'? [-Werror=implicit-function-declaration] 2348 | ret = wc_SSH_KDF(hashId, keyId, key, keySz, | ^~~~~~~~~~ | wc_HKDF src/internal.c:2348:15: error: nested extern declaration of 'wc_SSH_KDF' [-Werror=nested-externs] cc1: all warnings being treated as errors make[1]: *** [Makefile:1483: src/libwolfssh_la-internal.lo] Error 1 make[1]: Leaving directory '/home/hans/projects/wolfssh' make: *** [Makefile:1020: all] Error 2
|
Can one of the admins verify this patch? |
|
@ejohnstown or @JacobBarthelmeh do we want to allow building wolfSSH without |
I have no knowledge of any wolfssh contributor agreement. I'd be okay with signing one.
I need a small SSH server on a system where I don’t have root or package-install privileges. I can run my own executables, but I can’t touch /etc/passwd or /etc/shadow, so I need a self-contained SSH server that does its own authentication. wolfssh echoserver seems like a good fit 👍 |
|
Hi @divinity76 , wolfSSH is a great fit for this! Where are you located and is this a commercial use? If you'd like to keep that information private just email support at wolfssl dot com and reference this PR. Its likely we will not accept this change. To resolve this please make sure you build wolfCrypt with |
|
I'm going to take this as a bug report. I see what I overlooked in the guard checking. I'll do a new PR to fix this. |
|
It is logically similar, but not identical. Do you want to go through with a contributor agreement? I can reopen this and close mine. I don't mind. I figured for two lines, I would do my own fix and treat this as a bug report. |
|
David had left directions for emailing us at support at wolfssl dot com and to reference this PR. Please still do this. |
|
@divinity76 is an approved wolfSSL contributor |
hopefully resolves #781
when compiling from git master ( 967d6c5 dated Fri Oct 24 09:44:15 2025 -0700) with
I got: