#53108 closed update (fixed)
openssh: Update to 7.6p1
Reported by: | danielluke (Daniel J. Luke) | Owned by: | Ionic (Mihai Moldovan) |
---|---|---|---|
Priority: | Normal | Milestone: | |
Component: | ports | Version: | |
Keywords: | Cc: | H3ik0, myrkraverk (Johann 'Myrkraverk' Oskarsson), Schamschula (Marius Schamschula), l2dy (Zero King) | |
Port: | openssh |
Description (last modified by Schamschula (Marius Schamschula))
(for 7.4p1):
As per usual, a simple version bump works for me (but I did not test the +hpn or +gsskex variants - which usually need some attention).
(for 7.5p1):
Unfortunately, a simple version bump fails earlier now (launchd.patch for channels.c fails).
(for 7.6p1):
Simple version bump with fix to launchd.patch. No support for other variants.
Attachments (3)
Change History (30)
Changed 8 years ago by danielluke (Daniel J. Luke)
Attachment: | openssh_version_bump.diff added |
---|
comment:1 Changed 8 years ago by mf2k (Frank Schima)
Keywords: | haspatch added |
---|---|
Type: | defect → update |
comment:2 Changed 8 years ago by H3ik0
Cc: | H3ik0 added |
---|
comment:3 follow-up: 5 Changed 8 years ago by myrkraverk (Johann 'Myrkraverk' Oskarsson)
There are at least two relevant CVEs so I'd like to bump up the priority.
https://bugs.chromium.org/p/project-zero/issues/detail?id=1010
https://bugs.chromium.org/p/project-zero/issues/detail?id=1009
comment:4 Changed 8 years ago by myrkraverk (Johann 'Myrkraverk' Oskarsson)
Cc: | myrkraverk added |
---|
comment:5 follow-up: 6 Changed 8 years ago by raimue (Rainer Müller)
Replying to myrkraverk:
There are at least two relevant CVEs so I'd like to bump up the priority.
https://bugs.chromium.org/p/project-zero/issues/detail?id=1010
Bug against sshd.
https://bugs.chromium.org/p/project-zero/issues/detail?id=1009
Bug against ssh-agent.
Both of these binaries are usually the version provided by Apple, unless you explicitly exposed the MacPorts version by changing your system configuration. Upgrading MacPorts will not remove the attack vector.
comment:6 Changed 8 years ago by myrkraverk (Johann 'Myrkraverk' Oskarsson)
Replying to raimue:
Both of these binaries are usually the version provided by Apple, unless you explicitly exposed the MacPorts version by changing your system configuration. Upgrading MacPorts will not remove the attack vector.
Ok, so purely installing MacPorts is not sufficient, but can I still use the provided ssh-agent by changing my system config to use it? Or is it incompatible somehow?
Right now I'm less concerned about the SSH daemon than the agent.
comment:7 Changed 8 years ago by Schamschula (Marius Schamschula)
Cc: | Schamschula added |
---|
comment:8 Changed 8 years ago by Ionic (Mihai Moldovan)
Owner: | set to Ionic |
---|---|
Status: | new → accepted |
You could, but it doesn't make a whole lot of sense. In theory, the MacPorts ssh-agent binary should be compatible to the Apple-provided one, although I've had reports of it crashing for users with me being unable to reproduce it.
The gist is that switching to the MacPorts-provided ssh-agent binary as your system daemon only really makes sense if you want to use key types that are not supported by the system version, especially on older systems, as Apple is generally not updating software they ship within a release (short of bugfixes.)
I can't promise an update soonishly, will probably take me few weeks.
comment:9 Changed 8 years ago by Schamschula (Marius Schamschula)
I'm more concerned about missing security fixes, than I am about the latest key types. Apple's sshd for Sierra currently is 7.3p1 - libressl 2.4.1, but on my El Capitan machine it is only 6.9p1 - libressl 2.1.8.
For the same reason I don't run the OS openssh under FreeBSD either (currently OpenSSH_7.2p2, OpenSSL 1.0.2j-freebsd vs. OpenSSH_7.4p1, OpenSSL 1.0.2k from the openssh-portable package).
comment:10 Changed 8 years ago by l2dy (Zero King)
Cc: | l2dy added |
---|
comment:11 Changed 8 years ago by danielluke (Daniel J. Luke)
7.5p1 is out now.
Unfortunately, a simple version bump fails earlier now (launchd.patch for channels.c fails). I don't know when I'll have time to look at it, but I'll try and get the default build working if no one beats me to it).
comment:12 Changed 8 years ago by danielluke (Daniel J. Luke)
Summary: | openssh 7.4p1 release → openssh 7.5p1 release |
---|
comment:13 Changed 8 years ago by danielluke (Daniel J. Luke)
Description: | modified (diff) |
---|
comment:14 Changed 8 years ago by danielluke (Daniel J. Luke)
Keywords: | haspatch removed |
---|
comment:15 Changed 7 years ago by ryandesign (Ryan Carsten Schmidt)
Summary: | openssh 7.5p1 release → openssh: Update to 7.6p1 |
---|
7.6p1 is now out.
comment:16 follow-up: 27 Changed 7 years ago by Schamschula (Marius Schamschula)
Beside th ewabove mention issue with the launchd patch, the gsskex variant keeps causing the update hang ups. I looks like this patch is no longer being updated upstream (Debian), and the OpenBSD folks never agreed to integrate GSSAPI support into the mainstream package (potential security issues). This means a lot of patching of the patch file...
Changed 7 years ago by Schamschula (Marius Schamschula)
Attachment: | launchd.patch added |
---|
Changed 7 years ago by Schamschula (Marius Schamschula)
Portfile for 7.6p1 only default variant fixed
comment:17 Changed 7 years ago by Schamschula (Marius Schamschula)
Description: | modified (diff) |
---|
comment:18 Changed 7 years ago by Schamschula (Marius Schamschula)
Description: | modified (diff) |
---|
comment:19 follow-up: 21 Changed 7 years ago by Ionic (Mihai Moldovan)
Please be patient. Openssh is next on my list. I'll work on it soon!
comment:20 follow-up: 22 Changed 7 years ago by danielluke (Daniel J. Luke)
I think we should consider splitting this port into a 'vanilla' openssh (minimal patching) and a version with the problematic patches that takes longer to update.
comment:21 Changed 7 years ago by Schamschula (Marius Schamschula)
Replying to Ionic:
Please be patient. Openssh is next on my list. I'll work on it soon!
Good to hear! +1
comment:22 follow-up: 24 Changed 7 years ago by Schamschula (Marius Schamschula)
Replying to danielluke:
I think we should consider splitting this port into a 'vanilla' openssh (minimal patching) and a version with the problematic patches that takes longer to update.
+1
comment:23 Changed 7 years ago by Schamschula (Marius Schamschula)
Also, please consider #54762 - merging ssh-copy-id as a subport of openssh.
comment:24 Changed 7 years ago by H3ik0
Replying to Schamschula:
Replying to danielluke:
I think we should consider splitting this port into a 'vanilla' openssh (minimal patching) and a version with the problematic patches that takes longer to update.
+1
+1
comment:25 Changed 7 years ago by Ionic (Mihai Moldovan)
Well, at least I started working on it. So far have managed to update all patches but the huge GSSKEX one. Will try to get this into shape tomorrow, do some other cleanups and finally make the newest version available.
comment:26 Changed 7 years ago by Mihai Moldovan <ionic@…>
Resolution: | → fixed |
---|---|
Status: | accepted → closed |
comment:27 Changed 7 years ago by Ionic (Mihai Moldovan)
Replying to Schamschula:
Beside the above mentioned issue with the launchd patch
Incidentally, that patch is a bit less intrusive now since some functionality has been merged upstream.
the gsskex variant keeps causing the update hang ups. I looks like this patch is no longer being updated upstream (Debian)
How did you come to this conclusion? Both Debian and RedHat/Fedora still very much maintain this patch, though RH/F has been slower to update to 7.6p1 this time around. Not a real problem, since Debian had a 7.6p1 version available and most of the changes between 7.5p1 and 7.6p1 relate to dropping SSH1 support.
and the OpenBSD folks never agreed to integrate GSSAPI support into the mainstream package (potential security issues).
Yes, but it's still a default patch in a lot of distributions and its security track record is pretty good. Most CVEs revolved around RH-specific additions.
This means a lot of patching of the patch file...
Well, yeah, it's always a pain to update this stuff...
simple version bump