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)
Thu, Nov 7, 1:13 PM
Unknown Object (File)
Sun, Oct 20, 6:30 PM
Unknown Object (File)
Fri, Oct 18, 4:39 PM
Unknown Object (File)
Fri, Oct 18, 3:45 AM
Unknown Object (File)
Thu, Oct 17, 10:39 PM
Unknown Object (File)
Tue, Oct 15, 12:42 PM
Unknown Object (File)
Sep 23 2024, 7:27 AM
Unknown Object (File)
Sep 18 2024, 5:41 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.