Damien Miller | 82c5587 | 2011-06-23 08:20:30 +1000 | [diff] [blame] | 1 | 20110623 |
| 2 | - OpenBSD CVS Sync |
| 3 | - djm@cvs.openbsd.org 2011/06/22 21:47:28 |
| 4 | [servconf.c] |
| 5 | reuse the multistate option arrays to pretty-print options for "sshd -T" |
Damien Miller | 69ff1df | 2011-06-23 08:30:03 +1000 | [diff] [blame] | 6 | - djm@cvs.openbsd.org 2011/06/22 21:57:01 |
| 7 | [servconf.c servconf.h sshd.c sshd_config.5] |
| 8 | [configure.ac Makefile.in] |
| 9 | introduce sandboxing of the pre-auth privsep child using systrace(4). |
| 10 | |
| 11 | This introduces a new "UsePrivilegeSeparation=sandbox" option for |
| 12 | sshd_config that applies mandatory restrictions on the syscalls the |
| 13 | privsep child can perform. This prevents a compromised privsep child |
| 14 | from being used to attack other hosts (by opening sockets and proxying) |
| 15 | or probing local kernel attack surface. |
| 16 | |
| 17 | The sandbox is implemented using systrace(4) in unsupervised "fast-path" |
| 18 | mode, where a list of permitted syscalls is supplied. Any syscall not |
| 19 | on the list results in SIGKILL being sent to the privsep child. Note |
| 20 | that this requires a kernel with the new SYSTR_POLICY_KILL option. |
| 21 | |
| 22 | UsePrivilegeSeparation=sandbox will become the default in the future |
| 23 | so please start testing it now. |
| 24 | |
| 25 | feedback dtucker@; ok markus@ |
Damien Miller | 6d7b437 | 2011-06-23 08:31:57 +1000 | [diff] [blame] | 26 | - djm@cvs.openbsd.org 2011/06/22 22:08:42 |
| 27 | [channels.c channels.h clientloop.c clientloop.h mux.c ssh.c] |
| 28 | hook up a channel confirm callback to warn the user then requested X11 |
| 29 | forwarding was refused by the server; ok markus@ |
Damien Miller | 80b62e3 | 2011-06-23 19:03:18 +1000 | [diff] [blame^] | 30 | - (djm) [sandbox-null.c] Dummy sandbox for platforms that don't support |
| 31 | setrlimit(2) |
Damien Miller | 82c5587 | 2011-06-23 08:20:30 +1000 | [diff] [blame] | 32 | |
Damien Miller | 6029e07 | 2011-06-20 14:22:49 +1000 | [diff] [blame] | 33 | 20110620 |
| 34 | - OpenBSD CVS Sync |
| 35 | - djm@cvs.openbsd.org 2011/06/04 00:10:26 |
| 36 | [ssh_config.5] |
| 37 | explain IdentifyFile's semantics a little better, prompted by bz#1898 |
| 38 | ok dtucker jmc |
Damien Miller | e7ac2bd | 2011-06-20 14:23:25 +1000 | [diff] [blame] | 39 | - markus@cvs.openbsd.org 2011/06/14 22:49:18 |
| 40 | [authfile.c] |
| 41 | make sure key_parse_public/private_rsa1() no longer consumes its input |
| 42 | buffer. fixes ssh-add for passphrase-protected ssh1-keys; |
| 43 | noted by naddy@; ok djm@ |
Damien Miller | 8f0bf23 | 2011-06-20 14:42:23 +1000 | [diff] [blame] | 44 | - djm@cvs.openbsd.org 2011/06/17 21:44:31 |
| 45 | [log.c log.h monitor.c monitor.h monitor_wrap.c monitor_wrap.h sshd.c] |
| 46 | make the pre-auth privsep slave log via a socketpair shared with the |
| 47 | monitor rather than /var/empty/dev/log; ok dtucker@ deraadt@ markus@ |
Damien Miller | f145a5b | 2011-06-20 14:42:51 +1000 | [diff] [blame] | 48 | - djm@cvs.openbsd.org 2011/06/17 21:46:16 |
| 49 | [sftp-server.c] |
| 50 | the protocol version should be unsigned; bz#1913 reported by mb AT |
| 51 | smartftp.com |
Damien Miller | 3332212 | 2011-06-20 14:43:11 +1000 | [diff] [blame] | 52 | - djm@cvs.openbsd.org 2011/06/17 21:47:35 |
| 53 | [servconf.c] |
| 54 | factor out multi-choice option parsing into a parse_multistate label |
| 55 | and some support structures; ok dtucker@ |
Damien Miller | 4ac99c3 | 2011-06-20 14:43:31 +1000 | [diff] [blame] | 56 | - djm@cvs.openbsd.org 2011/06/17 21:57:25 |
| 57 | [clientloop.c] |
| 58 | setproctitle for a mux master that has been gracefully stopped; |
| 59 | bz#1911 from Bert.Wesarg AT googlemail.com |
Damien Miller | 6029e07 | 2011-06-20 14:22:49 +1000 | [diff] [blame] | 60 | |
Darren Tucker | c412c15 | 2011-06-03 10:35:23 +1000 | [diff] [blame] | 61 | 20110603 |
| 62 | - (dtucker) [README version.h contrib/caldera/openssh.spec |
| 63 | contrib/redhat/openssh.spec contrib/suse/openssh.spec] Pull the version |
| 64 | bumps from the 5.8p2 branch into HEAD. ok djm. |
Tim Rice | 90f42b0 | 2011-06-02 18:17:49 -0700 | [diff] [blame] | 65 | - (tim) [configure.ac defines.h] Run test program to detect system mail |
| 66 | directory. Add --with-maildir option to override. Fixed OpenServer 6 |
| 67 | getting it wrong. Fixed many systems having MAIL=/var/mail//username |
| 68 | ok dtucker |
Darren Tucker | c3c7227 | 2011-06-03 11:20:06 +1000 | [diff] [blame] | 69 | - (dtucker) [monitor.c] Remove the !HAVE_SOCKETPAIR case. We use socketpair |
| 70 | unconditionally in other places and the survey data we have does not show |
| 71 | any systems that use it. "nuke it" djm@ |
Damien Miller | c09182f | 2011-06-03 12:11:38 +1000 | [diff] [blame] | 72 | - (djm) [configure.ac] enable setproctitle emulation for OS X |
| 73 | - (djm) OpenBSD CVS Sync |
Damien Miller | ea2c1a4 | 2011-06-03 12:10:22 +1000 | [diff] [blame] | 74 | - djm@cvs.openbsd.org 2011/06/03 00:54:38 |
| 75 | [ssh.c] |
| 76 | bz#1883 - setproctitle() to identify mux master; patch from Bert.Wesarg |
| 77 | AT googlemail.com; ok dtucker@ |
| 78 | NB. includes additional portability code to enable setproctitle emulation |
| 79 | on platforms that don't support it. |
Darren Tucker | 3e78a51 | 2011-06-03 14:14:16 +1000 | [diff] [blame] | 80 | - dtucker@cvs.openbsd.org 2011/06/03 01:37:40 |
| 81 | [ssh-agent.c] |
| 82 | Check current parent process ID against saved one to determine if the parent |
| 83 | has exited, rather than attempting to send a zero signal, since the latter |
| 84 | won't work if the parent has changed privs. bz#1905, patch from Daniel Kahn |
| 85 | Gillmor, ok djm@ |
Darren Tucker | 260c8fb | 2011-06-03 14:17:27 +1000 | [diff] [blame] | 86 | - dtucker@cvs.openbsd.org 2011/05/31 02:01:58 |
| 87 | [regress/dynamic-forward.sh] |
| 88 | back out revs 1.6 and 1.5 since it's not reliable |
Darren Tucker | 75e035c | 2011-06-03 14:18:17 +1000 | [diff] [blame] | 89 | - dtucker@cvs.openbsd.org 2011/05/31 02:03:34 |
| 90 | [regress/dynamic-forward.sh] |
| 91 | work around startup and teardown races; caught by deraadt |
Darren Tucker | bf4d05a | 2011-06-03 14:19:02 +1000 | [diff] [blame] | 92 | - dtucker@cvs.openbsd.org 2011/06/03 00:29:52 |
| 93 | [regress/dynamic-forward.sh] |
| 94 | Retry establishing the port forwarding after a small delay, should make |
| 95 | the tests less flaky when the previous test is slow to shut down and free |
| 96 | up the port. |
Tim Rice | bc48157 | 2011-06-02 22:26:19 -0700 | [diff] [blame] | 97 | - (tim) [regress/cfgmatch.sh] Build/test out of tree fix. |
Darren Tucker | c412c15 | 2011-06-03 10:35:23 +1000 | [diff] [blame] | 98 | |
Damien Miller | d8478b6 | 2011-05-29 21:39:36 +1000 | [diff] [blame] | 99 | 20110529 |
| 100 | - (djm) OpenBSD CVS Sync |
| 101 | - djm@cvs.openbsd.org 2011/05/23 03:30:07 |
| 102 | [auth-rsa.c auth.c auth.h auth2-pubkey.c monitor.c monitor_wrap.c] |
| 103 | [pathnames.h servconf.c servconf.h sshd.8 sshd_config sshd_config.5] |
| 104 | allow AuthorizedKeysFile to specify multiple files, separated by spaces. |
| 105 | Bring back authorized_keys2 as a default search path (to avoid breaking |
| 106 | existing users of this file), but override this in sshd_config so it will |
| 107 | be no longer used on fresh installs. Maybe in 2015 we can remove it |
| 108 | entierly :) |
| 109 | |
| 110 | feedback and ok markus@ dtucker@ |
Damien Miller | 1dd66e5 | 2011-05-29 21:40:42 +1000 | [diff] [blame] | 111 | - djm@cvs.openbsd.org 2011/05/23 03:33:38 |
| 112 | [auth.c] |
| 113 | make secure_filename() spam debug logs less |
Damien Miller | 201f425 | 2011-05-29 21:41:03 +1000 | [diff] [blame] | 114 | - djm@cvs.openbsd.org 2011/05/23 03:52:55 |
| 115 | [sshconnect.c] |
| 116 | remove extra newline |
Damien Miller | b9132fc | 2011-05-29 21:41:40 +1000 | [diff] [blame] | 117 | - jmc@cvs.openbsd.org 2011/05/23 07:10:21 |
| 118 | [sshd.8 sshd_config.5] |
| 119 | tweak previous; ok djm |
Damien Miller | 04bb56e | 2011-05-29 21:42:08 +1000 | [diff] [blame] | 120 | - djm@cvs.openbsd.org 2011/05/23 07:24:57 |
| 121 | [authfile.c] |
| 122 | read in key comments for v.2 keys (though note that these are not |
| 123 | passed over the agent protocol); bz#439, based on patch from binder |
| 124 | AT arago.de; ok markus@ |
Damien Miller | 295ee63 | 2011-05-29 21:42:31 +1000 | [diff] [blame] | 125 | - djm@cvs.openbsd.org 2011/05/24 07:15:47 |
| 126 | [readconf.c readconf.h ssh.c ssh_config.5 sshconnect.c sshconnect2.c] |
| 127 | Remove undocumented legacy options UserKnownHostsFile2 and |
| 128 | GlobalKnownHostsFile2 by making UserKnownHostsFile/GlobalKnownHostsFile |
| 129 | accept multiple paths per line and making their defaults include |
| 130 | known_hosts2; ok markus |
Damien Miller | 8cb3587 | 2011-05-29 21:59:10 +1000 | [diff] [blame] | 131 | - djm@cvs.openbsd.org 2011/05/23 03:31:31 |
| 132 | [regress/cfgmatch.sh] |
| 133 | include testing of multiple/overridden AuthorizedKeysFiles |
| 134 | refactor to simply daemon start/stop and get rid of racy constructs |
Damien Miller | d8478b6 | 2011-05-29 21:39:36 +1000 | [diff] [blame] | 135 | |
Damien Miller | 14684a1 | 2011-05-20 11:23:07 +1000 | [diff] [blame] | 136 | 20110520 |
| 137 | - (djm) [session.c] call setexeccon() before executing passwd for pw |
| 138 | changes; bz#1891 reported by jchadima AT redhat.com; ok dtucker@ |
Damien Miller | 989bb7f | 2011-05-20 18:56:30 +1000 | [diff] [blame] | 139 | - (djm) [aclocal.m4 configure.ac] since gcc-4.x ignores all -Wno-options |
| 140 | options, we should corresponding -W-option when trying to determine |
| 141 | whether it is accepted. Also includes a warning fix on the program |
| 142 | fragment uses (bad main() return type). |
| 143 | bz#1900 and bz#1901 reported by g.esp AT free.fr; ok dtucker@ |
Damien Miller | ec2eaa3 | 2011-05-20 18:57:14 +1000 | [diff] [blame] | 144 | - (djm) [servconf.c] remove leftover droppings of AuthorizedKeysFile2 |
Damien Miller | 814ace0 | 2011-05-20 19:02:47 +1000 | [diff] [blame] | 145 | - OpenBSD CVS Sync |
| 146 | - djm@cvs.openbsd.org 2011/05/15 08:09:01 |
| 147 | [authfd.c monitor.c serverloop.c] |
| 148 | use FD_CLOEXEC consistently; patch from zion AT x96.org |
Damien Miller | 8f639fe | 2011-05-20 19:03:08 +1000 | [diff] [blame] | 149 | - djm@cvs.openbsd.org 2011/05/17 07:13:31 |
| 150 | [key.c] |
| 151 | fatal() if asked to generate a legacy ECDSA cert (these don't exist) |
| 152 | and fix the regress test that was trying to generate them :) |
Damien Miller | 5d74e58 | 2011-05-20 19:03:31 +1000 | [diff] [blame] | 153 | - djm@cvs.openbsd.org 2011/05/20 00:55:02 |
| 154 | [servconf.c] |
| 155 | the options TrustedUserCAKeys, RevokedKeysFile, AuthorizedKeysFile |
| 156 | and AuthorizedPrincipalsFile were not being correctly applied in |
| 157 | Match blocks, despite being overridable there; ok dtucker@ |
Damien Miller | c241190 | 2011-05-20 19:03:49 +1000 | [diff] [blame] | 158 | - dtucker@cvs.openbsd.org 2011/05/20 02:00:19 |
| 159 | [servconf.c] |
| 160 | Add comment documenting what should be after the preauth check. ok djm |
Damien Miller | f2e407e | 2011-05-20 19:04:14 +1000 | [diff] [blame] | 161 | - djm@cvs.openbsd.org 2011/05/20 03:25:45 |
| 162 | [monitor.c monitor_wrap.c servconf.c servconf.h] |
| 163 | use a macro to define which string options to copy between configs |
| 164 | for Match. This avoids problems caused by forgetting to keep three |
| 165 | code locations in perfect sync and ordering |
| 166 | |
| 167 | "this is at once beautiful and horrible" + ok dtucker@ |
Damien Miller | f67188f | 2011-05-20 19:06:48 +1000 | [diff] [blame] | 168 | - djm@cvs.openbsd.org 2011/05/17 07:13:31 |
| 169 | [regress/cert-userkey.sh] |
| 170 | fatal() if asked to generate a legacy ECDSA cert (these don't exist) |
| 171 | and fix the regress test that was trying to generate them :) |
Damien Miller | 3045b45 | 2011-05-20 19:07:45 +1000 | [diff] [blame] | 172 | - djm@cvs.openbsd.org 2011/05/20 02:43:36 |
| 173 | [cert-hostkey.sh] |
| 174 | another attempt to generate a v00 ECDSA key that broke the test |
| 175 | ID sync only - portable already had this somehow |
Damien Miller | 7b9451f | 2011-05-20 19:08:11 +1000 | [diff] [blame] | 176 | - dtucker@cvs.openbsd.org 2011/05/20 05:19:50 |
| 177 | [dynamic-forward.sh] |
| 178 | Prevent races in dynamic forwarding test; ok djm |
Damien Miller | acacced | 2011-05-20 19:08:40 +1000 | [diff] [blame] | 179 | - dtucker@cvs.openbsd.org 2011/05/20 06:32:30 |
| 180 | [dynamic-forward.sh] |
| 181 | fix dumb error in dynamic-forward test |
Damien Miller | 14684a1 | 2011-05-20 11:23:07 +1000 | [diff] [blame] | 182 | |
Damien Miller | 60432d8 | 2011-05-15 08:34:46 +1000 | [diff] [blame] | 183 | 20110515 |
| 184 | - (djm) OpenBSD CVS Sync |
| 185 | - djm@cvs.openbsd.org 2011/05/05 05:12:08 |
| 186 | [mux.c] |
| 187 | gracefully fall back when ControlPath is too large for a |
| 188 | sockaddr_un. ok markus@ as part of a larger diff |
Damien Miller | fd53abd | 2011-05-15 08:36:02 +1000 | [diff] [blame] | 189 | - dtucker@cvs.openbsd.org 2011/05/06 01:03:35 |
| 190 | [sshd_config] |
| 191 | clarify language about overriding defaults. bz#1892, from Petr Cerny |
Damien Miller | 58a77e2 | 2011-05-15 08:36:29 +1000 | [diff] [blame] | 192 | - djm@cvs.openbsd.org 2011/05/06 01:09:53 |
| 193 | [sftp.1] |
| 194 | mention that IPv6 addresses must be enclosed in square brackets; |
| 195 | bz#1845 |
Damien Miller | 78c40c3 | 2011-05-15 08:36:59 +1000 | [diff] [blame] | 196 | - djm@cvs.openbsd.org 2011/05/06 02:05:41 |
| 197 | [sshconnect2.c] |
| 198 | fix memory leak; bz#1849 ok dtucker@ |
Damien Miller | d2ac5d7 | 2011-05-15 08:43:13 +1000 | [diff] [blame] | 199 | - djm@cvs.openbsd.org 2011/05/06 21:14:05 |
| 200 | [packet.c packet.h] |
| 201 | set traffic class for IPv6 traffic as we do for IPv4 TOS; |
| 202 | patch from lionel AT mamane.lu via Colin Watson in bz#1855; |
| 203 | ok markus@ |
Damien Miller | dfc85fa | 2011-05-15 08:44:02 +1000 | [diff] [blame] | 204 | - djm@cvs.openbsd.org 2011/05/06 21:18:02 |
| 205 | [ssh.c ssh_config.5] |
| 206 | add a %L expansion (short-form of the local host name) for ControlPath; |
| 207 | sync some more expansions with LocalCommand; ok markus@ |
Damien Miller | fe92421 | 2011-05-15 08:44:45 +1000 | [diff] [blame] | 208 | - djm@cvs.openbsd.org 2011/05/06 21:31:38 |
| 209 | [readconf.c ssh_config.5] |
| 210 | support negated Host matching, e.g. |
| 211 | |
| 212 | Host *.example.org !c.example.org |
| 213 | User mekmitasdigoat |
| 214 | |
| 215 | Will match "a.example.org", "b.example.org", but not "c.example.org" |
| 216 | ok markus@ |
Damien Miller | 21771e2 | 2011-05-15 08:45:50 +1000 | [diff] [blame] | 217 | - djm@cvs.openbsd.org 2011/05/06 21:34:32 |
| 218 | [clientloop.c mux.c readconf.c readconf.h ssh.c ssh_config.5] |
| 219 | Add a RequestTTY ssh_config option to allow configuration-based |
| 220 | control over tty allocation (like -t/-T); ok markus@ |
Damien Miller | a6bbbe4 | 2011-05-15 08:46:29 +1000 | [diff] [blame] | 221 | - djm@cvs.openbsd.org 2011/05/06 21:38:58 |
| 222 | [ssh.c] |
| 223 | fix dropping from previous diff |
Damien Miller | c067f62 | 2011-05-15 08:46:54 +1000 | [diff] [blame] | 224 | - djm@cvs.openbsd.org 2011/05/06 22:20:10 |
| 225 | [PROTOCOL.mux] |
| 226 | fix numbering; from bert.wesarg AT googlemail.com |
Damien Miller | 486dd2e | 2011-05-15 08:47:18 +1000 | [diff] [blame] | 227 | - jmc@cvs.openbsd.org 2011/05/07 23:19:39 |
| 228 | [ssh_config.5] |
| 229 | - tweak previous |
| 230 | - come consistency fixes |
| 231 | ok djm |
Damien Miller | f4b32aa | 2011-05-15 08:47:43 +1000 | [diff] [blame] | 232 | - jmc@cvs.openbsd.org 2011/05/07 23:20:25 |
| 233 | [ssh.1] |
| 234 | +.It RequestTTY |
Damien Miller | 555f3b8 | 2011-05-15 08:48:05 +1000 | [diff] [blame] | 235 | - djm@cvs.openbsd.org 2011/05/08 12:52:01 |
| 236 | [PROTOCOL.mux clientloop.c clientloop.h mux.c] |
| 237 | improve our behaviour when TTY allocation fails: if we are in |
| 238 | RequestTTY=auto mode (the default), then do not treat at TTY |
| 239 | allocation error as fatal but rather just restore the local TTY |
| 240 | to cooked mode and continue. This is more graceful on devices that |
| 241 | never allocate TTYs. |
| 242 | |
| 243 | If RequestTTY is set to "yes" or "force", then failure to allocate |
| 244 | a TTY is fatal. |
| 245 | |
| 246 | ok markus@ |
Damien Miller | 3219824 | 2011-05-15 08:50:32 +1000 | [diff] [blame] | 247 | - djm@cvs.openbsd.org 2011/05/10 05:46:46 |
| 248 | [authfile.c] |
| 249 | despam debug() logs by detecting that we are trying to load a private key |
| 250 | in key_try_load_public() and returning early; ok markus@ |
Damien Miller | 7c1b2c4 | 2011-05-15 08:51:05 +1000 | [diff] [blame] | 251 | - djm@cvs.openbsd.org 2011/05/11 04:47:06 |
| 252 | [auth.c auth.h auth2-pubkey.c pathnames.h servconf.c servconf.h] |
| 253 | remove support for authorized_keys2; it is a relic from the early days |
| 254 | of protocol v.2 support and has been undocumented for many years; |
| 255 | ok markus@ |
Damien Miller | 9d276b8 | 2011-05-15 08:51:43 +1000 | [diff] [blame] | 256 | - djm@cvs.openbsd.org 2011/05/13 00:05:36 |
| 257 | [authfile.c] |
| 258 | warn on unexpected key type in key_parse_private_type() |
Damien Miller | 23f425b | 2011-05-15 08:58:15 +1000 | [diff] [blame] | 259 | - (djm) [packet.c] unbreak portability #endif |
Damien Miller | 60432d8 | 2011-05-15 08:34:46 +1000 | [diff] [blame] | 260 | |
Darren Tucker | d6548fe | 2011-05-10 11:13:36 +1000 | [diff] [blame] | 261 | 20110510 |
| 262 | - (dtucker) [openbsd-compat/openssl-compat.{c,h}] Bug #1882: fix |
| 263 | --with-ssl-engine which was broken with the change from deprecated |
| 264 | SSLeay_add_all_algorithms(). ok djm |
| 265 | |
Darren Tucker | 343f75f | 2011-05-06 10:43:50 +1000 | [diff] [blame] | 266 | 20110506 |
| 267 | - (dtucker) [openbsd-compat/regress/closefromtest.c] Bug #1875: add prototype |
| 268 | for closefrom() in test code. Report from Dan Wallis via Gentoo. |
| 269 | |
Damien Miller | 68790fe | 2011-05-05 11:19:13 +1000 | [diff] [blame] | 270 | 20110505 |
| 271 | - (djm) [defines.h] Move up include of netinet/ip.h for IPTOS |
| 272 | definitions. From des AT des.no |
Damien Miller | f22019b | 2011-05-05 13:48:37 +1000 | [diff] [blame] | 273 | - (djm) [Makefile.in WARNING.RNG aclocal.m4 buildpkg.sh.in configure.ac] |
| 274 | [entropy.c ssh-add.c ssh-agent.c ssh-keygen.c ssh-keyscan.c] |
| 275 | [ssh-keysign.c ssh-pkcs11-helper.c ssh-rand-helper.8 ssh-rand-helper.c] |
| 276 | [ssh.c ssh_prng_cmds.in sshd.c contrib/aix/buildbff.sh] |
| 277 | [regress/README.regress] Remove ssh-rand-helper and all its |
| 278 | tentacles. PRNGd seeding has been rolled into entropy.c directly. |
| 279 | Thanks to tim@ for testing on affected platforms. |
Damien Miller | 3fcdfd5 | 2011-05-05 14:04:11 +1000 | [diff] [blame] | 280 | - OpenBSD CVS Sync |
| 281 | - djm@cvs.openbsd.org 2011/03/10 02:52:57 |
Damien Miller | b2da7d1 | 2011-05-05 14:04:50 +1000 | [diff] [blame] | 282 | [auth2-gss.c auth2.c auth.h] |
Damien Miller | 3fcdfd5 | 2011-05-05 14:04:11 +1000 | [diff] [blame] | 283 | allow GSSAPI authentication to detect when a server-side failure causes |
| 284 | authentication failure and don't count such failures against MaxAuthTries; |
| 285 | bz#1244 from simon AT sxw.org.uk; ok markus@ before lock |
Damien Miller | c5219e7 | 2011-05-05 14:05:12 +1000 | [diff] [blame] | 286 | - okan@cvs.openbsd.org 2011/03/15 10:36:02 |
| 287 | [ssh-keyscan.c] |
| 288 | use timerclear macro |
| 289 | ok djm@ |
Damien Miller | 58f1baf | 2011-05-05 14:06:15 +1000 | [diff] [blame] | 290 | - stevesk@cvs.openbsd.org 2011/03/23 15:16:22 |
| 291 | [ssh-keygen.1 ssh-keygen.c] |
| 292 | Add -A option. For each of the key types (rsa1, rsa, dsa and ecdsa) |
| 293 | for which host keys do not exist, generate the host keys with the |
| 294 | default key file path, an empty passphrase, default bits for the key |
| 295 | type, and default comment. This will be used by /etc/rc to generate |
| 296 | new host keys. Idea from deraadt. |
| 297 | ok deraadt |
Damien Miller | 4a4d161 | 2011-05-05 14:06:39 +1000 | [diff] [blame] | 298 | - stevesk@cvs.openbsd.org 2011/03/23 16:24:56 |
| 299 | [ssh-keygen.1] |
| 300 | -q not used in /etc/rc now so remove statement. |
Damien Miller | 1114319 | 2011-05-05 14:13:25 +1000 | [diff] [blame] | 301 | - stevesk@cvs.openbsd.org 2011/03/23 16:50:04 |
| 302 | [ssh-keygen.c] |
| 303 | remove -d, documentation removed >10 years ago; ok markus |
Damien Miller | 3ca1eb3 | 2011-05-05 14:13:50 +1000 | [diff] [blame] | 304 | - jmc@cvs.openbsd.org 2011/03/24 15:29:30 |
| 305 | [ssh-keygen.1] |
| 306 | zap trailing whitespace; |
Damien Miller | 044f4a6 | 2011-05-05 14:14:08 +1000 | [diff] [blame] | 307 | - stevesk@cvs.openbsd.org 2011/03/24 22:14:54 |
| 308 | [ssh-keygen.c] |
| 309 | use strcasecmp() for "clear" cert permission option also; ok djm |
Damien Miller | 9147586 | 2011-05-05 14:14:34 +1000 | [diff] [blame] | 310 | - stevesk@cvs.openbsd.org 2011/03/29 18:54:17 |
| 311 | [misc.c misc.h servconf.c] |
| 312 | print ipqos friendly string for sshd -T; ok markus |
| 313 | # sshd -Tf sshd_config|grep ipqos |
| 314 | ipqos lowdelay throughput |
Damien Miller | 884b63a | 2011-05-05 14:14:52 +1000 | [diff] [blame] | 315 | - djm@cvs.openbsd.org 2011/04/12 04:23:50 |
| 316 | [ssh-keygen.c] |
| 317 | fix -Wshadow |
Damien Miller | 26b57ce | 2011-05-05 14:15:09 +1000 | [diff] [blame] | 318 | - djm@cvs.openbsd.org 2011/04/12 05:32:49 |
| 319 | [sshd.c] |
| 320 | exit with 0 status on SIGTERM; bz#1879 |
Damien Miller | 085c90f | 2011-05-05 14:15:33 +1000 | [diff] [blame] | 321 | - djm@cvs.openbsd.org 2011/04/13 04:02:48 |
| 322 | [ssh-keygen.1] |
| 323 | improve wording; bz#1861 |
Damien Miller | ad21032 | 2011-05-05 14:15:54 +1000 | [diff] [blame] | 324 | - djm@cvs.openbsd.org 2011/04/13 04:09:37 |
| 325 | [ssh-keygen.1] |
| 326 | mention valid -b sizes for ECDSA keys; bz#1862 |
Damien Miller | 6c3eec7 | 2011-05-05 14:16:22 +1000 | [diff] [blame] | 327 | - djm@cvs.openbsd.org 2011/04/17 22:42:42 |
| 328 | [PROTOCOL.mux clientloop.c clientloop.h mux.c ssh.1 ssh.c] |
| 329 | allow graceful shutdown of multiplexing: request that a mux server |
| 330 | removes its listener socket and refuse future multiplexing requests; |
| 331 | ok markus@ |
Damien Miller | 8cb1cda | 2011-05-05 14:16:56 +1000 | [diff] [blame] | 332 | - djm@cvs.openbsd.org 2011/04/18 00:46:05 |
| 333 | [ssh-keygen.c] |
| 334 | certificate options are supposed to be packed in lexical order of |
| 335 | option name (though we don't actually enforce this at present). |
| 336 | Move one up that was out of sequence |
Damien Miller | 2ce12ef | 2011-05-05 14:17:18 +1000 | [diff] [blame] | 337 | - djm@cvs.openbsd.org 2011/05/04 21:15:29 |
| 338 | [authfile.c authfile.h ssh-add.c] |
| 339 | allow "ssh-add - < key"; feedback and ok markus@ |
Tim Rice | 19d8181 | 2011-05-04 21:44:25 -0700 | [diff] [blame] | 340 | - (tim) [configure.ac] Add AC_LANG_SOURCE to OPENSSH_CHECK_CFLAG_COMPILE |
| 341 | so autoreconf 2.68 is happy. |
Tim Rice | 9abb697 | 2011-05-04 23:06:59 -0700 | [diff] [blame] | 342 | - (tim) [defines.h] Deal with platforms that do not have S_IFSOCK ok djm@ |
Damien Miller | 68790fe | 2011-05-05 11:19:13 +1000 | [diff] [blame] | 343 | |
Darren Tucker | e541aaa | 2011-02-21 21:41:29 +1100 | [diff] [blame] | 344 | 20110221 |
| 345 | - (dtucker) [contrib/cygwin/ssh-host-config] From Corinna: revamp of the |
| 346 | Cygwin-specific service installer script ssh-host-config. The actual |
| 347 | functionality is the same, the revisited version is just more |
| 348 | exact when it comes to check for problems which disallow to run |
| 349 | certain aspects of the script. So, part of this script and the also |
| 350 | rearranged service helper script library "csih" is to check if all |
| 351 | the tools required to run the script are available on the system. |
| 352 | The new script also is more thorough to inform the user why the |
| 353 | script failed. Patch from vinschen at redhat com. |
| 354 | |
Damien Miller | 0588beb | 2011-02-18 09:18:45 +1100 | [diff] [blame] | 355 | 20110218 |
| 356 | - OpenBSD CVS Sync |
| 357 | - djm@cvs.openbsd.org 2011/02/16 00:31:14 |
| 358 | [ssh-keysign.c] |
| 359 | make hostbased auth with ECDSA keys work correctly. Based on patch |
| 360 | by harvey.eneman AT oracle.com in bz#1858; ok markus@ (pre-lock) |
| 361 | |
Darren Tucker | 3b9617e | 2011-02-06 13:24:35 +1100 | [diff] [blame] | 362 | 20110206 |
| 363 | - (dtucker) [openbsd-compat/port-linux.c] Bug #1851: fix syntax error in |
| 364 | selinux code. Patch from Leonardo Chiquitto |
Darren Tucker | ea676a6 | 2011-02-06 13:31:23 +1100 | [diff] [blame] | 365 | - (dtucker) [contrib/cygwin/ssh-{host,user}-config] Add ECDSA key |
| 366 | generation and simplify. Patch from Corinna Vinschen. |
Darren Tucker | 3b9617e | 2011-02-06 13:24:35 +1100 | [diff] [blame] | 367 | |
Damien Miller | b407dd8 | 2011-02-04 11:46:39 +1100 | [diff] [blame] | 368 | 20110204 |
| 369 | - OpenBSD CVS Sync |
| 370 | - djm@cvs.openbsd.org 2011/01/31 21:42:15 |
| 371 | [PROTOCOL.mux] |
| 372 | cut'n'pasto; from bert.wesarg AT googlemail.com |
Damien Miller | 0a5f012 | 2011-02-04 11:47:01 +1100 | [diff] [blame] | 373 | - djm@cvs.openbsd.org 2011/02/04 00:44:21 |
| 374 | [key.c] |
| 375 | fix uninitialised nonce variable; reported by Mateusz Kocielski |
Damien Miller | a698127 | 2011-02-04 11:47:20 +1100 | [diff] [blame] | 376 | - djm@cvs.openbsd.org 2011/02/04 00:44:43 |
| 377 | [version.h] |
| 378 | openssh-5.8 |
Damien Miller | 0d30b09 | 2011-02-04 12:43:36 +1100 | [diff] [blame] | 379 | - (djm) [README contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 380 | [contrib/suse/openssh.spec] update versions in docs and spec files. |
| 381 | - Release OpenSSH 5.8p1 |
Damien Miller | b407dd8 | 2011-02-04 11:46:39 +1100 | [diff] [blame] | 382 | |
Damien Miller | d4a5504 | 2011-01-28 10:30:18 +1100 | [diff] [blame] | 383 | 20110128 |
| 384 | - (djm) [openbsd-compat/port-linux.c] Check whether SELinux is enabled |
| 385 | before attempting setfscreatecon(). Check whether matchpathcon() |
| 386 | succeeded before using its result. Patch from cjwatson AT debian.org; |
| 387 | bz#1851 |
| 388 | |
Tim Rice | d069c48 | 2011-01-26 12:32:12 -0800 | [diff] [blame] | 389 | 20110127 |
| 390 | - (tim) [config.guess config.sub] Sync with upstream. |
Tim Rice | 648f876 | 2011-01-26 12:38:57 -0800 | [diff] [blame] | 391 | - (tim) [configure.ac] Consistent M4 quoting throughout, updated obsolete |
| 392 | AC_TRY_COMPILE with AC_COMPILE_IFELSE, updated obsolete AC_TRY_LINK with |
| 393 | AC_LINK_IFELSE, updated obsolete AC_TRY_RUN with AC_RUN_IFELSE, misc white |
| 394 | space changes for consistency/readability. Makes autoconf 2.68 happy. |
| 395 | "Nice work" djm |
Tim Rice | d069c48 | 2011-01-26 12:32:12 -0800 | [diff] [blame] | 396 | |
Damien Miller | 71adf12 | 2011-01-25 12:16:15 +1100 | [diff] [blame] | 397 | 20110125 |
| 398 | - (djm) [configure.ac Makefile.in ssh.c openbsd-compat/port-linux.c |
| 399 | openbsd-compat/port-linux.h] Move SELinux-specific code from ssh.c to |
| 400 | port-linux.c to avoid compilation errors. Add -lselinux to ssh when |
| 401 | building with SELinux support to avoid linking failure; report from |
| 402 | amk AT spamfence.net; ok dtucker |
| 403 | |
Darren Tucker | 7924137 | 2011-01-22 09:37:01 +1100 | [diff] [blame] | 404 | 20110122 |
| 405 | - (dtucker) [configure.ac openbsd-compat/openssl-compat.{c,h}] Add |
| 406 | RSA_get_default_method() for the benefit of openssl versions that don't |
| 407 | have it (at least openssl-engine-0.9.6b). Found and tested by Kevin Brott, |
| 408 | ok djm@. |
Damien Miller | ad4b1ad | 2011-01-22 20:21:33 +1100 | [diff] [blame] | 409 | - OpenBSD CVS Sync |
| 410 | - djm@cvs.openbsd.org 2011/01/22 09:18:53 |
| 411 | [version.h] |
| 412 | crank to OpenSSH-5.7 |
Damien Miller | 966accc | 2011-01-22 20:23:10 +1100 | [diff] [blame] | 413 | - (djm) [README contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 414 | [contrib/suse/openssh.spec] update versions in docs and spec files. |
Damien Miller | 6f8f04b | 2011-01-22 20:25:11 +1100 | [diff] [blame] | 415 | - (djm) Release 5.7p1 |
Darren Tucker | 7924137 | 2011-01-22 09:37:01 +1100 | [diff] [blame] | 416 | |
Tim Rice | 15e1b4d | 2011-01-18 20:47:04 -0800 | [diff] [blame] | 417 | 20110119 |
| 418 | - (tim) [contrib/caldera/openssh.spec] Use CFLAGS from Makefile instead |
| 419 | of RPM so build completes. Signatures were changed to .asc since 4.1p1. |
Damien Miller | e323ebc | 2011-01-19 23:12:27 +1100 | [diff] [blame] | 420 | - (djm) [configure.ac] Disable ECC on OpenSSL <0.9.8g. Releases prior to |
| 421 | 0.9.8 lacked it, and 0.9.8a through 0.9.8d have proven buggy in pre- |
| 422 | release testing (random crashes and failure to load ECC keys). |
| 423 | ok dtucker@ |
Tim Rice | 15e1b4d | 2011-01-18 20:47:04 -0800 | [diff] [blame] | 424 | |
Damien Miller | 369c0e8 | 2011-01-17 10:51:40 +1100 | [diff] [blame] | 425 | 20110117 |
| 426 | - (djm) [regress/Makefile] use $TEST_SSH_KEYGEN instead of the one in |
| 427 | $PATH, fix cleanup of droppings; reported by openssh AT |
| 428 | roumenpetrov.info; ok dtucker@ |
Damien Miller | fd3669e | 2011-01-17 11:20:18 +1100 | [diff] [blame] | 429 | - (djm) [regress/agent-ptrace.sh] Fix false failure on OS X by adding |
| 430 | its unique snowflake of a gdb error to the ones we look for. |
Damien Miller | 1ccbfa8 | 2011-01-17 11:52:40 +1100 | [diff] [blame] | 431 | - (djm) [regress/agent-getpeereid.sh] leave stdout attached when running |
| 432 | ssh-add to avoid $SUDO failures on Linux |
Darren Tucker | 0c93adc | 2011-01-17 11:55:59 +1100 | [diff] [blame] | 433 | - (dtucker) [openbsd-compat/port-linux.c] Bug #1838: Add support for the new |
| 434 | Linux OOM-killer magic values that changed in 2.6.36 kernels, with fallback |
| 435 | to the old values. Feedback from vapier at gentoo org and djm, ok djm. |
Damien Miller | 5849778 | 2011-01-17 16:17:09 +1100 | [diff] [blame] | 436 | - (djm) [configure.ac regress/agent-getpeereid.sh regress/multiplex.sh] |
| 437 | [regress/sftp-glob.sh regress/test-exec.sh] Rework how feature tests are |
| 438 | disabled on platforms that do not support them; add a "config_defined()" |
| 439 | shell function that greps for defines in config.h and use them to decide |
| 440 | on feature tests. |
| 441 | Convert a couple of existing grep's over config.h to use the new function |
| 442 | Add a define "FILESYSTEM_NO_BACKSLASH" for filesystem that can't represent |
| 443 | backslash characters in filenames, enable it for Cygwin and use it to turn |
| 444 | of tests for quotes backslashes in sftp-glob.sh. |
| 445 | based on discussion with vinschen AT redhat.com and dtucker@; ok dtucker@ |
Tim Rice | 6dfcd34 | 2011-01-16 22:53:56 -0800 | [diff] [blame] | 446 | - (tim) [regress/agent-getpeereid.sh] shell portability fix. |
Darren Tucker | 263d43d | 2011-01-17 18:50:22 +1100 | [diff] [blame] | 447 | - (dtucker) [openbsd-compat/port-linux.c] Fix minor bug caught by -Werror on |
| 448 | the tinderbox. |
Darren Tucker | ea52a82 | 2011-01-17 21:15:27 +1100 | [diff] [blame] | 449 | - (dtucker) [LICENCE Makefile.in audit-bsm.c audit-linux.c audit.c audit.h |
| 450 | configure.ac defines.h loginrec.c] Bug #1402: add linux audit subsystem |
| 451 | support, based on patches from Tomas Mraz and jchadima at redhat. |
Damien Miller | 369c0e8 | 2011-01-17 10:51:40 +1100 | [diff] [blame] | 452 | |
Darren Tucker | 50c61f8 | 2011-01-16 18:28:09 +1100 | [diff] [blame] | 453 | 20110116 |
| 454 | - (dtucker) [Makefile.in configure.ac regress/kextype.sh] Skip sha256-based |
| 455 | on configurations that don't have it. |
Damien Miller | 4791f9d | 2011-01-16 23:16:53 +1100 | [diff] [blame] | 456 | - OpenBSD CVS Sync |
| 457 | - djm@cvs.openbsd.org 2011/01/16 11:50:05 |
| 458 | [clientloop.c] |
| 459 | Use atomicio when flushing protocol 1 std{out,err} buffers at |
| 460 | session close. This was a latent bug exposed by setting a SIGCHLD |
| 461 | handler and spotted by kevin.brott AT gmail.com; ok dtucker@ |
Damien Miller | 6fb6fd5 | 2011-01-16 23:17:45 +1100 | [diff] [blame] | 462 | - djm@cvs.openbsd.org 2011/01/16 11:50:36 |
| 463 | [sshconnect.c] |
| 464 | reset the SIGPIPE handler when forking to execute child processes; |
| 465 | ok dtucker@ |
Damien Miller | cfd6e4f | 2011-01-16 23:18:33 +1100 | [diff] [blame] | 466 | - djm@cvs.openbsd.org 2011/01/16 12:05:59 |
| 467 | [clientloop.c] |
| 468 | a couple more tweaks to the post-close protocol 1 stderr/stdout flush: |
| 469 | now that we use atomicio(), convert them from while loops to if statements |
| 470 | add test and cast to compile cleanly with -Wsigned |
Darren Tucker | 50c61f8 | 2011-01-16 18:28:09 +1100 | [diff] [blame] | 471 | |
Darren Tucker | 08f8388 | 2011-01-16 18:24:04 +1100 | [diff] [blame] | 472 | 20110114 |
Damien Miller | 445c9a5 | 2011-01-14 12:01:29 +1100 | [diff] [blame] | 473 | - OpenBSD CVS Sync |
| 474 | - djm@cvs.openbsd.org 2011/01/13 21:54:53 |
| 475 | [mux.c] |
| 476 | correct error messages; patch from bert.wesarg AT googlemail.com |
Damien Miller | 42747df | 2011-01-14 12:01:50 +1100 | [diff] [blame] | 477 | - djm@cvs.openbsd.org 2011/01/13 21:55:25 |
| 478 | [PROTOCOL.mux] |
| 479 | correct protocol names and add a couple of missing protocol number |
| 480 | defines; patch from bert.wesarg AT googlemail.com |
Damien Miller | e9b4048 | 2011-01-14 14:47:37 +1100 | [diff] [blame] | 481 | - (djm) [Makefile.in] Use shell test to disable ecdsa key generating in |
| 482 | host-key-force target rather than a substitution that is replaced with a |
| 483 | comment so that the Makefile.in is still a syntactically valid Makefile |
| 484 | (useful to run the distprep target) |
Tim Rice | 02d99da | 2011-01-13 22:20:27 -0800 | [diff] [blame] | 485 | - (tim) [regress/cert-hostkey.sh] Typo. Missing $ on variable name. |
Tim Rice | c5c346b | 2011-01-13 22:36:14 -0800 | [diff] [blame] | 486 | - (tim) [regress/cert-hostkey.sh] Add missing TEST_SSH_ECC guard around some |
| 487 | ecdsa bits. |
Damien Miller | 445c9a5 | 2011-01-14 12:01:29 +1100 | [diff] [blame] | 488 | |
Darren Tucker | 08f8388 | 2011-01-16 18:24:04 +1100 | [diff] [blame] | 489 | 20110113 |
Damien Miller | 1708cb7 | 2011-01-13 12:21:34 +1100 | [diff] [blame] | 490 | - (djm) [misc.c] include time.h for nanosleep() prototype |
Tim Rice | cce927c | 2011-01-12 19:06:31 -0800 | [diff] [blame] | 491 | - (tim) [Makefile.in] test the ECC bits if we have the capability. ok djm |
Tim Rice | 9b87a5c | 2011-01-12 22:35:43 -0800 | [diff] [blame] | 492 | - (tim) [Makefile.in configure.ac opensshd.init.in] Add support for generating |
| 493 | ecdsa keys. ok djm. |
Damien Miller | ff22df5 | 2011-01-13 21:05:27 +1100 | [diff] [blame] | 494 | - (djm) [entropy.c] cast OPENSSL_VERSION_NUMBER to u_long to avoid |
| 495 | gcc warning on platforms where it defaults to int |
Damien Miller | cbaf8e6 | 2011-01-13 21:08:27 +1100 | [diff] [blame] | 496 | - (djm) [regress/Makefile] add a few more generated files to the clean |
| 497 | target |
Damien Miller | 9b16086 | 2011-01-13 22:00:20 +1100 | [diff] [blame] | 498 | - (djm) [myproposal.h] Fix reversed OPENSSL_VERSION_NUMBER test and bad |
| 499 | #define that was causing diffie-hellman-group-exchange-sha256 to be |
| 500 | incorrectly disabled |
Damien Miller | 5278806 | 2011-01-13 22:05:14 +1100 | [diff] [blame] | 501 | - (djm) [regress/kextype.sh] Testing diffie-hellman-group-exchange-sha256 |
| 502 | should not depend on ECC support |
Damien Miller | 1708cb7 | 2011-01-13 12:21:34 +1100 | [diff] [blame] | 503 | |
Darren Tucker | 08f8388 | 2011-01-16 18:24:04 +1100 | [diff] [blame] | 504 | 20110112 |
Damien Miller | b66e917 | 2011-01-12 13:30:18 +1100 | [diff] [blame] | 505 | - OpenBSD CVS Sync |
| 506 | - nicm@cvs.openbsd.org 2010/10/08 21:48:42 |
| 507 | [openbsd-compat/glob.c] |
| 508 | Extend GLOB_LIMIT to cover readdir and stat and bump the malloc limit |
| 509 | from ARG_MAX to 64K. |
| 510 | Fixes glob-using programs (notably ftp) able to be triggered to hit |
| 511 | resource limits. |
| 512 | Idea from a similar NetBSD change, original problem reported by jasper@. |
| 513 | ok millert tedu jasper |
Damien Miller | 4927aaf | 2011-01-12 13:32:03 +1100 | [diff] [blame] | 514 | - djm@cvs.openbsd.org 2011/01/12 01:53:14 |
| 515 | avoid some integer overflows mostly with GLOB_APPEND and GLOB_DOOFFS |
| 516 | and sanity check arguments (these will be unnecessary when we switch |
| 517 | struct glob members from being type into to size_t in the future); |
| 518 | "looks ok" tedu@ feedback guenther@ |
Damien Miller | 945aa0c | 2011-01-12 13:34:02 +1100 | [diff] [blame] | 519 | - (djm) [configure.ac] Turn on -Wno-unused-result for gcc >= 4.4 to avoid |
| 520 | silly warnings on write() calls we don't care succeed or not. |
Damien Miller | 134d02a | 2011-01-12 16:00:37 +1100 | [diff] [blame] | 521 | - (djm) [configure.ac] Fix broken test for gcc >= 4.4 with per-compiler |
| 522 | flag tests that don't depend on gcc version at all; suggested by and |
| 523 | ok dtucker@ |
Damien Miller | b66e917 | 2011-01-12 13:30:18 +1100 | [diff] [blame] | 524 | |
Tim Rice | 076a3b9 | 2011-01-10 12:56:26 -0800 | [diff] [blame] | 525 | 20110111 |
| 526 | - (tim) [regress/host-expand.sh] Fix for building outside of read only |
| 527 | source tree. |
Damien Miller | 81ad4b1 | 2011-01-11 17:02:23 +1100 | [diff] [blame] | 528 | - (djm) [platform.c] Some missing includes that show up under -Werror |
Damien Miller | b73b6fd | 2011-01-11 17:18:56 +1100 | [diff] [blame] | 529 | - OpenBSD CVS Sync |
| 530 | - djm@cvs.openbsd.org 2011/01/08 10:51:51 |
| 531 | [clientloop.c] |
| 532 | use host and not options.hostname, as the latter may have unescaped |
| 533 | substitution characters |
Damien Miller | a256c8d | 2011-01-11 17:20:05 +1100 | [diff] [blame] | 534 | - djm@cvs.openbsd.org 2011/01/11 06:06:09 |
| 535 | [sshlogin.c] |
| 536 | fd leak on error paths; from zinovik@ |
| 537 | NB. Id sync only; we use loginrec.c that was also audited and fixed |
| 538 | recently |
Damien Miller | 821de0a | 2011-01-11 17:20:29 +1100 | [diff] [blame] | 539 | - djm@cvs.openbsd.org 2011/01/11 06:13:10 |
| 540 | [clientloop.c ssh-keygen.c sshd.c] |
| 541 | some unsigned long long casts that make things a bit easier for |
| 542 | portable without resorting to dropping PRIu64 formats everywhere |
Tim Rice | 076a3b9 | 2011-01-10 12:56:26 -0800 | [diff] [blame] | 543 | |
Damien Miller | e63b7f2 | 2011-01-09 09:19:50 +1100 | [diff] [blame] | 544 | 20110109 |
| 545 | - (djm) [Makefile.in] list ssh_host_ecdsa key in PATHSUBS; spotted by |
| 546 | openssh AT roumenpetrov.info |
| 547 | |
Damien Miller | 996384d | 2011-01-08 21:58:20 +1100 | [diff] [blame] | 548 | 20110108 |
| 549 | - (djm) [regress/keytype.sh] s/echo -n/echon/ to repair failing regress |
| 550 | test on OSX and others. Reported by imorgan AT nas.nasa.gov |
| 551 | |
Damien Miller | 322125b | 2011-01-07 09:50:08 +1100 | [diff] [blame] | 552 | 20110107 |
| 553 | - (djm) [regress/cert-hostkey.sh regress/cert-userkey.sh] fix shell test |
| 554 | for no-ECC case. Patch from cristian.ionescu-idbohrn AT axis.com |
Damien Miller | 83f8a40 | 2011-01-07 09:51:17 +1100 | [diff] [blame] | 555 | - djm@cvs.openbsd.org 2011/01/06 22:23:53 |
| 556 | [ssh.c] |
| 557 | unbreak %n expansion in LocalCommand; patch from bert.wesarg AT |
| 558 | googlemail.com; ok markus@ |
Damien Miller | 64abf31 | 2011-01-07 09:51:52 +1100 | [diff] [blame] | 559 | - djm@cvs.openbsd.org 2011/01/06 22:23:02 |
| 560 | [clientloop.c] |
| 561 | when exiting due to ServerAliveTimeout, mention the hostname that caused |
| 562 | it (useful with backgrounded controlmaster) |
Damien Miller | 7d06b00 | 2011-01-07 09:54:20 +1100 | [diff] [blame] | 563 | - djm@cvs.openbsd.org 2011/01/06 22:46:21 |
| 564 | [regress/Makefile regress/host-expand.sh] |
| 565 | regress test for LocalCommand %n expansion from bert.wesarg AT |
| 566 | googlemail.com; ok markus@ |
Damien Miller | ed3a8eb | 2011-01-07 10:02:52 +1100 | [diff] [blame] | 567 | - djm@cvs.openbsd.org 2011/01/06 23:01:35 |
| 568 | [sshconnect.c] |
| 569 | reset SIGCHLD handler to SIG_DFL when execuring LocalCommand; |
| 570 | ok markus@ |
Damien Miller | 322125b | 2011-01-07 09:50:08 +1100 | [diff] [blame] | 571 | |
Damien Miller | f121143 | 2011-01-06 22:40:30 +1100 | [diff] [blame] | 572 | 20110106 |
| 573 | - (djm) OpenBSD CVS Sync |
| 574 | - markus@cvs.openbsd.org 2010/12/08 22:46:03 |
| 575 | [scp.1 scp.c] |
| 576 | add a new -3 option to scp: Copies between two remote hosts are |
| 577 | transferred through the local host. Without this option the data |
| 578 | is copied directly between the two remote hosts. ok djm@ (bugzilla #1837) |
Damien Miller | 907998d | 2011-01-06 22:41:21 +1100 | [diff] [blame] | 579 | - jmc@cvs.openbsd.org 2010/12/09 14:13:33 |
| 580 | [scp.1 scp.c] |
| 581 | scp.1: grammer fix |
| 582 | scp.c: add -3 to usage() |
Damien Miller | 05c8997 | 2011-01-06 22:42:04 +1100 | [diff] [blame] | 583 | - markus@cvs.openbsd.org 2010/12/14 11:59:06 |
| 584 | [sshconnect.c] |
| 585 | don't mention key type in key-changed-warning, since we also print |
| 586 | this warning if a new key type appears. ok djm@ |
Damien Miller | 106079c | 2011-01-06 22:43:44 +1100 | [diff] [blame] | 587 | - djm@cvs.openbsd.org 2010/12/15 00:49:27 |
| 588 | [readpass.c] |
| 589 | fix ControlMaster=ask regression |
| 590 | reset SIGCHLD handler before fork (and restore it after) so we don't miss |
| 591 | the the askpass child's exit status. Correct test for exit status/signal to |
| 592 | account for waitpid() failure; with claudio@ ok claudio@ markus@ |
Damien Miller | de53fd0 | 2011-01-06 22:44:18 +1100 | [diff] [blame] | 593 | - djm@cvs.openbsd.org 2010/12/24 21:41:48 |
| 594 | [auth-options.c] |
| 595 | don't send the actual forced command in a debug message; ok markus deraadt |
Damien Miller | 8ad960b | 2011-01-06 22:44:44 +1100 | [diff] [blame] | 596 | - otto@cvs.openbsd.org 2011/01/04 20:44:13 |
| 597 | [ssh-keyscan.c] |
| 598 | handle ecdsa-sha2 with various key lengths; hint and ok djm@ |
Damien Miller | f121143 | 2011-01-06 22:40:30 +1100 | [diff] [blame] | 599 | |
Damien Miller | 30a69e7 | 2011-01-04 08:16:27 +1100 | [diff] [blame] | 600 | 20110104 |
| 601 | - (djm) [configure.ac Makefile.in] Use mandoc as preferred manpage |
| 602 | formatter if it is present, followed by nroff and groff respectively. |
| 603 | Fixes distprep target on OpenBSD (which has bumped groff/nroff to ports |
| 604 | in favour of mandoc). feedback and ok tim |
| 605 | |
| 606 | 20110103 |
Damien Miller | d197fd6 | 2011-01-03 14:48:14 +1100 | [diff] [blame] | 607 | - (djm) [Makefile.in] revert local hack I didn't intend to commit |
| 608 | |
| 609 | 20110102 |
Damien Miller | 4a06f92 | 2011-01-02 21:43:59 +1100 | [diff] [blame] | 610 | - (djm) [loginrec.c] Fix some fd leaks on error paths. ok dtucker |
Damien Miller | 41bccf7 | 2011-01-02 21:53:07 +1100 | [diff] [blame] | 611 | - (djm) [configure.ac] Check whether libdes is needed when building |
| 612 | with Heimdal krb5 support. On OpenBSD this library no longer exists, |
| 613 | so linking it unconditionally causes a build failure; ok dtucker |
Damien Miller | 4a06f92 | 2011-01-02 21:43:59 +1100 | [diff] [blame] | 614 | |
Damien Miller | 928362d | 2010-12-26 14:26:45 +1100 | [diff] [blame] | 615 | 20101226 |
| 616 | - (dtucker) OpenBSD CVS Sync |
| 617 | - djm@cvs.openbsd.org 2010/12/08 04:02:47 |
| 618 | [ssh_config.5 sshd_config.5] |
| 619 | explain that IPQoS arguments are separated by whitespace; iirc requested |
| 620 | by jmc@ a while back |
| 621 | |
Darren Tucker | 37bb756 | 2010-12-05 08:46:05 +1100 | [diff] [blame] | 622 | 20101205 |
| 623 | - (dtucker) openbsd-compat/openssl-compat.c] remove sleep leftover from |
| 624 | debugging. Spotted by djm. |
Darren Tucker | 7336b90 | 2010-12-05 09:00:30 +1100 | [diff] [blame] | 625 | - (dtucker) OpenBSD CVS Sync |
| 626 | - djm@cvs.openbsd.org 2010/12/03 23:49:26 |
| 627 | [schnorr.c] |
| 628 | check that g^x^q === 1 mod p; recommended by JPAKE author Feng Hao |
| 629 | (this code is still disabled, but apprently people are treating it as |
| 630 | a reference implementation) |
Darren Tucker | adab6f1 | 2010-12-05 09:01:47 +1100 | [diff] [blame] | 631 | - djm@cvs.openbsd.org 2010/12/03 23:55:27 |
| 632 | [auth-rsa.c] |
| 633 | move check for revoked keys to run earlier (in auth_rsa_key_allowed) |
| 634 | bz#1829; patch from ldv AT altlinux.org; ok markus@ |
Darren Tucker | af1f909 | 2010-12-05 09:02:47 +1100 | [diff] [blame] | 635 | - djm@cvs.openbsd.org 2010/12/04 00:18:01 |
| 636 | [sftp-server.c sftp.1 sftp-client.h sftp.c PROTOCOL sftp-client.c] |
| 637 | add a protocol extension to support a hard link operation. It is |
| 638 | available through the "ln" command in the client. The old "ln" |
| 639 | behaviour of creating a symlink is available using its "-s" option |
| 640 | or through the preexisting "symlink" command; based on a patch from |
| 641 | miklos AT szeredi.hu in bz#1555; ok markus@ |
Darren Tucker | 094f1e9 | 2010-12-05 09:03:31 +1100 | [diff] [blame] | 642 | - djm@cvs.openbsd.org 2010/12/04 13:31:37 |
| 643 | [hostfile.c] |
| 644 | fix fd leak; spotted and ok dtucker |
Darren Tucker | 4288c53 | 2010-12-05 09:45:50 +1100 | [diff] [blame] | 645 | - djm@cvs.openbsd.org 2010/12/04 00:21:19 |
| 646 | [regress/sftp-cmds.sh] |
| 647 | adjust for hard-link support |
Darren Tucker | 7e1a5a4 | 2010-12-05 09:29:31 +1100 | [diff] [blame] | 648 | - (dtucker) [regress/Makefile] Id sync. |
Darren Tucker | 37bb756 | 2010-12-05 08:46:05 +1100 | [diff] [blame] | 649 | |
Damien Miller | d89745b | 2010-12-03 10:50:26 +1100 | [diff] [blame] | 650 | 20101204 |
| 651 | - (djm) [openbsd-compat/bindresvport.c] Use arc4random_uniform(range) |
| 652 | instead of (arc4random() % range) |
Darren Tucker | ebdef76 | 2010-12-04 23:20:50 +1100 | [diff] [blame] | 653 | - (dtucker) [configure.ac moduli.c openbsd-compat/openssl-compat.{c,h}] Add |
| 654 | shims for the new, non-deprecated OpenSSL key generation functions for |
| 655 | platforms that don't have the new interfaces. |
Damien Miller | d89745b | 2010-12-03 10:50:26 +1100 | [diff] [blame] | 656 | |
Damien Miller | 188ea81 | 2010-12-01 11:50:14 +1100 | [diff] [blame] | 657 | 20101201 |
| 658 | - OpenBSD CVS Sync |
| 659 | - deraadt@cvs.openbsd.org 2010/11/20 05:12:38 |
| 660 | [auth2-pubkey.c] |
| 661 | clean up cases of ;; |
Damien Miller | 2cd6293 | 2010-12-01 11:50:35 +1100 | [diff] [blame] | 662 | - djm@cvs.openbsd.org 2010/11/21 01:01:13 |
| 663 | [clientloop.c misc.c misc.h ssh-agent.1 ssh-agent.c] |
| 664 | honour $TMPDIR for client xauth and ssh-agent temporary directories; |
| 665 | feedback and ok markus@ |
Damien Miller | a232792 | 2010-12-01 12:01:21 +1100 | [diff] [blame] | 666 | - djm@cvs.openbsd.org 2010/11/21 10:57:07 |
| 667 | [authfile.c] |
| 668 | Refactor internals of private key loading and saving to work on memory |
| 669 | buffers rather than directly on files. This will make a few things |
| 670 | easier to do in the future; ok markus@ |
Damien Miller | 6a740e7 | 2010-12-01 12:01:51 +1100 | [diff] [blame] | 671 | - djm@cvs.openbsd.org 2010/11/23 02:35:50 |
| 672 | [auth.c] |
| 673 | use strict_modes already passed as function argument over referencing |
| 674 | global options.strict_modes |
Damien Miller | d0fdd68 | 2010-12-01 12:02:14 +1100 | [diff] [blame] | 675 | - djm@cvs.openbsd.org 2010/11/23 23:57:24 |
| 676 | [clientloop.c] |
| 677 | avoid NULL deref on receiving a channel request on an unknown or invalid |
| 678 | channel; report bz#1842 from jchadima AT redhat.com; ok dtucker@ |
Damien Miller | b7f827a | 2010-12-01 12:02:35 +1100 | [diff] [blame] | 679 | - djm@cvs.openbsd.org 2010/11/24 01:24:14 |
| 680 | [channels.c] |
| 681 | remove a debug() that pollutes stderr on client connecting to a server |
| 682 | in debug mode (channel_close_fds is called transitively from the session |
| 683 | code post-fork); bz#1719, ok dtucker |
Damien Miller | f80c3de | 2010-12-01 12:02:59 +1100 | [diff] [blame] | 684 | - djm@cvs.openbsd.org 2010/11/25 04:10:09 |
| 685 | [session.c] |
| 686 | replace close() loop for fds 3->64 with closefrom(); |
| 687 | ok markus deraadt dtucker |
Damien Miller | 87dc0a4 | 2010-12-01 12:03:19 +1100 | [diff] [blame] | 688 | - djm@cvs.openbsd.org 2010/11/26 05:52:49 |
| 689 | [scp.c] |
| 690 | Pass through ssh command-line flags and options when doing remote-remote |
| 691 | transfers, e.g. to enable agent forwarding which is particularly useful |
| 692 | in this case; bz#1837 ok dtucker@ |
Damien Miller | 03c0e53 | 2010-12-01 12:03:39 +1100 | [diff] [blame] | 693 | - markus@cvs.openbsd.org 2010/11/29 18:57:04 |
| 694 | [authfile.c] |
| 695 | correctly load comment for encrypted rsa1 keys; |
| 696 | report/fix Joachim Schipper; ok djm@ |
Damien Miller | d925dcd | 2010-12-01 12:21:51 +1100 | [diff] [blame] | 697 | - djm@cvs.openbsd.org 2010/11/29 23:45:51 |
| 698 | [auth.c hostfile.c hostfile.h ssh.c ssh_config.5 sshconnect.c] |
| 699 | [sshconnect.h sshconnect2.c] |
| 700 | automatically order the hostkeys requested by the client based on |
| 701 | which hostkeys are already recorded in known_hosts. This avoids |
| 702 | hostkey warnings when connecting to servers with new ECDSA keys |
| 703 | that are preferred by default; with markus@ |
Damien Miller | 188ea81 | 2010-12-01 11:50:14 +1100 | [diff] [blame] | 704 | |
Darren Tucker | d995712 | 2010-11-24 10:09:13 +1100 | [diff] [blame] | 705 | 20101124 |
| 706 | - (dtucker) [platform.c session.c] Move the getluid call out of session.c and |
| 707 | into the platform-specific code Only affects SCO, tested by and ok tim@. |
Damien Miller | 88e341e | 2010-11-24 10:36:15 +1100 | [diff] [blame] | 708 | - (djm) [loginrec.c] Relax permission requirement on btmp logs to allow |
| 709 | group read/write. ok dtucker@ |
Darren Tucker | 4b6cbf7 | 2010-11-24 10:46:37 +1100 | [diff] [blame] | 710 | - (dtucker) [packet.c] Remove redundant local declaration of "int tos". |
Damien Miller | 73de86a | 2010-11-24 10:50:04 +1100 | [diff] [blame] | 711 | - (djm) [defines.h] Add IP DSCP defines |
Darren Tucker | d995712 | 2010-11-24 10:09:13 +1100 | [diff] [blame] | 712 | |
Darren Tucker | 9e0ff7a | 2010-11-22 17:59:00 +1100 | [diff] [blame] | 713 | 20101122 |
| 714 | - (dtucker) Bug #1840: fix warning when configuring --with-ssl-engine, patch |
| 715 | from vapier at gentoo org. |
| 716 | |
Damien Miller | 7a221a1 | 2010-11-20 15:14:29 +1100 | [diff] [blame] | 717 | 20101120 |
| 718 | - OpenBSD CVS Sync |
| 719 | - djm@cvs.openbsd.org 2010/11/05 02:46:47 |
| 720 | [packet.c] |
| 721 | whitespace KNF |
Damien Miller | 4499f4c | 2010-11-20 15:15:49 +1100 | [diff] [blame] | 722 | - djm@cvs.openbsd.org 2010/11/10 01:33:07 |
| 723 | [kexdhc.c kexdhs.c kexgexc.c kexgexs.c key.c moduli.c] |
| 724 | use only libcrypto APIs that are retained with OPENSSL_NO_DEPRECATED. |
| 725 | these have been around for years by this time. ok markus |
Damien Miller | 0dac6fb | 2010-11-20 15:19:38 +1100 | [diff] [blame] | 726 | - djm@cvs.openbsd.org 2010/11/13 23:27:51 |
| 727 | [clientloop.c misc.c misc.h packet.c packet.h readconf.c readconf.h] |
| 728 | [servconf.c servconf.h session.c ssh.c ssh_config.5 sshd_config.5] |
| 729 | allow ssh and sshd to set arbitrary TOS/DSCP/QoS values instead of |
| 730 | hardcoding lowdelay/throughput. |
| 731 | |
| 732 | bz#1733 patch from philipp AT redfish-solutions.com; ok markus@ deraadt@ |
Damien Miller | 8e1ea4e | 2010-11-20 15:20:10 +1100 | [diff] [blame] | 733 | - jmc@cvs.openbsd.org 2010/11/15 07:40:14 |
| 734 | [ssh_config.5] |
| 735 | libary -> library; |
Damien Miller | 0a18473 | 2010-11-20 15:21:03 +1100 | [diff] [blame] | 736 | - jmc@cvs.openbsd.org 2010/11/18 15:01:00 |
| 737 | [scp.1 sftp.1 ssh.1 sshd_config.5] |
| 738 | add IPQoS to the various -o lists, and zap some trailing whitespace; |
Damien Miller | 7a221a1 | 2010-11-20 15:14:29 +1100 | [diff] [blame] | 739 | |
Damien Miller | dd190dd | 2010-11-11 14:17:02 +1100 | [diff] [blame] | 740 | 20101111 |
| 741 | - (djm) [servconf.c ssh-add.c ssh-keygen.c] don't look for ECDSA keys on |
| 742 | platforms that don't support ECC. Fixes some spurious warnings reported |
| 743 | by tim@ |
| 744 | |
Tim Rice | e426f5e | 2010-11-08 09:15:14 -0800 | [diff] [blame] | 745 | 20101109 |
| 746 | - (tim) [regress/kextype.sh] Not all platforms have time in /usr/bin. |
| 747 | Feedback from dtucker@ |
Tim Rice | c7a8af0 | 2010-11-08 14:26:23 -0800 | [diff] [blame] | 748 | - (tim) [configure.ac openbsd-compat/bsd-misc.h openbsd-compat/bsd-misc.c] Add |
| 749 | support for platforms missing isblank(). ok djm@ |
Tim Rice | e426f5e | 2010-11-08 09:15:14 -0800 | [diff] [blame] | 750 | |
Tim Rice | 522262f | 2010-11-07 13:00:27 -0800 | [diff] [blame] | 751 | 20101108 |
| 752 | - (tim) [regress/Makefile] Fixes to allow building/testing outside source |
| 753 | tree. |
Tim Rice | c10aeaa | 2010-11-07 13:03:11 -0800 | [diff] [blame] | 754 | - (tim) [regress/kextype.sh] Shell portability fix. |
Tim Rice | 522262f | 2010-11-07 13:00:27 -0800 | [diff] [blame] | 755 | |
Darren Tucker | d1ece6e | 2010-11-07 18:05:54 +1100 | [diff] [blame] | 756 | 20101107 |
| 757 | - (dtucker) [platform.c] includes.h instead of defines.h so that we get |
| 758 | the correct typedefs. |
| 759 | |
Damien Miller | 3a0e9f6 | 2010-11-05 10:16:34 +1100 | [diff] [blame] | 760 | 20101105 |
Damien Miller | 34ee420 | 2010-11-05 10:52:37 +1100 | [diff] [blame] | 761 | - (djm) [loginrec.c loginrec.h] Use correct uid_t/pid_t types instead of |
| 762 | int. Should fix bz#1817 cleanly; ok dtucker@ |
Damien Miller | 3a0e9f6 | 2010-11-05 10:16:34 +1100 | [diff] [blame] | 763 | - OpenBSD CVS Sync |
| 764 | - djm@cvs.openbsd.org 2010/09/22 12:26:05 |
| 765 | [regress/Makefile regress/kextype.sh] |
| 766 | regress test for each of the key exchange algorithms that we support |
Damien Miller | b472a90 | 2010-11-05 10:19:49 +1100 | [diff] [blame] | 767 | - djm@cvs.openbsd.org 2010/10/28 11:22:09 |
| 768 | [authfile.c key.c key.h ssh-keygen.c] |
| 769 | fix a possible NULL deref on loading a corrupt ECDH key |
| 770 | |
| 771 | store ECDH group information in private keys files as "named groups" |
| 772 | rather than as a set of explicit group parameters (by setting |
| 773 | the OPENSSL_EC_NAMED_CURVE flag). This makes for shorter key files and |
| 774 | retrieves the group's OpenSSL NID that we need for various things. |
Damien Miller | 55fa565 | 2010-11-05 10:20:14 +1100 | [diff] [blame] | 775 | - jmc@cvs.openbsd.org 2010/10/28 18:33:28 |
| 776 | [scp.1 ssh-add.1 ssh-keygen.1 ssh.1 ssh_config.5 sshd.8 sshd_config.5] |
| 777 | knock out some "-*- nroff -*-" lines; |
Damien Miller | 0733121 | 2010-11-05 10:20:31 +1100 | [diff] [blame] | 778 | - djm@cvs.openbsd.org 2010/11/04 02:45:34 |
| 779 | [sftp-server.c] |
| 780 | umask should be parsed as octal. reported by candland AT xmission.com; |
| 781 | ok markus@ |
Darren Tucker | 9752835 | 2010-11-05 12:03:05 +1100 | [diff] [blame] | 782 | - (dtucker) [configure.ac platform.{c,h} session.c |
| 783 | openbsd-compat/port-solaris.{c,h}] Bug #1824: Add Solaris Project support. |
| 784 | Patch from cory.erickson at csu mnscu edu with a bit of rework from me. |
| 785 | ok djm@ |
Darren Tucker | 920612e | 2010-11-05 12:36:15 +1100 | [diff] [blame] | 786 | - (dtucker) [platform.c platform.h session.c] Add a platform hook to run |
| 787 | after the user's groups are established and move the selinux calls into it. |
Darren Tucker | 4db3807 | 2010-11-05 12:41:13 +1100 | [diff] [blame] | 788 | - (dtucker) [platform.c session.c] Move the AIX setpcred+chroot hack into |
| 789 | platform.c |
Darren Tucker | 44a97be | 2010-11-05 12:45:18 +1100 | [diff] [blame] | 790 | - (dtucker) [platform.c session.c] Move the BSDI setpgrp into platform.c. |
Darren Tucker | fd4d8aa | 2010-11-05 12:50:41 +1100 | [diff] [blame] | 791 | - (dtucker) [platform.c] Only call setpgrp on BSDI if running as root to |
| 792 | retain previous behavior. |
Darren Tucker | 728d837 | 2010-11-05 13:00:05 +1100 | [diff] [blame] | 793 | - (dtucker) [platform.c session.c] Move the PAM credential establishment for |
| 794 | the LOGIN_CAP case into platform.c. |
Darren Tucker | 7a8afe3 | 2010-11-05 13:07:24 +1100 | [diff] [blame] | 795 | - (dtucker) platform.c session.c] Move the USE_LIBIAF fragment into |
| 796 | platform.c |
Darren Tucker | 0b2ee64 | 2010-11-05 13:29:25 +1100 | [diff] [blame] | 797 | - (dtucker) [platform.c session.c] Move aix_usrinfo frament into platform.c. |
| 798 | - (dtucker) [platform.c session.c] Move irix setusercontext fragment into |
| 799 | platform.c. |
Darren Tucker | cc12418 | 2010-11-05 13:32:52 +1100 | [diff] [blame] | 800 | - (dtucker) [platform.c session.c] Move PAM credential establishment for the |
| 801 | non-LOGIN_CAP case into platform.c. |
Darren Tucker | b12fe27 | 2010-11-05 14:47:01 +1100 | [diff] [blame] | 802 | - (dtucker) [platform.c platform.h session.c] Move the Cygwin special-case |
| 803 | check into platform.c |
Darren Tucker | b69e033 | 2010-11-05 18:19:15 +1100 | [diff] [blame] | 804 | - (dtucker) [regress/keytype.sh] Import new test. |
Darren Tucker | eab5f0d | 2010-11-05 18:23:38 +1100 | [diff] [blame] | 805 | - (dtucker) [Makefile configure.ac regress/Makefile regress/keytype.sh] |
| 806 | Import recent changes to regress/Makefile, pass a flag to enable ECC tests |
| 807 | from configure through to regress/Makefile and use it in the tests. |
Darren Tucker | 345178d | 2010-11-05 18:35:52 +1100 | [diff] [blame] | 808 | - (dtucker) [regress/kextype.sh] Add missing "test". |
Darren Tucker | f619d1c | 2010-11-05 18:41:50 +1100 | [diff] [blame] | 809 | - (dtucker) [regress/kextype.sh] Make sha256 test depend on ECC. This is not |
| 810 | strictly correct since while ECC requires sha256 the reverse is not true |
| 811 | however it does prevent spurious test failures. |
Darren Tucker | 9283d8c | 2010-11-05 18:56:08 +1100 | [diff] [blame] | 812 | - (dtucker) [platform.c] Need servconf.h and extern options. |
Damien Miller | 3a0e9f6 | 2010-11-05 10:16:34 +1100 | [diff] [blame] | 813 | |
Tim Rice | bdd3e67 | 2010-10-24 18:35:55 -0700 | [diff] [blame] | 814 | 20101025 |
| 815 | - (tim) [openbsd-compat/glob.h] Remove sys/cdefs.h include that came with |
| 816 | 1.12 to unbreak Solaris build. |
| 817 | ok djm@ |
Darren Tucker | 54b1f31 | 2010-10-25 16:54:28 +1100 | [diff] [blame] | 818 | - (dtucker) [defines.h] Use SIZE_T_MAX for SIZE_MAX for platforms that have a |
| 819 | native one. |
Tim Rice | bdd3e67 | 2010-10-24 18:35:55 -0700 | [diff] [blame] | 820 | |
Darren Tucker | a539393 | 2010-10-24 10:47:30 +1100 | [diff] [blame] | 821 | 20101024 |
| 822 | - (dtucker) [includes.h] Add missing ifdef GLOB_HAS_GL_STATV to fix build. |
Darren Tucker | bfd9b1b | 2010-10-24 11:19:26 +1100 | [diff] [blame] | 823 | - (dtucker) [regress/cert-hostkey.sh] Disable ECC-based tests on platforms |
| 824 | which don't have ECC support in libcrypto. |
Darren Tucker | d633fef | 2010-10-24 11:33:07 +1100 | [diff] [blame] | 825 | - (dtucker) [regress/cert-userkey.sh] Disable ECC-based tests on platforms |
| 826 | which don't have ECC support in libcrypto. |
Darren Tucker | 7bc236d | 2010-10-24 11:58:43 +1100 | [diff] [blame] | 827 | - (dtucker) [defines.h] Add SIZE_MAX for the benefit of platforms that don't |
| 828 | have it. |
Darren Tucker | d78739a | 2010-10-24 10:56:32 +1100 | [diff] [blame] | 829 | - (dtucker) OpenBSD CVS Sync |
| 830 | - sthen@cvs.openbsd.org 2010/10/23 22:06:12 |
| 831 | [sftp.c] |
| 832 | escape '[' in filename tab-completion; fix a type while there. |
| 833 | ok djm@ |
Darren Tucker | a539393 | 2010-10-24 10:47:30 +1100 | [diff] [blame] | 834 | |
Damien Miller | 68512c0 | 2010-10-21 15:21:11 +1100 | [diff] [blame] | 835 | 20101021 |
| 836 | - OpenBSD CVS Sync |
| 837 | - dtucker@cvs.openbsd.org 2010/10/12 02:22:24 |
| 838 | [mux.c] |
| 839 | Typo in confirmation message. bz#1827, patch from imorgan at |
| 840 | nas nasa gov |
Damien Miller | 6fd2d7d | 2010-10-21 15:27:14 +1100 | [diff] [blame] | 841 | - djm@cvs.openbsd.org 2010/08/31 12:24:09 |
| 842 | [regress/cert-hostkey.sh regress/cert-userkey.sh] |
| 843 | tests for ECDSA certificates |
Damien Miller | 68512c0 | 2010-10-21 15:21:11 +1100 | [diff] [blame] | 844 | |
Damien Miller | 1f78980 | 2010-10-11 22:35:22 +1100 | [diff] [blame] | 845 | 20101011 |
Damien Miller | 47e57bf | 2010-10-12 13:28:12 +1100 | [diff] [blame] | 846 | - (djm) [canohost.c] Zero a4 instead of addr to better match type. |
| 847 | bz#1825, reported by foo AT mailinator.com |
Damien Miller | 9c0c31d | 2010-10-12 13:30:44 +1100 | [diff] [blame] | 848 | - (djm) [sshconnect.c] Need signal.h for prototype for kill(2) |
Damien Miller | 47e57bf | 2010-10-12 13:28:12 +1100 | [diff] [blame] | 849 | |
| 850 | 20101011 |
Damien Miller | 1f78980 | 2010-10-11 22:35:22 +1100 | [diff] [blame] | 851 | - (djm) [configure.ac] Use = instead of == in shell tests. Patch from |
| 852 | dr AT vasco.com |
| 853 | |
Damien Miller | aa18063 | 2010-10-07 21:25:27 +1100 | [diff] [blame] | 854 | 20101007 |
Damien Miller | 9a3d0dc | 2010-10-07 22:06:42 +1100 | [diff] [blame] | 855 | - (djm) [ssh-agent.c] Fix type for curve name. |
Damien Miller | aa18063 | 2010-10-07 21:25:27 +1100 | [diff] [blame] | 856 | - (djm) OpenBSD CVS Sync |
| 857 | - matthew@cvs.openbsd.org 2010/09/24 13:33:00 |
| 858 | [misc.c misc.h configure.ac openbsd-compat/openbsd-compat.h] |
| 859 | [openbsd-compat/timingsafe_bcmp.c] |
| 860 | Add timingsafe_bcmp(3) to libc, mention that it's already in the |
| 861 | kernel in kern(9), and remove it from OpenSSH. |
| 862 | ok deraadt@, djm@ |
| 863 | NB. re-added under openbsd-compat/ for portable OpenSSH |
Damien Miller | a6e121a | 2010-10-07 21:39:17 +1100 | [diff] [blame] | 864 | - djm@cvs.openbsd.org 2010/09/25 09:30:16 |
| 865 | [sftp.c configure.ac openbsd-compat/glob.c openbsd-compat/glob.h] |
| 866 | make use of new glob(3) GLOB_KEEPSTAT extension to save extra server |
| 867 | rountrips to fetch per-file stat(2) information. |
| 868 | NB. update openbsd-compat/ glob(3) implementation from OpenBSD libc to |
| 869 | match. |
Damien Miller | 68e2e56 | 2010-10-07 21:39:55 +1100 | [diff] [blame] | 870 | - djm@cvs.openbsd.org 2010/09/26 22:26:33 |
| 871 | [sftp.c] |
| 872 | when performing an "ls" in columnated (short) mode, only call |
| 873 | ioctl(TIOCGWINSZ) once to get the window width instead of per- |
| 874 | filename |
Damien Miller | c54b02c | 2010-10-07 21:40:17 +1100 | [diff] [blame] | 875 | - djm@cvs.openbsd.org 2010/09/30 11:04:51 |
| 876 | [servconf.c] |
| 877 | prevent free() of string in .rodata when overriding AuthorizedKeys in |
| 878 | a Match block; patch from rein AT basefarm.no |
Damien Miller | 9a3d0dc | 2010-10-07 22:06:42 +1100 | [diff] [blame] | 879 | - djm@cvs.openbsd.org 2010/10/01 23:05:32 |
| 880 | [cipher-3des1.c cipher-bf1.c cipher-ctr.c openbsd-compat/openssl-compat.h] |
| 881 | adapt to API changes in openssl-1.0.0a |
| 882 | NB. contains compat code to select correct API for older OpenSSL |
Damien Miller | 38d9a96 | 2010-10-07 22:07:11 +1100 | [diff] [blame] | 883 | - djm@cvs.openbsd.org 2010/10/05 05:13:18 |
| 884 | [sftp.c sshconnect.c] |
| 885 | use default shell /bin/sh if $SHELL is ""; ok markus@ |
Damien Miller | a41ccca | 2010-10-07 22:07:32 +1100 | [diff] [blame] | 886 | - djm@cvs.openbsd.org 2010/10/06 06:39:28 |
| 887 | [clientloop.c ssh.c sshconnect.c sshconnect.h] |
| 888 | kill proxy command on fatal() (we already kill it on clean exit); |
| 889 | ok markus@ |
Damien Miller | 45fcdaa | 2010-10-07 22:07:58 +1100 | [diff] [blame] | 890 | - djm@cvs.openbsd.org 2010/10/06 21:10:21 |
| 891 | [sshconnect.c] |
| 892 | swapped args to kill(2) |
Damien Miller | 37f4f18 | 2010-10-07 22:10:38 +1100 | [diff] [blame] | 893 | - (djm) [openbsd-compat/glob.c] restore ARG_MAX compat code. |
Damien Miller | 80e9953 | 2010-10-07 22:12:08 +1100 | [diff] [blame] | 894 | - (djm) [cipher-acss.c] Add missing header. |
Damien Miller | 88b844f | 2010-10-07 22:19:23 +1100 | [diff] [blame] | 895 | - (djm) [openbsd-compat/Makefile.in] Actually link timingsafe_bcmp |
Damien Miller | aa18063 | 2010-10-07 21:25:27 +1100 | [diff] [blame] | 896 | |
Damien Miller | 6186bbc | 2010-09-24 22:00:54 +1000 | [diff] [blame] | 897 | 20100924 |
| 898 | - (djm) OpenBSD CVS Sync |
| 899 | - naddy@cvs.openbsd.org 2010/09/10 15:19:29 |
| 900 | [ssh-keygen.1] |
| 901 | * mention ECDSA in more places |
| 902 | * less repetition in FILES section |
| 903 | * SSHv1 keys are still encrypted with 3DES |
| 904 | help and ok jmc@ |
Damien Miller | 1ca9469 | 2010-09-24 22:01:22 +1000 | [diff] [blame] | 905 | - djm@cvs.openbsd.org 2010/09/11 21:44:20 |
| 906 | [ssh.1] |
| 907 | mention RFC 5656 for ECC stuff |
Damien Miller | 881adf7 | 2010-09-24 22:01:54 +1000 | [diff] [blame] | 908 | - jmc@cvs.openbsd.org 2010/09/19 21:30:05 |
| 909 | [sftp.1] |
| 910 | more wacky macro fixing; |
Damien Miller | 857b02e | 2010-09-24 22:02:56 +1000 | [diff] [blame] | 911 | - djm@cvs.openbsd.org 2010/09/20 04:41:47 |
| 912 | [ssh.c] |
| 913 | install a SIGCHLD handler to reap expiried child process; ok markus@ |
Damien Miller | f7540cd | 2010-09-24 22:03:24 +1000 | [diff] [blame] | 914 | - djm@cvs.openbsd.org 2010/09/20 04:50:53 |
| 915 | [jpake.c schnorr.c] |
| 916 | check that received values are smaller than the group size in the |
| 917 | disabled and unfinished J-PAKE code. |
| 918 | avoids catastrophic security failure found by Sebastien Martini |
Damien Miller | 18e1cab | 2010-09-24 22:07:17 +1000 | [diff] [blame] | 919 | - djm@cvs.openbsd.org 2010/09/20 04:54:07 |
| 920 | [jpake.c] |
| 921 | missing #include |
Damien Miller | 603134e | 2010-09-24 22:07:55 +1000 | [diff] [blame] | 922 | - djm@cvs.openbsd.org 2010/09/20 07:19:27 |
| 923 | [mux.c] |
| 924 | "atomically" create the listening mux socket by binding it on a temorary |
| 925 | name and then linking it into position after listen() has succeeded. |
| 926 | this allows the mux clients to determine that the server socket is |
| 927 | either ready or stale without races. stale server sockets are now |
| 928 | automatically removed |
| 929 | ok deraadt |
Damien Miller | d5f62bf | 2010-09-24 22:11:14 +1000 | [diff] [blame] | 930 | - djm@cvs.openbsd.org 2010/09/22 05:01:30 |
| 931 | [kex.c kex.h kexecdh.c kexecdhc.c kexecdhs.c readconf.c readconf.h] |
| 932 | [servconf.c servconf.h ssh_config.5 sshconnect2.c sshd.c sshd_config.5] |
| 933 | add a KexAlgorithms knob to the client and server configuration to allow |
| 934 | selection of which key exchange methods are used by ssh(1) and sshd(8) |
| 935 | and their order of preference. |
| 936 | ok markus@ |
Damien Miller | 7fe2b1f | 2010-09-24 22:11:53 +1000 | [diff] [blame] | 937 | - jmc@cvs.openbsd.org 2010/09/22 08:30:08 |
| 938 | [ssh.1 ssh_config.5] |
| 939 | ssh.1: add kexalgorithms to the -o list |
| 940 | ssh_config.5: format the kexalgorithms in a more consistent |
| 941 | (prettier!) way |
| 942 | ok djm |
Damien Miller | 65e42f8 | 2010-09-24 22:15:11 +1000 | [diff] [blame] | 943 | - djm@cvs.openbsd.org 2010/09/22 22:58:51 |
| 944 | [atomicio.c atomicio.h misc.c misc.h scp.c sftp-client.c] |
| 945 | [sftp-client.h sftp.1 sftp.c] |
| 946 | add an option per-read/write callback to atomicio |
| 947 | |
| 948 | factor out bandwidth limiting code from scp(1) into a generic bandwidth |
| 949 | limiter that can be attached using the atomicio callback mechanism |
| 950 | |
| 951 | add a bandwidth limit option to sftp(1) using the above |
| 952 | "very nice" markus@ |
Damien Miller | 56883e1 | 2010-09-24 22:15:39 +1000 | [diff] [blame] | 953 | - jmc@cvs.openbsd.org 2010/09/23 13:34:43 |
| 954 | [sftp.c] |
| 955 | add [-l limit] to usage(); |
Damien Miller | 2beb32f | 2010-09-24 22:16:03 +1000 | [diff] [blame] | 956 | - jmc@cvs.openbsd.org 2010/09/23 13:36:46 |
| 957 | [scp.1 sftp.1] |
| 958 | add KexAlgorithms to the -o list; |
Damien Miller | 6186bbc | 2010-09-24 22:00:54 +1000 | [diff] [blame] | 959 | |
Damien Miller | 4314c2b | 2010-09-10 11:12:09 +1000 | [diff] [blame] | 960 | 20100910 |
Darren Tucker | 50e3bab | 2010-09-10 10:30:25 +1000 | [diff] [blame] | 961 | - (dtucker) [openbsd-compat/port-linux.c] Check is_selinux_enabled for exact |
| 962 | return code since it can apparently return -1 under some conditions. From |
| 963 | openssh bugs werbittewas de, ok djm@ |
Damien Miller | 4314c2b | 2010-09-10 11:12:09 +1000 | [diff] [blame] | 964 | - OpenBSD CVS Sync |
| 965 | - djm@cvs.openbsd.org 2010/08/31 12:33:38 |
| 966 | [ssh-add.c ssh-agent.c ssh-keygen.c ssh-keysign.c ssh.c sshd.c] |
| 967 | reintroduce commit from tedu@, which I pulled out for release |
| 968 | engineering: |
| 969 | OpenSSL_add_all_algorithms is the name of the function we have a |
| 970 | man page for, so use that. ok djm |
Damien Miller | de735ea | 2010-09-10 11:12:38 +1000 | [diff] [blame] | 971 | - jmc@cvs.openbsd.org 2010/08/31 17:40:54 |
| 972 | [ssh-agent.1] |
| 973 | fix some macro abuse; |
Damien Miller | d442790 | 2010-09-10 11:15:10 +1000 | [diff] [blame] | 974 | - jmc@cvs.openbsd.org 2010/08/31 21:14:58 |
| 975 | [ssh.1] |
| 976 | small text tweak to accommodate previous; |
Damien Miller | e13cadf | 2010-09-10 11:15:33 +1000 | [diff] [blame] | 977 | - naddy@cvs.openbsd.org 2010/09/01 15:21:35 |
| 978 | [servconf.c] |
| 979 | pick up ECDSA host key by default; ok djm@ |
Damien Miller | 390f153 | 2010-09-10 11:17:54 +1000 | [diff] [blame] | 980 | - markus@cvs.openbsd.org 2010/09/02 16:07:25 |
Damien Miller | 5773794 | 2010-09-10 11:16:37 +1000 | [diff] [blame] | 981 | [ssh-keygen.c] |
| 982 | permit -b 256, 384 or 521 as key size for ECDSA; ok djm@ |
Damien Miller | 390f153 | 2010-09-10 11:17:54 +1000 | [diff] [blame] | 983 | - markus@cvs.openbsd.org 2010/09/02 16:08:39 |
Damien Miller | 5929c52 | 2010-09-10 11:17:02 +1000 | [diff] [blame] | 984 | [ssh.c] |
| 985 | unbreak ControlPersist=yes for ControlMaster=yes; ok djm@ |
Damien Miller | 6e9f680 | 2010-09-10 11:17:38 +1000 | [diff] [blame] | 986 | - naddy@cvs.openbsd.org 2010/09/02 17:21:50 |
| 987 | [ssh-keygen.c] |
| 988 | Switch ECDSA default key size to 256 bits, which according to RFC5656 |
| 989 | should still be better than our current RSA-2048 default. |
| 990 | ok djm@, markus@ |
Damien Miller | 390f153 | 2010-09-10 11:17:54 +1000 | [diff] [blame] | 991 | - jmc@cvs.openbsd.org 2010/09/03 11:09:29 |
| 992 | [scp.1] |
| 993 | add an EXIT STATUS section for /usr/bin; |
Damien Miller | daa7b22 | 2010-09-10 11:19:33 +1000 | [diff] [blame] | 994 | - jmc@cvs.openbsd.org 2010/09/04 09:38:34 |
| 995 | [ssh-add.1 ssh.1] |
| 996 | two more EXIT STATUS sections; |
Damien Miller | 80ed82a | 2010-09-10 11:20:11 +1000 | [diff] [blame] | 997 | - naddy@cvs.openbsd.org 2010/09/06 17:10:19 |
| 998 | [sshd_config] |
| 999 | add ssh_host_ecdsa_key to /etc; from Mattieu Baptiste |
| 1000 | <mattieu.b@gmail.com> |
| 1001 | ok deraadt@ |
Damien Miller | bf0423e | 2010-09-10 11:20:38 +1000 | [diff] [blame] | 1002 | - djm@cvs.openbsd.org 2010/09/08 03:54:36 |
| 1003 | [authfile.c] |
| 1004 | typo |
Damien Miller | 3796ab4 | 2010-09-10 11:20:59 +1000 | [diff] [blame] | 1005 | - deraadt@cvs.openbsd.org 2010/09/08 04:13:31 |
| 1006 | [compress.c] |
| 1007 | work around name-space collisions some buggy compilers (looking at you |
| 1008 | gcc, at least in earlier versions, but this does not forgive your current |
| 1009 | transgressions) seen between zlib and openssl |
| 1010 | ok djm |
Damien Miller | 041ab7c | 2010-09-10 11:23:34 +1000 | [diff] [blame] | 1011 | - djm@cvs.openbsd.org 2010/09/09 10:45:45 |
| 1012 | [kex.c kex.h kexecdh.c key.c key.h monitor.c ssh-ecdsa.c] |
| 1013 | ECDH/ECDSA compliance fix: these methods vary the hash function they use |
| 1014 | (SHA256/384/512) depending on the length of the curve in use. The previous |
| 1015 | code incorrectly used SHA256 in all cases. |
| 1016 | |
| 1017 | This fix will cause authentication failure when using 384 or 521-bit curve |
| 1018 | keys if one peer hasn't been upgraded and the other has. (256-bit curve |
| 1019 | keys work ok). In particular you may need to specify HostkeyAlgorithms |
| 1020 | when connecting to a server that has not been upgraded from an upgraded |
| 1021 | client. |
| 1022 | |
| 1023 | ok naddy@ |
Damien Miller | 6af914a | 2010-09-10 11:39:26 +1000 | [diff] [blame] | 1024 | - (djm) [authfd.c authfile.c bufec.c buffer.h configure.ac kex.h kexecdh.c] |
| 1025 | [kexecdhc.c kexecdhs.c key.c key.h myproposal.h packet.c readconf.c] |
| 1026 | [ssh-agent.c ssh-ecdsa.c ssh-keygen.c ssh.c] Disable ECDH and ECDSA on |
| 1027 | platforms that don't have the requisite OpenSSL support. ok dtucker@ |
Darren Tucker | 8ccb739 | 2010-09-10 12:28:24 +1000 | [diff] [blame] | 1028 | - (dtucker) [kex.h key.c packet.h ssh-agent.c ssh.c] A few more ECC ifdefs |
| 1029 | for missing headers and compiler warnings. |
Darren Tucker | 50e3bab | 2010-09-10 10:30:25 +1000 | [diff] [blame] | 1030 | |
| 1031 | 20100831 |
Damien Miller | afdae61 | 2010-08-31 22:31:14 +1000 | [diff] [blame] | 1032 | - OpenBSD CVS Sync |
| 1033 | - jmc@cvs.openbsd.org 2010/08/08 19:36:30 |
| 1034 | [ssh-keysign.8 ssh.1 sshd.8] |
| 1035 | use the same template for all FILES sections; i.e. -compact/.Pp where we |
| 1036 | have multiple items, and .Pa for path names; |
Damien Miller | 9b87e79 | 2010-08-31 22:31:37 +1000 | [diff] [blame] | 1037 | - tedu@cvs.openbsd.org 2010/08/12 23:34:39 |
| 1038 | [ssh-add.c ssh-agent.c ssh-keygen.c ssh-keysign.c ssh.c sshd.c] |
| 1039 | OpenSSL_add_all_algorithms is the name of the function we have a man page |
| 1040 | for, so use that. ok djm |
Damien Miller | d96546f | 2010-08-31 22:32:12 +1000 | [diff] [blame] | 1041 | - djm@cvs.openbsd.org 2010/08/16 04:06:06 |
| 1042 | [ssh-add.c ssh-agent.c ssh-keygen.c ssh-keysign.c ssh.c sshd.c] |
| 1043 | backout previous temporarily; discussed with deraadt@ |
Damien Miller | da108ec | 2010-08-31 22:36:39 +1000 | [diff] [blame] | 1044 | - djm@cvs.openbsd.org 2010/08/31 09:58:37 |
| 1045 | [auth-options.c auth1.c auth2.c bufaux.c buffer.h kex.c key.c packet.c] |
| 1046 | [packet.h ssh-dss.c ssh-rsa.c] |
| 1047 | Add buffer_get_cstring() and related functions that verify that the |
| 1048 | string extracted from the buffer contains no embedded \0 characters* |
| 1049 | This prevents random (possibly malicious) crap from being appended to |
| 1050 | strings where it would not be noticed if the string is used with |
| 1051 | a string(3) function. |
| 1052 | |
| 1053 | Use the new API in a few sensitive places. |
| 1054 | |
| 1055 | * actually, we allow a single one at the end of the string for now because |
| 1056 | we don't know how many deployed implementations get this wrong, but don't |
| 1057 | count on this to remain indefinitely. |
Damien Miller | eb8b60e | 2010-08-31 22:41:14 +1000 | [diff] [blame] | 1058 | - djm@cvs.openbsd.org 2010/08/31 11:54:45 |
| 1059 | [PROTOCOL PROTOCOL.agent PROTOCOL.certkeys auth2-jpake.c authfd.c] |
| 1060 | [authfile.c buffer.h dns.c kex.c kex.h key.c key.h monitor.c] |
| 1061 | [monitor_wrap.c myproposal.h packet.c packet.h pathnames.h readconf.c] |
| 1062 | [ssh-add.1 ssh-add.c ssh-agent.1 ssh-agent.c ssh-keygen.1 ssh-keygen.c] |
| 1063 | [ssh-keyscan.1 ssh-keyscan.c ssh-keysign.8 ssh.1 ssh.c ssh2.h] |
| 1064 | [ssh_config.5 sshconnect.c sshconnect2.c sshd.8 sshd.c sshd_config.5] |
| 1065 | [uuencode.c uuencode.h bufec.c kexecdh.c kexecdhc.c kexecdhs.c ssh-ecdsa.c] |
| 1066 | Implement Elliptic Curve Cryptography modes for key exchange (ECDH) and |
| 1067 | host/user keys (ECDSA) as specified by RFC5656. ECDH and ECDSA offer |
| 1068 | better performance than plain DH and DSA at the same equivalent symmetric |
| 1069 | key length, as well as much shorter keys. |
| 1070 | |
| 1071 | Only the mandatory sections of RFC5656 are implemented, specifically the |
| 1072 | three REQUIRED curves nistp256, nistp384 and nistp521 and only ECDH and |
| 1073 | ECDSA. Point compression (optional in RFC5656 is NOT implemented). |
| 1074 | |
| 1075 | Certificate host and user keys using the new ECDSA key types are supported. |
| 1076 | |
| 1077 | Note that this code has not been tested for interoperability and may be |
| 1078 | subject to change. |
| 1079 | |
| 1080 | feedback and ok markus@ |
Damien Miller | b5a62d0 | 2010-08-31 22:47:15 +1000 | [diff] [blame] | 1081 | - (djm) [Makefile.in] Add new ECC files |
Damien Miller | c79ff07 | 2010-08-31 22:50:48 +1000 | [diff] [blame] | 1082 | - (djm) [bufec.c kexecdh.c kexecdhc.c kexecdhs.c ssh-ecdsa.c] include |
| 1083 | includes.h |
Damien Miller | afdae61 | 2010-08-31 22:31:14 +1000 | [diff] [blame] | 1084 | |
Darren Tucker | 6889abd | 2010-08-27 10:12:54 +1000 | [diff] [blame] | 1085 | 20100827 |
| 1086 | - (dtucker) [contrib/redhat/sshd.init] Bug #1810: initlog is deprecated, |
| 1087 | remove. Patch from martynas at venck us |
| 1088 | |
Damien Miller | a536202 | 2010-08-23 21:20:20 +1000 | [diff] [blame] | 1089 | 20100823 |
| 1090 | - (djm) Release OpenSSH-5.6p1 |
| 1091 | |
Darren Tucker | aa74f67 | 2010-08-16 13:15:23 +1000 | [diff] [blame] | 1092 | 20100816 |
| 1093 | - (dtucker) [configure.ac openbsd-compat/Makefile.in |
| 1094 | openbsd-compat/openbsd-compat.h openbsd-compat/strptime.c] Add strptime to |
| 1095 | the compat library which helps on platforms like old IRIX. Based on work |
| 1096 | by djm, tested by Tom Christensen. |
Damien Miller | 00d9ae2 | 2010-08-17 01:59:31 +1000 | [diff] [blame] | 1097 | - OpenBSD CVS Sync |
| 1098 | - djm@cvs.openbsd.org 2010/08/12 21:49:44 |
| 1099 | [ssh.c] |
| 1100 | close any extra file descriptors inherited from parent at start and |
| 1101 | reopen stdin/stdout to /dev/null when forking for ControlPersist. |
| 1102 | |
| 1103 | prevents tools that fork and run a captive ssh for communication from |
| 1104 | failing to exit when the ssh completes while they wait for these fds to |
| 1105 | close. The inherited fds may persist arbitrarily long if a background |
| 1106 | mux master has been started by ControlPersist. cvs and scp were effected |
| 1107 | by this. |
| 1108 | |
| 1109 | "please commit" markus@ |
Damien Miller | 07ad389 | 2010-08-17 07:04:28 +1000 | [diff] [blame] | 1110 | - (djm) [regress/README.regress] typo |
Darren Tucker | aa74f67 | 2010-08-16 13:15:23 +1000 | [diff] [blame] | 1111 | |
Tim Rice | 722b8d1 | 2010-08-12 09:43:13 -0700 | [diff] [blame] | 1112 | 20100812 |
| 1113 | - (tim) [regress/login-timeout.sh regress/reconfigure.sh regress/reexec.sh |
| 1114 | regress/test-exec.sh] Under certain conditions when testing with sudo |
| 1115 | tests would fail because the pidfile could not be read by a regular user. |
| 1116 | "cat: cannot open ...../regress/pidfile: Permission denied (error 13)" |
| 1117 | Make sure cat is run by $SUDO. no objection from me. djm@ |
Tim Rice | ad7d547 | 2010-08-12 10:33:01 -0700 | [diff] [blame] | 1118 | - (tim) [auth.c] add cast to quiet compiler. Change only affects SVR5 systems. |
Tim Rice | 722b8d1 | 2010-08-12 09:43:13 -0700 | [diff] [blame] | 1119 | |
Damien Miller | 7e569b8 | 2010-08-09 02:28:37 +1000 | [diff] [blame] | 1120 | 20100809 |
Damien Miller | 2c4b13a | 2010-08-10 12:47:40 +1000 | [diff] [blame] | 1121 | - (djm) bz#1561: don't bother setting IFF_UP on tun(4) device if it is |
| 1122 | already set. Makes FreeBSD user openable tunnels useful; patch from |
| 1123 | richard.burakowski+ossh AT mrburak.net, ok dtucker@ |
Darren Tucker | 02c4734 | 2010-08-10 13:36:09 +1000 | [diff] [blame] | 1124 | - (dtucker) bug #1530: strip trailing ":" from hostname in ssh-copy-id. |
| 1125 | based in part on a patch from Colin Watson, ok djm@ |
Damien Miller | 2c4b13a | 2010-08-10 12:47:40 +1000 | [diff] [blame] | 1126 | |
| 1127 | 20100809 |
Damien Miller | 7e569b8 | 2010-08-09 02:28:37 +1000 | [diff] [blame] | 1128 | - OpenBSD CVS Sync |
| 1129 | - djm@cvs.openbsd.org 2010/08/08 16:26:42 |
| 1130 | [version.h] |
| 1131 | crank to 5.6 |
Damien Miller | 792010b | 2010-08-09 02:32:05 +1000 | [diff] [blame] | 1132 | - (djm) [README contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 1133 | [contrib/suse/openssh.spec] Crank version numbers |
Damien Miller | 7e569b8 | 2010-08-09 02:28:37 +1000 | [diff] [blame] | 1134 | |
Damien Miller | 8e604ac | 2010-08-09 02:28:10 +1000 | [diff] [blame] | 1135 | 20100805 |
Damien Miller | 7fa9660 | 2010-08-05 13:03:13 +1000 | [diff] [blame] | 1136 | - OpenBSD CVS Sync |
| 1137 | - djm@cvs.openbsd.org 2010/08/04 05:37:01 |
| 1138 | [ssh.1 ssh_config.5 sshd.8] |
| 1139 | Remove mentions of weird "addr/port" alternate address format for IPv6 |
| 1140 | addresses combinations. It hasn't worked for ages and we have supported |
| 1141 | the more commen "[addr]:port" format for a long time. ok jmc@ markus@ |
Damien Miller | 1da6388 | 2010-08-05 13:03:51 +1000 | [diff] [blame] | 1142 | - djm@cvs.openbsd.org 2010/08/04 05:40:39 |
| 1143 | [PROTOCOL.certkeys ssh-keygen.c] |
| 1144 | tighten the rules for certificate encoding by requiring that options |
| 1145 | appear in lexical order and make our ssh-keygen comply. ok markus@ |
Damien Miller | c158331 | 2010-08-05 13:04:50 +1000 | [diff] [blame] | 1146 | - djm@cvs.openbsd.org 2010/08/04 05:42:47 |
| 1147 | [auth.c auth2-hostbased.c authfile.c authfile.h ssh-keysign.8] |
| 1148 | [ssh-keysign.c ssh.c] |
| 1149 | enable certificates for hostbased authentication, from Iain Morgan; |
| 1150 | "looks ok" markus@ |
Damien Miller | 5458c4d | 2010-08-05 13:05:15 +1000 | [diff] [blame] | 1151 | - djm@cvs.openbsd.org 2010/08/04 05:49:22 |
| 1152 | [authfile.c] |
| 1153 | commited the wrong version of the hostbased certificate diff; this |
| 1154 | version replaces some strlc{py,at} verbosity with xasprintf() at |
| 1155 | the request of markus@ |
Damien Miller | 757f34e | 2010-08-05 13:05:31 +1000 | [diff] [blame] | 1156 | - djm@cvs.openbsd.org 2010/08/04 06:07:11 |
| 1157 | [ssh-keygen.1 ssh-keygen.c] |
| 1158 | Support CA keys in PKCS#11 tokens; feedback and ok markus@ |
Damien Miller | b89e6b7 | 2010-08-05 13:06:20 +1000 | [diff] [blame] | 1159 | - djm@cvs.openbsd.org 2010/08/04 06:08:40 |
| 1160 | [ssh-keysign.c] |
| 1161 | clean for -Wuninitialized (Id sync only; portable had this change) |
Damien Miller | 7d45718 | 2010-08-05 23:09:48 +1000 | [diff] [blame] | 1162 | - djm@cvs.openbsd.org 2010/08/05 13:08:42 |
| 1163 | [channels.c] |
| 1164 | Fix a trio of bugs in the local/remote window calculation for datagram |
| 1165 | data channels (i.e. TunnelForward): |
| 1166 | |
| 1167 | Calculate local_consumed correctly in channel_handle_wfd() by measuring |
| 1168 | the delta to buffer_len(c->output) from when we start to when we finish. |
| 1169 | The proximal problem here is that the output_filter we use in portable |
| 1170 | modified the length of the dequeued datagram (to futz with the headers |
| 1171 | for !OpenBSD). |
| 1172 | |
| 1173 | In channel_output_poll(), don't enqueue datagrams that won't fit in the |
| 1174 | peer's advertised packet size (highly unlikely to ever occur) or which |
| 1175 | won't fit in the peer's remaining window (more likely). |
| 1176 | |
| 1177 | In channel_input_data(), account for the 4-byte string header in |
| 1178 | datagram packets that we accept from the peer and enqueue in c->output. |
| 1179 | |
| 1180 | report, analysis and testing 2/3 cases from wierbows AT us.ibm.com; |
| 1181 | "looks good" markus@ |
Damien Miller | 7fa9660 | 2010-08-05 13:03:13 +1000 | [diff] [blame] | 1182 | |
Damien Miller | 8e604ac | 2010-08-09 02:28:10 +1000 | [diff] [blame] | 1183 | 20100803 |
Darren Tucker | 8b7a055 | 2010-08-03 15:50:16 +1000 | [diff] [blame] | 1184 | - (dtucker) [monitor.c] Bug #1795: Initialize the values to be returned from |
| 1185 | PAM to sane values in case the PAM method doesn't write to them. Spotted by |
| 1186 | Bitman Zhou, ok djm@. |
Damien Miller | 844cccf | 2010-08-03 16:03:29 +1000 | [diff] [blame] | 1187 | - OpenBSD CVS Sync |
| 1188 | - djm@cvs.openbsd.org 2010/07/16 04:45:30 |
| 1189 | [ssh-keygen.c] |
| 1190 | avoid bogus compiler warning |
Damien Miller | 4e8285e | 2010-08-03 16:04:03 +1000 | [diff] [blame] | 1191 | - djm@cvs.openbsd.org 2010/07/16 14:07:35 |
| 1192 | [ssh-rsa.c] |
| 1193 | more timing paranoia - compare all parts of the expected decrypted |
| 1194 | data before returning. AFAIK not exploitable in the SSH protocol. |
| 1195 | "groovy" deraadt@ |
Damien Miller | c4bb91c | 2010-08-03 16:04:22 +1000 | [diff] [blame] | 1196 | - djm@cvs.openbsd.org 2010/07/19 03:16:33 |
| 1197 | [sftp-client.c] |
| 1198 | bz#1797: fix swapped args in upload_dir_internal(), breaking recursive |
| 1199 | upload depth checks and causing verbose printing of transfers to always |
| 1200 | be turned on; patch from imorgan AT nas.nasa.gov |
Damien Miller | e11e1ea | 2010-08-03 16:04:46 +1000 | [diff] [blame] | 1201 | - djm@cvs.openbsd.org 2010/07/19 09:15:12 |
| 1202 | [clientloop.c readconf.c readconf.h ssh.c ssh_config.5] |
| 1203 | add a "ControlPersist" option that automatically starts a background |
| 1204 | ssh(1) multiplex master when connecting. This connection can stay alive |
| 1205 | indefinitely, or can be set to automatically close after a user-specified |
| 1206 | duration of inactivity. bz#1330 - patch by dwmw2 AT infradead.org, but |
| 1207 | further hacked on by wmertens AT cisco.com, apb AT cequrux.com, |
| 1208 | martin-mindrot-bugzilla AT earth.li and myself; "looks ok" markus@ |
Damien Miller | 8c1eb11 | 2010-08-03 16:05:05 +1000 | [diff] [blame] | 1209 | - djm@cvs.openbsd.org 2010/07/21 02:10:58 |
| 1210 | [misc.c] |
| 1211 | sync timingsafe_bcmp() with the one dempsky@ committed to sys/lib/libkern |
Damien Miller | 081f3c7 | 2010-08-03 16:05:25 +1000 | [diff] [blame] | 1212 | - dtucker@cvs.openbsd.org 2010/07/23 08:49:25 |
| 1213 | [ssh.1] |
| 1214 | Ciphers is documented in ssh_config(5) these days |
Darren Tucker | 8b7a055 | 2010-08-03 15:50:16 +1000 | [diff] [blame] | 1215 | |
| 1216 | 20100819 |
Darren Tucker | 12b29db | 2010-07-19 21:24:13 +1000 | [diff] [blame] | 1217 | - (dtucker) [contrib/ssh-copy-ud.1] Bug #1786: update ssh-copy-id.1 with more |
| 1218 | details about its behaviour WRT existing directories. Patch from |
| 1219 | asguthrie at gmail com, ok djm. |
| 1220 | |
Damien Miller | 9308fc7 | 2010-07-16 13:56:01 +1000 | [diff] [blame] | 1221 | 20100716 |
| 1222 | - (djm) OpenBSD CVS Sync |
| 1223 | - djm@cvs.openbsd.org 2010/07/02 04:32:44 |
| 1224 | [misc.c] |
| 1225 | unbreak strdelim() skipping past quoted strings, e.g. |
| 1226 | AllowUsers "blah blah" blah |
| 1227 | was broken; report and fix in bz#1757 from bitman.zhou AT centrify.com |
| 1228 | ok dtucker; |
Damien Miller | 1f25ab4 | 2010-07-16 13:56:23 +1000 | [diff] [blame] | 1229 | - djm@cvs.openbsd.org 2010/07/12 22:38:52 |
| 1230 | [ssh.c] |
| 1231 | Make ExitOnForwardFailure work with fork-after-authentication ("ssh -f") |
| 1232 | for protocol 2. ok markus@ |
Damien Miller | d0244d4 | 2010-07-16 13:56:43 +1000 | [diff] [blame] | 1233 | - djm@cvs.openbsd.org 2010/07/12 22:41:13 |
| 1234 | [ssh.c ssh_config.5] |
| 1235 | expand %h to the hostname in ssh_config Hostname options. While this |
| 1236 | sounds useless, it is actually handy for working with unqualified |
| 1237 | hostnames: |
| 1238 | |
| 1239 | Host *.* |
| 1240 | Hostname %h |
| 1241 | Host * |
| 1242 | Hostname %h.example.org |
| 1243 | |
| 1244 | "I like it" markus@ |
Damien Miller | 8a0268f | 2010-07-16 13:57:51 +1000 | [diff] [blame] | 1245 | - djm@cvs.openbsd.org 2010/07/13 11:52:06 |
| 1246 | [auth-rsa.c channels.c jpake.c key.c misc.c misc.h monitor.c] |
| 1247 | [packet.c ssh-rsa.c] |
| 1248 | implement a timing_safe_cmp() function to compare memory without leaking |
| 1249 | timing information by short-circuiting like memcmp() and use it for |
| 1250 | some of the more sensitive comparisons (though nothing high-value was |
| 1251 | readily attackable anyway); "looks ok" markus@ |
Damien Miller | ea1651c | 2010-07-16 13:58:37 +1000 | [diff] [blame] | 1252 | - djm@cvs.openbsd.org 2010/07/13 23:13:16 |
| 1253 | [auth-rsa.c channels.c jpake.c key.c misc.c misc.h monitor.c packet.c] |
| 1254 | [ssh-rsa.c] |
| 1255 | s/timing_safe_cmp/timingsafe_bcmp/g |
Damien Miller | bcfbc48 | 2010-07-16 13:59:11 +1000 | [diff] [blame] | 1256 | - jmc@cvs.openbsd.org 2010/07/14 17:06:58 |
| 1257 | [ssh.1] |
| 1258 | finally ssh synopsis looks nice again! this commit just removes a ton of |
| 1259 | hacks we had in place to make it work with old groff; |
Damien Miller | bad5e03 | 2010-07-16 13:59:59 +1000 | [diff] [blame] | 1260 | - schwarze@cvs.openbsd.org 2010/07/15 21:20:38 |
| 1261 | [ssh-keygen.1] |
| 1262 | repair incorrect block nesting, which screwed up indentation; |
| 1263 | problem reported and fix OK by jmc@ |
Damien Miller | 9308fc7 | 2010-07-16 13:56:01 +1000 | [diff] [blame] | 1264 | |
Tim Rice | cfbdc28 | 2010-07-14 13:42:28 -0700 | [diff] [blame] | 1265 | 20100714 |
| 1266 | - (tim) [contrib/redhat/openssh.spec] Bug 1796: Test for skip_x11_askpass |
| 1267 | (line 77) should have been for no_x11_askpass. |
| 1268 | |
Damien Miller | cede1db | 2010-07-02 13:33:48 +1000 | [diff] [blame] | 1269 | 20100702 |
| 1270 | - (djm) OpenBSD CVS Sync |
| 1271 | - jmc@cvs.openbsd.org 2010/06/26 00:57:07 |
| 1272 | [ssh_config.5] |
| 1273 | tweak previous; |
Damien Miller | b96c441 | 2010-07-02 13:34:24 +1000 | [diff] [blame] | 1274 | - djm@cvs.openbsd.org 2010/06/26 23:04:04 |
| 1275 | [ssh.c] |
| 1276 | oops, forgot to #include <canohost.h>; spotted and patch from chl@ |
Damien Miller | 44b2504 | 2010-07-02 13:35:01 +1000 | [diff] [blame] | 1277 | - djm@cvs.openbsd.org 2010/06/29 23:15:30 |
| 1278 | [ssh-keygen.1 ssh-keygen.c] |
| 1279 | allow import (-i) and export (-e) of PEM and PKCS#8 encoded keys; |
| 1280 | bz#1749; ok markus@ |
Damien Miller | 6018a36 | 2010-07-02 13:35:19 +1000 | [diff] [blame] | 1281 | - djm@cvs.openbsd.org 2010/06/29 23:16:46 |
| 1282 | [auth2-pubkey.c sshd_config.5] |
| 1283 | allow key options (command="..." and friends) in AuthorizedPrincipals; |
| 1284 | ok markus@ |
Damien Miller | ea72728 | 2010-07-02 13:35:34 +1000 | [diff] [blame] | 1285 | - jmc@cvs.openbsd.org 2010/06/30 07:24:25 |
| 1286 | [ssh-keygen.1] |
| 1287 | tweak previous; |
Damien Miller | 6022f58 | 2010-07-02 13:37:01 +1000 | [diff] [blame] | 1288 | - jmc@cvs.openbsd.org 2010/06/30 07:26:03 |
| 1289 | [ssh-keygen.c] |
| 1290 | sort usage(); |
Damien Miller | d59dab8 | 2010-07-02 13:37:17 +1000 | [diff] [blame] | 1291 | - jmc@cvs.openbsd.org 2010/06/30 07:28:34 |
| 1292 | [sshd_config.5] |
| 1293 | tweak previous; |
Damien Miller | 0979b40 | 2010-07-02 13:37:33 +1000 | [diff] [blame] | 1294 | - millert@cvs.openbsd.org 2010/07/01 13:06:59 |
| 1295 | [scp.c] |
| 1296 | Fix a longstanding problem where if you suspend scp at the |
| 1297 | password/passphrase prompt the terminal mode is not restored. |
| 1298 | OK djm@ |
Damien Miller | 527ded7 | 2010-07-02 13:40:16 +1000 | [diff] [blame] | 1299 | - phessler@cvs.openbsd.org 2010/06/27 19:19:56 |
| 1300 | [regress/Makefile] |
| 1301 | fix how we run the tests so we can successfully use SUDO='sudo -E' |
| 1302 | in our env |
Damien Miller | ab139cd | 2010-07-02 13:42:18 +1000 | [diff] [blame] | 1303 | - djm@cvs.openbsd.org 2010/06/29 23:59:54 |
| 1304 | [cert-userkey.sh] |
| 1305 | regress tests for key options in AuthorizedPrincipals |
Damien Miller | cede1db | 2010-07-02 13:33:48 +1000 | [diff] [blame] | 1306 | |
Tim Rice | 3fd307d | 2010-06-26 16:45:15 -0700 | [diff] [blame] | 1307 | 20100627 |
| 1308 | - (tim) [openbsd-compat/port-uw.c] Reorder includes. auth-options.h now needs |
| 1309 | key.h. |
| 1310 | |
Damien Miller | 2e77446 | 2010-06-26 09:30:47 +1000 | [diff] [blame] | 1311 | 20100626 |
| 1312 | - (djm) OpenBSD CVS Sync |
| 1313 | - djm@cvs.openbsd.org 2010/05/21 05:00:36 |
| 1314 | [misc.c] |
| 1315 | colon() returns char*, so s/return (0)/return NULL/ |
Damien Miller | 4fe686d | 2010-06-26 09:36:10 +1000 | [diff] [blame] | 1316 | - markus@cvs.openbsd.org 2010/06/08 21:32:19 |
| 1317 | [ssh-pkcs11.c] |
| 1318 | check length of value returned C_GetAttributValue for != 0 |
| 1319 | from mdrtbugzilla@codefive.co.uk; bugzilla #1773; ok dtucker@ |
Damien Miller | c094d1e | 2010-06-26 09:36:34 +1000 | [diff] [blame] | 1320 | - djm@cvs.openbsd.org 2010/06/17 07:07:30 |
| 1321 | [mux.c] |
| 1322 | Correct sizing of object to be allocated by calloc(), replacing |
| 1323 | sizeof(state) with sizeof(*state). This worked by accident since |
| 1324 | the struct contained a single int at present, but could have broken |
| 1325 | in the future. patch from hyc AT symas.com |
Damien Miller | 99ac4e9 | 2010-06-26 09:36:58 +1000 | [diff] [blame] | 1326 | - djm@cvs.openbsd.org 2010/06/18 00:58:39 |
| 1327 | [sftp.c] |
| 1328 | unbreak ls in working directories that contains globbing characters in |
| 1329 | their pathnames. bz#1655 reported by vgiffin AT apple.com |
Damien Miller | 7aa46ec | 2010-06-26 09:37:57 +1000 | [diff] [blame] | 1330 | - djm@cvs.openbsd.org 2010/06/18 03:16:03 |
| 1331 | [session.c] |
| 1332 | Missing check for chroot_director == "none" (we already checked against |
| 1333 | NULL); bz#1564 from Jan.Pechanec AT Sun.COM |
Damien Miller | 4956631 | 2010-06-26 09:38:23 +1000 | [diff] [blame] | 1334 | - djm@cvs.openbsd.org 2010/06/18 04:43:08 |
| 1335 | [sftp-client.c] |
| 1336 | fix memory leak in do_realpath() error path; bz#1771, patch from |
| 1337 | anicka AT suse.cz |
Damien Miller | ab6de35 | 2010-06-26 09:38:45 +1000 | [diff] [blame] | 1338 | - djm@cvs.openbsd.org 2010/06/22 04:22:59 |
| 1339 | [servconf.c sshd_config.5] |
| 1340 | expose some more sshd_config options inside Match blocks: |
| 1341 | AuthorizedKeysFile AuthorizedPrincipalsFile |
| 1342 | HostbasedUsesNameFromPacketOnly PermitTunnel |
| 1343 | bz#1764; feedback from imorgan AT nas.nasa.gov; ok dtucker@ |
Damien Miller | ba3420a | 2010-06-26 09:39:07 +1000 | [diff] [blame] | 1344 | - djm@cvs.openbsd.org 2010/06/22 04:32:06 |
| 1345 | [ssh-keygen.c] |
| 1346 | standardise error messages when attempting to open private key |
| 1347 | files to include "progname: filename: error reason" |
| 1348 | bz#1783; ok dtucker@ |
Damien Miller | 48147d6 | 2010-06-26 09:39:25 +1000 | [diff] [blame] | 1349 | - djm@cvs.openbsd.org 2010/06/22 04:49:47 |
| 1350 | [auth.c] |
| 1351 | queue auth debug messages for bad ownership or permissions on the user's |
| 1352 | keyfiles. These messages will be sent after the user has successfully |
| 1353 | authenticated (where our client will display them with LogLevel=debug). |
Damien Miller | 0e76c5e | 2010-06-26 09:39:59 +1000 | [diff] [blame] | 1354 | bz#1554; ok dtucker@ |
| 1355 | - djm@cvs.openbsd.org 2010/06/22 04:54:30 |
| 1356 | [ssh-keyscan.c] |
| 1357 | replace verbose and overflow-prone Linebuf code with read_keyfile_line() |
| 1358 | based on patch from joachim AT joachimschipper.nl; bz#1565; ok dtucker@ |
Damien Miller | 1b2b61e | 2010-06-26 09:47:43 +1000 | [diff] [blame] | 1359 | - djm@cvs.openbsd.org 2010/06/22 04:59:12 |
| 1360 | [session.c] |
| 1361 | include the user name on "subsystem request for ..." log messages; |
| 1362 | bz#1571; ok dtucker@ |
Damien Miller | d834d35 | 2010-06-26 09:48:02 +1000 | [diff] [blame] | 1363 | - djm@cvs.openbsd.org 2010/06/23 02:59:02 |
| 1364 | [ssh-keygen.c] |
| 1365 | fix printing of extensions in v01 certificates that I broke in r1.190 |
Damien Miller | 232cfb1 | 2010-06-26 09:50:30 +1000 | [diff] [blame] | 1366 | - djm@cvs.openbsd.org 2010/06/25 07:14:46 |
| 1367 | [channels.c mux.c readconf.c readconf.h ssh.h] |
| 1368 | bz#1327: remove hardcoded limit of 100 permitopen clauses and port |
| 1369 | forwards per direction; ok markus@ stevesk@ |
Damien Miller | 8853ca5 | 2010-06-26 10:00:14 +1000 | [diff] [blame] | 1370 | - djm@cvs.openbsd.org 2010/06/25 07:20:04 |
| 1371 | [channels.c session.c] |
| 1372 | bz#1750: fix requirement for /dev/null inside ChrootDirectory for |
| 1373 | internal-sftp accidentally introduced in r1.253 by removing the code |
| 1374 | that opens and dup /dev/null to stderr and modifying the channels code |
| 1375 | to read stderr but discard it instead; ok markus@ |
Damien Miller | bda3eca | 2010-06-26 10:01:33 +1000 | [diff] [blame] | 1376 | - djm@cvs.openbsd.org 2010/06/25 08:46:17 |
| 1377 | [auth1.c auth2-none.c] |
| 1378 | skip the initial check for access with an empty password when |
| 1379 | PermitEmptyPasswords=no; bz#1638; ok markus@ |
Damien Miller | 383ffe6 | 2010-06-26 10:02:03 +1000 | [diff] [blame] | 1380 | - djm@cvs.openbsd.org 2010/06/25 23:10:30 |
| 1381 | [ssh.c] |
| 1382 | log the hostname and address that we connected to at LogLevel=verbose |
| 1383 | after authentication is successful to mitigate "phishing" attacks by |
| 1384 | servers with trusted keys that accept authentication silently and |
| 1385 | automatically before presenting fake password/passphrase prompts; |
| 1386 | "nice!" markus@ |
Damien Miller | 1ab6a51 | 2010-06-26 10:02:24 +1000 | [diff] [blame] | 1387 | - djm@cvs.openbsd.org 2010/06/25 23:10:30 |
| 1388 | [ssh.c] |
| 1389 | log the hostname and address that we connected to at LogLevel=verbose |
| 1390 | after authentication is successful to mitigate "phishing" attacks by |
| 1391 | servers with trusted keys that accept authentication silently and |
| 1392 | automatically before presenting fake password/passphrase prompts; |
| 1393 | "nice!" markus@ |
Damien Miller | 2e77446 | 2010-06-26 09:30:47 +1000 | [diff] [blame] | 1394 | |
Damien Miller | d82a260 | 2010-06-22 15:02:39 +1000 | [diff] [blame] | 1395 | 20100622 |
| 1396 | - (djm) [loginrec.c] crank LINFO_NAMESIZE (username length) to 512 |
| 1397 | bz#1579; ok dtucker |
| 1398 | |
Damien Miller | ea90979 | 2010-06-18 11:09:24 +1000 | [diff] [blame] | 1399 | 20100618 |
| 1400 | - (djm) [contrib/ssh-copy-id] Update key file explicitly under ~ |
| 1401 | rather than assuming that $CWD == $HOME. bz#1500, patch from |
| 1402 | timothy AT gelter.com |
| 1403 | |
Tim Rice | b9ae4ec | 2010-06-17 11:11:44 -0700 | [diff] [blame] | 1404 | 20100617 |
| 1405 | - (tim) [contrib/cygwin/README] Remove a reference to the obsolete |
| 1406 | minires-devel package, and to add the reference to the libedit-devel |
| 1407 | package since CYgwin now provides libedit. Patch from Corinna Vinschen. |
| 1408 | |
Damien Miller | 3bcce80 | 2010-05-21 14:48:16 +1000 | [diff] [blame] | 1409 | 20100521 |
| 1410 | - (djm) OpenBSD CVS Sync |
| 1411 | - djm@cvs.openbsd.org 2010/05/07 11:31:26 |
| 1412 | [regress/Makefile regress/cert-userkey.sh] |
| 1413 | regress tests for AuthorizedPrincipalsFile and "principals=" key option. |
| 1414 | feedback and ok markus@ |
Damien Miller | 3b90382 | 2010-05-21 14:56:25 +1000 | [diff] [blame] | 1415 | - djm@cvs.openbsd.org 2010/05/11 02:58:04 |
| 1416 | [auth-rsa.c] |
| 1417 | don't accept certificates marked as "cert-authority" here; ok markus@ |
Damien Miller | c6afb5f | 2010-05-21 14:56:47 +1000 | [diff] [blame] | 1418 | - djm@cvs.openbsd.org 2010/05/14 00:47:22 |
| 1419 | [ssh-add.c] |
| 1420 | check that the certificate matches the corresponding private key before |
| 1421 | grafting it on |
Damien Miller | d530f5f | 2010-05-21 14:57:10 +1000 | [diff] [blame] | 1422 | - djm@cvs.openbsd.org 2010/05/14 23:29:23 |
| 1423 | [channels.c channels.h mux.c ssh.c] |
| 1424 | Pause the mux channel while waiting for reply from aynch callbacks. |
| 1425 | Prevents misordering of replies if new requests arrive while waiting. |
| 1426 | |
| 1427 | Extend channel open confirm callback to allow signalling failure |
| 1428 | conditions as well as success. Use this to 1) fix a memory leak, 2) |
| 1429 | start using the above pause mechanism and 3) delay sending a success/ |
| 1430 | failure message on mux slave session open until we receive a reply from |
| 1431 | the server. |
| 1432 | |
| 1433 | motivated by and with feedback from markus@ |
Damien Miller | 388f6fc | 2010-05-21 14:57:35 +1000 | [diff] [blame] | 1434 | - markus@cvs.openbsd.org 2010/05/16 12:55:51 |
| 1435 | [PROTOCOL.mux clientloop.h mux.c readconf.c readconf.h ssh.1 ssh.c] |
| 1436 | mux support for remote forwarding with dynamic port allocation, |
| 1437 | use with |
| 1438 | LPORT=`ssh -S muxsocket -R0:localhost:25 -O forward somehost` |
| 1439 | feedback and ok djm@ |
Damien Miller | 8439955 | 2010-05-21 14:58:12 +1000 | [diff] [blame] | 1440 | - djm@cvs.openbsd.org 2010/05/20 11:25:26 |
| 1441 | [auth2-pubkey.c] |
| 1442 | fix logspam when key options (from="..." especially) deny non-matching |
| 1443 | keys; reported by henning@ also bz#1765; ok markus@ dtucker@ |
Damien Miller | d0e4a8e | 2010-05-21 14:58:32 +1000 | [diff] [blame] | 1444 | - djm@cvs.openbsd.org 2010/05/20 23:46:02 |
| 1445 | [PROTOCOL.certkeys auth-options.c ssh-keygen.c] |
| 1446 | Move the permit-* options to the non-critical "extensions" field for v01 |
| 1447 | certificates. The logic is that if another implementation fails to |
| 1448 | implement them then the connection just loses features rather than fails |
| 1449 | outright. |
| 1450 | |
| 1451 | ok markus@ |
Damien Miller | 3bcce80 | 2010-05-21 14:48:16 +1000 | [diff] [blame] | 1452 | |
Darren Tucker | 5b6d0d0 | 2010-05-12 16:51:38 +1000 | [diff] [blame] | 1453 | 20100511 |
| 1454 | - (dtucker) [Makefile.in] Bug #1770: Link libopenbsd-compat twice to solve |
| 1455 | circular dependency problem on old or odd platforms. From Tom Lane, ok |
| 1456 | djm@. |
Damien Miller | 4b1ec83 | 2010-05-12 17:49:59 +1000 | [diff] [blame] | 1457 | - (djm) [openbsd-compat/openssl-compat.h] Fix build breakage on older |
| 1458 | libcrypto by defining OPENSSL_[DR]SA_MAX_MODULUS_BITS if they aren't |
| 1459 | already. ok dtucker@ |
Darren Tucker | 5b6d0d0 | 2010-05-12 16:51:38 +1000 | [diff] [blame] | 1460 | |
Damien Miller | 50af79b | 2010-05-10 11:52:00 +1000 | [diff] [blame] | 1461 | 20100510 |
| 1462 | - OpenBSD CVS Sync |
| 1463 | - djm@cvs.openbsd.org 2010/04/23 01:47:41 |
| 1464 | [ssh-keygen.c] |
| 1465 | bz#1740: display a more helpful error message when $HOME is |
| 1466 | inaccessible while trying to create .ssh directory. Based on patch |
| 1467 | from jchadima AT redhat.com; ok dtucker@ |
Damien Miller | 85c50d7 | 2010-05-10 11:53:02 +1000 | [diff] [blame] | 1468 | - djm@cvs.openbsd.org 2010/04/23 22:27:38 |
| 1469 | [mux.c] |
| 1470 | set "detach_close" flag when registering channel cleanup callbacks. |
| 1471 | This causes the channel to close normally when its fds close and |
| 1472 | hangs when terminating a mux slave using ~. bz#1758; ok markus@ |
Damien Miller | 22a2988 | 2010-05-10 11:53:54 +1000 | [diff] [blame] | 1473 | - djm@cvs.openbsd.org 2010/04/23 22:42:05 |
| 1474 | [session.c] |
| 1475 | set stderr to /dev/null for subsystems rather than just closing it. |
| 1476 | avoids hangs if a subsystem or shell initialisation writes to stderr. |
| 1477 | bz#1750; ok markus@ |
Damien Miller | bebbb7e | 2010-05-10 11:54:38 +1000 | [diff] [blame] | 1478 | - djm@cvs.openbsd.org 2010/04/23 22:48:31 |
| 1479 | [ssh-keygen.c] |
| 1480 | refuse to generate keys longer than OPENSSL_[RD]SA_MAX_MODULUS_BITS, |
| 1481 | since we would refuse to use them anyway. bz#1516; ok dtucker@ |
Damien Miller | 79442c0 | 2010-05-10 11:55:38 +1000 | [diff] [blame] | 1482 | - djm@cvs.openbsd.org 2010/04/26 22:28:24 |
| 1483 | [sshconnect2.c] |
| 1484 | bz#1502: authctxt.success is declared as an int, but passed by |
| 1485 | reference to function that accepts sig_atomic_t*. Convert it to |
| 1486 | the latter; ok markus@ dtucker@ |
Damien Miller | 2725c21 | 2010-05-10 11:56:14 +1000 | [diff] [blame] | 1487 | - djm@cvs.openbsd.org 2010/05/01 02:50:50 |
| 1488 | [PROTOCOL.certkeys] |
| 1489 | typo; jmeltzer@ |
Damien Miller | 099fc16 | 2010-05-10 11:56:50 +1000 | [diff] [blame] | 1490 | - dtucker@cvs.openbsd.org 2010/05/05 04:22:09 |
| 1491 | [sftp.c] |
| 1492 | restore mput and mget which got lost in the tab-completion changes. |
| 1493 | found by Kenneth Whitaker, ok djm@ |
Damien Miller | 30da344 | 2010-05-10 11:58:03 +1000 | [diff] [blame] | 1494 | - djm@cvs.openbsd.org 2010/05/07 11:30:30 |
| 1495 | [auth-options.c auth-options.h auth.c auth.h auth2-pubkey.c] |
| 1496 | [key.c servconf.c servconf.h sshd.8 sshd_config.5] |
| 1497 | add some optional indirection to matching of principal names listed |
| 1498 | in certificates. Currently, a certificate must include the a user's name |
| 1499 | to be accepted for authentication. This change adds the ability to |
| 1500 | specify a list of certificate principal names that are acceptable. |
| 1501 | |
| 1502 | When authenticating using a CA trusted through ~/.ssh/authorized_keys, |
| 1503 | this adds a new principals="name1[,name2,...]" key option. |
| 1504 | |
| 1505 | For CAs listed through sshd_config's TrustedCAKeys option, a new config |
| 1506 | option "AuthorizedPrincipalsFile" specifies a per-user file containing |
| 1507 | the list of acceptable names. |
| 1508 | |
| 1509 | If either option is absent, the current behaviour of requiring the |
| 1510 | username to appear in principals continues to apply. |
| 1511 | |
| 1512 | These options are useful for role accounts, disjoint account namespaces |
| 1513 | and "user@realm"-style naming policies in certificates. |
| 1514 | |
| 1515 | feedback and ok markus@ |
Damien Miller | 81d3fc5 | 2010-05-10 11:58:45 +1000 | [diff] [blame] | 1516 | - jmc@cvs.openbsd.org 2010/05/07 12:49:17 |
| 1517 | [sshd_config.5] |
| 1518 | tweak previous; |
Damien Miller | 50af79b | 2010-05-10 11:52:00 +1000 | [diff] [blame] | 1519 | |
Darren Tucker | 9f8703b | 2010-04-23 11:12:06 +1000 | [diff] [blame] | 1520 | 20100423 |
| 1521 | - (dtucker) [configure.ac] Bug #1756: Check for the existence of a lib64 dir |
| 1522 | in the openssl install directory (some newer openssl versions do this on at |
| 1523 | least some amd64 platforms). |
| 1524 | |
Damien Miller | c4eddee | 2010-04-18 08:07:43 +1000 | [diff] [blame] | 1525 | 20100418 |
| 1526 | - OpenBSD CVS Sync |
| 1527 | - jmc@cvs.openbsd.org 2010/04/16 06:45:01 |
| 1528 | [ssh_config.5] |
| 1529 | tweak previous; ok djm |
Damien Miller | 1f18142 | 2010-04-18 08:08:03 +1000 | [diff] [blame] | 1530 | - jmc@cvs.openbsd.org 2010/04/16 06:47:04 |
| 1531 | [ssh-keygen.1 ssh-keygen.c] |
| 1532 | tweak previous; ok djm |
Damien Miller | c617aa9 | 2010-04-18 08:08:20 +1000 | [diff] [blame] | 1533 | - djm@cvs.openbsd.org 2010/04/16 21:14:27 |
| 1534 | [sshconnect.c] |
| 1535 | oops, %r => remote username, not %u |
Damien Miller | 53f4bb6 | 2010-04-18 08:15:14 +1000 | [diff] [blame] | 1536 | - djm@cvs.openbsd.org 2010/04/16 01:58:45 |
| 1537 | [regress/cert-hostkey.sh regress/cert-userkey.sh] |
| 1538 | regression tests for v01 certificate format |
| 1539 | includes interop tests for v00 certs |
Darren Tucker | e25a9bd | 2010-04-18 13:35:00 +1000 | [diff] [blame] | 1540 | - (dtucker) [contrib/aix/buildbff.sh] Fix creation of ssh_prng_cmds.default |
| 1541 | file. |
Damien Miller | c4eddee | 2010-04-18 08:07:43 +1000 | [diff] [blame] | 1542 | |
Damien Miller | a45f1c0 | 2010-04-16 15:51:34 +1000 | [diff] [blame] | 1543 | 20100416 |
| 1544 | - (djm) Release openssh-5.5p1 |
Damien Miller | d6fc306 | 2010-04-16 15:51:45 +1000 | [diff] [blame] | 1545 | - OpenBSD CVS Sync |
| 1546 | - djm@cvs.openbsd.org 2010/03/26 03:13:17 |
| 1547 | [bufaux.c] |
| 1548 | allow buffer_get_int_ret/buffer_get_int64_ret to take a NULL pointer |
| 1549 | argument to allow skipping past values in a buffer |
Damien Miller | 67f30d7 | 2010-04-16 15:52:03 +1000 | [diff] [blame] | 1550 | - jmc@cvs.openbsd.org 2010/03/26 06:54:36 |
| 1551 | [ssh.1] |
| 1552 | tweak previous; |
Damien Miller | 544378d | 2010-04-16 15:52:24 +1000 | [diff] [blame] | 1553 | - jmc@cvs.openbsd.org 2010/03/27 14:26:55 |
| 1554 | [ssh_config.5] |
| 1555 | tweak previous; ok dtucker |
Damien Miller | deb5a14 | 2010-04-16 15:52:43 +1000 | [diff] [blame] | 1556 | - djm@cvs.openbsd.org 2010/04/10 00:00:16 |
| 1557 | [ssh.c] |
| 1558 | bz#1746 - suppress spurious tty warning when using -O and stdin |
| 1559 | is not a tty; ok dtucker@ markus@ |
Damien Miller | 6728399 | 2010-04-16 15:53:02 +1000 | [diff] [blame] | 1560 | - djm@cvs.openbsd.org 2010/04/10 00:04:30 |
| 1561 | [sshconnect.c] |
| 1562 | fix terminology: we didn't find a certificate in known_hosts, we found |
| 1563 | a CA key |
Damien Miller | 22c97f1 | 2010-04-16 15:53:23 +1000 | [diff] [blame] | 1564 | - djm@cvs.openbsd.org 2010/04/10 02:08:44 |
| 1565 | [clientloop.c] |
| 1566 | bz#1698: kill channel when pty allocation requests fail. Fixed |
| 1567 | stuck client if the server refuses pty allocation. |
| 1568 | ok dtucker@ "think so" markus@ |
Damien Miller | 8868065 | 2010-04-16 15:53:43 +1000 | [diff] [blame] | 1569 | - djm@cvs.openbsd.org 2010/04/10 02:10:56 |
| 1570 | [sshconnect2.c] |
| 1571 | show the key type that we are offering in debug(), helps distinguish |
| 1572 | between certs and plain keys as the path to the private key is usually |
| 1573 | the same. |
Damien Miller | 601a23c | 2010-04-16 15:54:01 +1000 | [diff] [blame] | 1574 | - djm@cvs.openbsd.org 2010/04/10 05:48:16 |
| 1575 | [mux.c] |
| 1576 | fix NULL dereference; from matthew.haub AT alumni.adelaide.edu.au |
Damien Miller | b1b1704 | 2010-04-16 15:54:19 +1000 | [diff] [blame] | 1577 | - djm@cvs.openbsd.org 2010/04/14 22:27:42 |
| 1578 | [ssh_config.5 sshconnect.c] |
| 1579 | expand %r => remote username in ssh_config:ProxyCommand; |
| 1580 | ok deraadt markus |
Damien Miller | 031c910 | 2010-04-16 15:54:44 +1000 | [diff] [blame] | 1581 | - markus@cvs.openbsd.org 2010/04/15 20:32:55 |
| 1582 | [ssh-pkcs11.c] |
| 1583 | retry lookup for private key if there's no matching key with CKA_SIGN |
| 1584 | attribute enabled; this fixes fixes MuscleCard support (bugzilla #1736) |
| 1585 | ok djm@ |
Damien Miller | 4e270b0 | 2010-04-16 15:56:21 +1000 | [diff] [blame] | 1586 | - djm@cvs.openbsd.org 2010/04/16 01:47:26 |
| 1587 | [PROTOCOL.certkeys auth-options.c auth-options.h auth-rsa.c] |
| 1588 | [auth2-pubkey.c authfd.c key.c key.h myproposal.h ssh-add.c] |
| 1589 | [ssh-agent.c ssh-dss.c ssh-keygen.1 ssh-keygen.c ssh-rsa.c] |
| 1590 | [sshconnect.c sshconnect2.c sshd.c] |
| 1591 | revised certificate format ssh-{dss,rsa}-cert-v01@openssh.com with the |
| 1592 | following changes: |
| 1593 | |
| 1594 | move the nonce field to the beginning of the certificate where it can |
| 1595 | better protect against chosen-prefix attacks on the signature hash |
| 1596 | |
| 1597 | Rename "constraints" field to "critical options" |
| 1598 | |
| 1599 | Add a new non-critical "extensions" field |
| 1600 | |
| 1601 | Add a serial number |
| 1602 | |
| 1603 | The older format is still support for authentication and cert generation |
| 1604 | (use "ssh-keygen -t v00 -s ca_key ..." to generate a v00 certificate) |
| 1605 | |
| 1606 | ok markus@ |