Page MenuHomeFreeBSD

libssh: correct libssh src file list
ClosedPublic

Authored by emaste on Oct 17 2021, 12:34 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 29, 12:39 AM
Unknown Object (File)
Tue, Nov 25, 4:26 PM
Unknown Object (File)
Mon, Nov 24, 10:14 AM
Unknown Object (File)
Sat, Nov 15, 8:37 AM
Unknown Object (File)
Nov 11 2025, 10:38 PM
Unknown Object (File)
Nov 5 2025, 1:21 PM
Unknown Object (File)
Nov 3 2025, 5:15 AM
Unknown Object (File)
Oct 29 2025, 2:06 PM
Subscribers

Details

Summary

Link against the ssh-sk-helper client rather than the sk internal implementation.

PR: 258384
Fixes: f448c3ed4ae1 ("openssh: Add new source files to libssl")
Fixes: 19261079b743 ("openssh: update to OpenSSH v8.7p1")
Sponsored by: The FreeBSD Foundation

Test Plan

Ad-hoc tested via login(1) with pam_ssh.so in /etc/pam.d/login. @madpilot (PR reporter) is going to confirm.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

emaste created this revision.
emaste added a child revision: D32509: ssh: enable FIDO/U2F keys.

I tried this right now but it fails to build for me:

===> sbin/geom (includes)                                                                                               
--- includes_subdir_secure ---                                                                                          
--- includes_subdir_secure/libexec/ssh-sk-helper ---                                                                    
make[5]: "/poudriere/jails/14amd64/usr/src/share/mk/src.libnames.mk" line 723: /poudriere/jails/14amd64/usr/src/secure/l
ibexec/ssh-sk-helper: Invalid LIBADD used which may need to be added to src.libnames.mk: fido2                          
                                                                                                                        
make[5]: stopped in /poudriere/jails/14amd64/usr/src/secure/libexec/ssh-sk-helper

Maybe I misapplied it?

crypto/openssh/config.h
140

Isn't this the opposite of what you want?

Oops I seem to have uploaded the wrong diff here.
D32509 should be adding U2F key support.
This one should be a small patch to libssh/Makefile.

Maybe I misapplied it?

Sorry I wasn't clear - D32448 adds libfido2 and is also needed as a dependency for D32509.

To test everything you'll need:

@emaste Thanks for explaining.

Then regarding this single change here I have tested it and see no regression.

This revision is now accepted and ready to land.Oct 18 2021, 2:42 PM
This revision was automatically updated to reflect the committed changes.