Darren Tucker | 28b973e | 2009-08-28 10:16:44 +1000 | [diff] [blame] | 1 | 20090828 |
| 2 | - dtucker [auth-sia.c] Roll back the change for bug #1241 as it apparently |
| 3 | causes problems in some Tru64 configurations. |
Damien Miller | 8aac993 | 2009-08-28 10:40:30 +1000 | [diff] [blame] | 4 | - (djm) [sshd_config.5] downgrade mention of login.conf to be an example |
| 5 | and mention PAM as another provider for ChallengeResponseAuthentication; |
| 6 | bz#1408; ok dtucker@ |
Damien Miller | 0e26551 | 2009-08-28 10:43:13 +1000 | [diff] [blame] | 7 | - (djm) [sftp-server.c] bz#1535: accept ENOSYS as a fallback error when |
| 8 | attempting atomic rename(); ok dtucker@ |
Damien Miller | 7d4a268 | 2009-08-28 10:47:38 +1000 | [diff] [blame] | 9 | - (djm) [Makefile.in] bz#1505: Solaris make(1) doesn't accept make variables |
| 10 | in argv, so pass them in the environment; ok dtucker@ |
Darren Tucker | 3980b63 | 2009-08-28 11:02:37 +1000 | [diff] [blame] | 11 | - (dtucker) [channels.c configure.ac] Bug #1528: skip the tcgetattr call on |
| 12 | the pty master on Solaris, since it never succeeds and can hang if large |
| 13 | amounts of data is sent to the slave (eg a copy-paste). Based on a patch |
| 14 | originally from Doke Scott, ok djm@ |
Darren Tucker | 86e30a0 | 2009-08-28 11:21:06 +1000 | [diff] [blame] | 15 | - (dtucker) [clientloop.c configure.ac defines.h] Make the client's IO buffer |
| 16 | size a compile-time option and set it to 64k on Cygwin, since Corinna |
| 17 | reports that it makes a significant difference to performance. ok djm@ |
Darren Tucker | ac9f1b9 | 2009-08-28 15:01:20 +1000 | [diff] [blame^] | 18 | - (dtucker) [configure.ac] Fix the syntax of the Solaris tcgetattr entry. |
Darren Tucker | 28b973e | 2009-08-28 10:16:44 +1000 | [diff] [blame] | 19 | |
Darren Tucker | 2a5588d | 2009-08-20 16:16:01 +1000 | [diff] [blame] | 20 | 20090820 |
| 21 | - (dtucker) [includes.h] Bug #1634: do not include system glob.h if we're not |
| 22 | using it since the type conflicts can cause problems on FreeBSD. Patch |
| 23 | from Jonathan Chen. |
Darren Tucker | 82edf23 | 2009-08-20 16:20:50 +1000 | [diff] [blame] | 24 | - (dtucker) [session.c openbsd-compat/port-aix.h] Bugs #1249 and #1567: move |
| 25 | the setpcred call on AIX to immediately before the permanently_set_uid(). |
| 26 | Ensures that we still have privileges when we call chroot and |
| 27 | pam_open_sesson. Based on a patch from David Leonard. |
Darren Tucker | 2a5588d | 2009-08-20 16:16:01 +1000 | [diff] [blame] | 28 | |
Darren Tucker | 83d8f28 | 2009-08-17 09:35:22 +1000 | [diff] [blame] | 29 | 20090817 |
| 30 | - (dtucker) [configure.ac] Check for headers before libraries for openssl an |
| 31 | zlib, which should make the errors slightly more meaningful on platforms |
| 32 | where there's separate "-devel" packages for those. |
Darren Tucker | b5d5ee1 | 2009-08-17 09:40:00 +1000 | [diff] [blame] | 33 | - (dtucker) [sshlogin.c openbsd-compat/port-aix.{c,h}] Bug #1595: make |
| 34 | PrintLastLog work on AIX. Based in part on a patch from Miguel Sanders. |
Darren Tucker | 83d8f28 | 2009-08-17 09:35:22 +1000 | [diff] [blame] | 35 | |
Tim Rice | caeb164 | 2009-07-29 07:21:13 -0700 | [diff] [blame] | 36 | 20090729 |
| 37 | - (tim) [contrib/cygwin/ssh-user-config] Change script to call correct error |
| 38 | function. Patch from Corinna Vinschen. |
| 39 | |
Darren Tucker | 440089a | 2009-07-13 11:38:23 +1000 | [diff] [blame] | 40 | 20090713 |
| 41 | - (dtucker) [openbsd-compat/getrrsetbyname.c] Reduce answer buffer size so it |
| 42 | fits into 16 bits to work around a bug in glibc's resolver where it masks |
| 43 | off the buffer size at 16 bits. Patch from Hauke Lampe, ok djm jakob. |
| 44 | |
Darren Tucker | c4b22ca | 2009-07-12 21:56:29 +1000 | [diff] [blame] | 45 | 20090712 |
| 46 | - (dtucker) [configure.ac] Include sys/param.h for the sys/mount.h test, |
| 47 | prevents configure complaining on older BSDs. |
Darren Tucker | 8fdcba5 | 2009-07-12 21:58:42 +1000 | [diff] [blame] | 48 | - (dtucker [contrib/cygwin/ssh-{host,user}-config] Add license text. Patch |
| 49 | from Corinna Vinschen. |
Darren Tucker | 622d5c5 | 2009-07-12 22:07:21 +1000 | [diff] [blame] | 50 | - (dtucker) [auth-pam.c] Bug #1534: move the deletion of PAM credentials on |
Darren Tucker | 916fdda | 2009-07-12 22:12:28 +1000 | [diff] [blame] | 51 | logout to after the session close. Patch from Anicka Bernathova, |
| 52 | originally from Andreas Schwab via Novelll ok djm. |
Darren Tucker | c4b22ca | 2009-07-12 21:56:29 +1000 | [diff] [blame] | 53 | |
Darren Tucker | 4d4fdc0 | 2009-07-07 21:19:11 +1000 | [diff] [blame] | 54 | 20090707 |
| 55 | - (dtucker) [contrib/cygwin/ssh-host-config] better support for automated |
| 56 | scripts and fix usage of eval. Patch from Corinna Vinschen. |
| 57 | |
| 58 | 20090705 |
Darren Tucker | e841eb0 | 2009-07-06 07:11:13 +1000 | [diff] [blame] | 59 | - (dtucker) OpenBSD CVS Sync |
| 60 | - andreas@cvs.openbsd.org 2009/06/27 09:29:06 |
| 61 | [packet.h packet.c] |
| 62 | packet_bacup_state() and packet_restore_state() will be used to |
| 63 | temporarily save the current state ren resuming a suspended connection. |
| 64 | ok markus@ |
Darren Tucker | 466df21 | 2009-07-06 07:11:52 +1000 | [diff] [blame] | 65 | - andreas@cvs.openbsd.org 2009/06/27 09:32:43 |
| 66 | [roaming_common.c roaming.h] |
| 67 | It may be necessary to retransmit some data when resuming, so add it |
| 68 | to a buffer when roaming is enabled. |
| 69 | Most of this code was written by Martin Forssen, maf at appgate dot com. |
| 70 | ok markus@ |
Darren Tucker | 71e4d54 | 2009-07-06 07:12:27 +1000 | [diff] [blame] | 71 | - andreas@cvs.openbsd.org 2009/06/27 09:35:06 |
| 72 | [readconf.h readconf.c] |
| 73 | Add client option UseRoaming. It doesn't do anything yet but will |
| 74 | control whether the client tries to use roaming if enabled on the |
| 75 | server. From Martin Forssen. |
| 76 | ok markus@ |
Darren Tucker | cd6b1a2 | 2009-07-06 07:13:04 +1000 | [diff] [blame] | 77 | - markus@cvs.openbsd.org 2009/06/30 14:54:40 |
| 78 | [version.h] |
| 79 | crank version; ok deraadt |
Darren Tucker | 199b134 | 2009-07-06 07:16:56 +1000 | [diff] [blame] | 80 | - dtucker@cvs.openbsd.org 2009/07/02 02:11:47 |
| 81 | [ssh.c] |
| 82 | allow for long home dir paths (bz #1615). ok deraadt |
| 83 | (based in part on a patch from jchadima at redhat) |
Darren Tucker | de0c025 | 2009-07-06 07:17:35 +1000 | [diff] [blame] | 84 | - stevesk@cvs.openbsd.org 2009/07/05 19:28:33 |
| 85 | [clientloop.c] |
| 86 | only send SSH2_MSG_DISCONNECT if we're in compat20; from dtucker@ |
| 87 | ok deraadt@ markus@ |
Darren Tucker | e841eb0 | 2009-07-06 07:11:13 +1000 | [diff] [blame] | 88 | |
Darren Tucker | 821d3db | 2009-06-22 16:11:06 +1000 | [diff] [blame] | 89 | 20090622 |
| 90 | - (dtucker) OpenBSD CVS Sync |
| 91 | - dtucker@cvs.openbsd.org 2009/06/22 05:39:28 |
| 92 | [monitor_wrap.c monitor_mm.c ssh-keygen.c auth2.c gss-genr.c sftp-client.c] |
| 93 | alphabetize includes; reduces diff vs portable and style(9). |
| 94 | ok stevesk djm |
| 95 | (Id sync only; these were already in order in -portable) |
| 96 | |
Darren Tucker | 72efd74 | 2009-06-21 17:48:00 +1000 | [diff] [blame] | 97 | 20090621 |
| 98 | - (dtucker) OpenBSD CVS Sync |
| 99 | - markus@cvs.openbsd.org 2009/03/17 21:37:00 |
| 100 | [ssh.c] |
| 101 | pass correct argv[0] to openlog(); ok djm@ |
Darren Tucker | 3a6a51f | 2009-06-21 17:48:52 +1000 | [diff] [blame] | 102 | - jmc@cvs.openbsd.org 2009/03/19 15:15:09 |
| 103 | [ssh.1] |
| 104 | for "Ciphers", just point the reader to the keyword in ssh_config(5), just |
| 105 | as we do for "MACs": this stops us getting out of sync when the lists |
| 106 | change; |
| 107 | fixes documentation/6102, submitted by Peter J. Philipp |
| 108 | alternative fix proposed by djm |
| 109 | ok markus |
Darren Tucker | a096450 | 2009-06-21 17:49:36 +1000 | [diff] [blame] | 110 | - tobias@cvs.openbsd.org 2009/03/23 08:31:19 |
| 111 | [ssh-agent.c] |
| 112 | Fixed a possible out-of-bounds memory access if the environment variable |
| 113 | SHELL is shorter than 3 characters. |
| 114 | with input by and ok dtucker |
Darren Tucker | 9013323 | 2009-06-21 17:50:15 +1000 | [diff] [blame] | 115 | - tobias@cvs.openbsd.org 2009/03/23 19:38:04 |
| 116 | [ssh-agent.c] |
| 117 | My previous commit didn't fix the problem at all, so stick at my first |
| 118 | version of the fix presented to dtucker. |
| 119 | Issue notified by Matthias Barkhoff (matthias dot barkhoff at gmx dot de). |
| 120 | ok dtucker |
Darren Tucker | 5837b51 | 2009-06-21 17:52:27 +1000 | [diff] [blame] | 121 | - sobrado@cvs.openbsd.org 2009/03/26 08:38:39 |
| 122 | [sftp-server.8 sshd.8 ssh-agent.1] |
| 123 | fix a few typographical errors found by spell(1). |
| 124 | ok dtucker@, jmc@ |
Darren Tucker | af501cf | 2009-06-21 17:53:04 +1000 | [diff] [blame] | 125 | - stevesk@cvs.openbsd.org 2009/04/13 19:07:44 |
| 126 | [sshd_config.5] |
| 127 | fix possessive; ok djm@ |
Darren Tucker | b62f1a8 | 2009-06-21 17:53:48 +1000 | [diff] [blame] | 128 | - stevesk@cvs.openbsd.org 2009/04/14 16:33:42 |
| 129 | [sftp-server.c] |
| 130 | remove unused option character from getopt() optstring; ok markus@ |
Darren Tucker | 3b59dfa | 2009-06-21 17:54:47 +1000 | [diff] [blame] | 131 | - jj@cvs.openbsd.org 2009/04/14 21:10:54 |
| 132 | [servconf.c] |
| 133 | Fixed a few the-the misspellings in comments. Skipped a bunch in |
| 134 | binutils,gcc and so on. ok jmc@ |
Darren Tucker | ac46a91 | 2009-06-21 17:55:23 +1000 | [diff] [blame] | 135 | - stevesk@cvs.openbsd.org 2009/04/17 19:23:06 |
| 136 | [session.c] |
| 137 | use INTERNAL_SFTP_NAME for setproctitle() of in-process sftp-server; |
| 138 | ok djm@ markus@ |
Darren Tucker | 00fcd71 | 2009-06-21 17:56:00 +1000 | [diff] [blame] | 139 | - stevesk@cvs.openbsd.org 2009/04/17 19:40:17 |
| 140 | [sshd_config.5] |
| 141 | clarify that even internal-sftp needs /dev/log for logging to work; ok |
| 142 | markus@ |
Darren Tucker | f92077f | 2009-06-21 17:56:25 +1000 | [diff] [blame] | 143 | - jmc@cvs.openbsd.org 2009/04/18 18:39:10 |
| 144 | [sshd_config.5] |
| 145 | tweak previous; ok stevesk |
Darren Tucker | 51dbe50 | 2009-06-21 17:56:51 +1000 | [diff] [blame] | 146 | - stevesk@cvs.openbsd.org 2009/04/21 15:13:17 |
| 147 | [sshd_config.5] |
| 148 | clarify we cd to user's home after chroot; ok markus@ on |
| 149 | earlier version; tweaks and ok jmc@ |
Darren Tucker | f7288d7 | 2009-06-21 18:12:20 +1000 | [diff] [blame] | 150 | - andreas@cvs.openbsd.org 2009/05/25 06:48:01 |
| 151 | [channels.c packet.c clientloop.c packet.h serverloop.c monitor_wrap.c |
| 152 | monitor.c] |
| 153 | Put the globals in packet.c into a struct and don't access it directly |
| 154 | from other files. No functional changes. |
| 155 | ok markus@ djm@ |
| 156 | - andreas@cvs.openbsd.org 2009/05/27 06:31:25 |
| 157 | [canohost.h canohost.c] |
| 158 | Add clear_cached_addr(), needed for upcoming changes allowing the peer |
| 159 | address to change. |
| 160 | ok markus@ |
Darren Tucker | 39c7632 | 2009-06-21 18:13:57 +1000 | [diff] [blame] | 161 | - andreas@cvs.openbsd.org 2009/05/27 06:33:39 |
| 162 | [clientloop.c] |
| 163 | Send SSH2_MSG_DISCONNECT when the client disconnects. From a larger |
| 164 | change from Martin Forssen, maf at appgate dot com. |
| 165 | ok markus@ |
Darren Tucker | 12b4a65 | 2009-06-21 18:14:48 +1000 | [diff] [blame] | 166 | - andreas@cvs.openbsd.org 2009/05/27 06:34:36 |
| 167 | [kex.c kex.h] |
| 168 | Move the KEX_COOKIE_LEN define to kex.h |
| 169 | ok markus@ |
Darren Tucker | 5b48cdd | 2009-06-21 18:15:25 +1000 | [diff] [blame] | 170 | - andreas@cvs.openbsd.org 2009/05/27 06:36:07 |
| 171 | [packet.h packet.c] |
| 172 | Add packet_put_int64() and packet_get_int64(), part of a larger change |
| 173 | from Martin Forssen. |
Darren Tucker | 761c389 | 2009-06-21 18:16:26 +1000 | [diff] [blame] | 174 | ok markus@ |
| 175 | - andreas@cvs.openbsd.org 2009/05/27 06:38:16 |
| 176 | [sshconnect.h sshconnect.c] |
| 177 | Un-static ssh_exchange_identification(), part of a larger change from |
| 178 | Martin Forssen and needed for upcoming changes. |
| 179 | ok markus@ |
Darren Tucker | 1cc55d7 | 2009-06-21 18:17:19 +1000 | [diff] [blame] | 180 | - andreas@cvs.openbsd.org 2009/05/28 16:50:16 |
| 181 | [sshd.c packet.c serverloop.c monitor_wrap.c clientloop.c sshconnect.c |
Darren Tucker | c5564e1 | 2009-06-21 18:53:53 +1000 | [diff] [blame] | 182 | monitor.c Added roaming.h roaming_common.c roaming_dummy.c] |
Darren Tucker | 1cc55d7 | 2009-06-21 18:17:19 +1000 | [diff] [blame] | 183 | Keep track of number of bytes read and written. Needed for upcoming |
| 184 | changes. Most code from Martin Forssen, maf at appgate dot com. |
| 185 | ok markus@ |
Darren Tucker | c5564e1 | 2009-06-21 18:53:53 +1000 | [diff] [blame] | 186 | Also, applied appropriate changes to Makefile.in |
Darren Tucker | b422afa | 2009-06-21 18:58:46 +1000 | [diff] [blame] | 187 | - andreas@cvs.openbsd.org 2009/06/12 20:43:22 |
| 188 | [monitor.c packet.c] |
| 189 | Fix warnings found by chl@ and djm@ and change roaming_atomicio's |
| 190 | return type to match atomicio's |
| 191 | Diff from djm@, ok markus@ |
Darren Tucker | 7b935c7 | 2009-06-21 18:59:36 +1000 | [diff] [blame] | 192 | - andreas@cvs.openbsd.org 2009/06/12 20:58:32 |
| 193 | [packet.c] |
| 194 | Move some more statics into session_state |
| 195 | ok markus@ djm@ |
Darren Tucker | 6ae35ac | 2009-06-21 19:00:20 +1000 | [diff] [blame] | 196 | - dtucker@cvs.openbsd.org 2009/06/21 07:37:15 |
| 197 | [kexdhs.c kexgexs.c] |
| 198 | abort if key_sign fails, preventing possible null deref. Based on report |
| 199 | from Paolo Ganci, ok markus@ djm@ |
Darren Tucker | e6b590e | 2009-06-21 19:08:48 +1000 | [diff] [blame] | 200 | - dtucker@cvs.openbsd.org 2009/06/21 09:04:03 |
| 201 | [roaming.h roaming_common.c roaming_dummy.c] |
| 202 | Add tags for the benefit of the sync scripts |
| 203 | Also: pull in the changes for 1.1->1.2 missed in the previous sync. |
Darren Tucker | 43e7a35 | 2009-06-21 19:50:08 +1000 | [diff] [blame] | 204 | - (dtucker) [auth2-jpake.c auth2.c canohost.h session.c] Whitespace and |
| 205 | header-order changes to reduce diff vs OpenBSD. |
Darren Tucker | 64cee36 | 2009-06-21 20:26:17 +1000 | [diff] [blame] | 206 | - (dtucker) [servconf.c sshd.c] More whitespace sync. |
Darren Tucker | 828c96d | 2009-06-21 22:22:08 +1000 | [diff] [blame] | 207 | - (dtucker) [roaming_common.c roaming_dummy.c] Wrap #include <inttypes.h> in |
| 208 | ifdef. |
Darren Tucker | 72efd74 | 2009-06-21 17:48:00 +1000 | [diff] [blame] | 209 | |
Darren Tucker | 3278062 | 2009-06-16 16:11:02 +1000 | [diff] [blame] | 210 | 20090616 |
| 211 | - (dtucker) [configure.ac defines.h] Bug #1607: handle the case where fsid_t |
| 212 | is a struct with a __val member. Fixes build on, eg, Redhat 6.2. |
| 213 | |
Darren Tucker | a422d97 | 2009-05-04 12:52:47 +1000 | [diff] [blame] | 214 | 20090504 |
| 215 | - (dtucker) [sshlogin.c] Move the NO_SSH_LASTLOG #ifndef line to include |
| 216 | variable declarations. Should prevent unused warnings anywhere it's set |
| 217 | (only Crays as far as I can tell) and be a no-op everywhere else. |
| 218 | |
Tim Rice | a74000e | 2009-03-18 11:25:02 -0700 | [diff] [blame] | 219 | 20090318 |
| 220 | - (tim) [configure.ac] Remove setting IP_TOS_IS_BROKEN for Cygwin. The problem |
| 221 | that setsockopt(IP_TOS) doesn't work on Cygwin has been fixed since 2005. |
| 222 | Based on patch from vinschen at redhat com. |
| 223 | |
Darren Tucker | 9d86e5d | 2009-03-08 11:40:27 +1100 | [diff] [blame] | 224 | 20090308 |
| 225 | - (dtucker) [auth-passwd.c auth1.c auth2-kbdint.c auth2-none.c auth2-passwd.c |
| 226 | auth2-pubkey.c session.c openbsd-compat/bsd-cygwin_util.{c,h} |
| 227 | openbsd-compat/daemon.c] Remove support for Windows 95/98/ME and very old |
| 228 | version of Cygwin. Patch from vinschen at redhat com. |
| 229 | |
Darren Tucker | 558d6ca | 2009-03-07 10:22:10 +1100 | [diff] [blame] | 230 | 20090307 |
| 231 | - (dtucker) [contrib/aix/buildbff.sh] Only try to rename ssh_prng_cmds if it |
| 232 | exists (it's not created if OpenSSL's PRNG is self-seeded, eg if the OS |
| 233 | has a /dev/random). |
Darren Tucker | 8aae6ff | 2009-03-07 12:01:47 +1100 | [diff] [blame] | 234 | - (dtucker) [schnorr.c openbsd-compat/openssl-compat.{c,h}] Add |
| 235 | EVP_DigestUpdate to the OLD_EVP compatibility functions and tell schnorr.c |
| 236 | to use them. Allows building with older OpenSSL versions. |
Darren Tucker | ccfee05 | 2009-03-07 12:32:22 +1100 | [diff] [blame] | 237 | - (dtucker) [configure.ac defines.h] Check for in_port_t and typedef if needed. |
Darren Tucker | 30ed668 | 2009-03-07 18:06:22 +1100 | [diff] [blame] | 238 | - (dtucker) [configure.ac] Missing comma in type list. |
Darren Tucker | 3e7e15f | 2009-03-07 22:22:35 +1100 | [diff] [blame] | 239 | - (dtucker) [configure.ac openbsd-compat/openssl-compat.{c,h}] |
| 240 | EVP_DigestUpdate does not exactly match the other OLD_EVP functions (eg |
| 241 | in openssl 0.9.6) so add an explicit test for it. |
Darren Tucker | 558d6ca | 2009-03-07 10:22:10 +1100 | [diff] [blame] | 242 | |
Damien Miller | cee8523 | 2009-03-06 00:58:22 +1100 | [diff] [blame] | 243 | 20090306 |
| 244 | - (djm) OpenBSD CVS Sync |
| 245 | - djm@cvs.openbsd.org 2009/03/05 07:18:19 |
| 246 | [auth2-jpake.c jpake.c jpake.h monitor_wrap.c monitor_wrap.h schnorr.c] |
| 247 | [sshconnect2.c] |
| 248 | refactor the (disabled) Schnorr proof code to make it a little more |
| 249 | generally useful |
Damien Miller | 447e387 | 2009-03-06 00:58:39 +1100 | [diff] [blame] | 250 | - djm@cvs.openbsd.org 2009/03/05 11:30:50 |
| 251 | [uuencode.c] |
| 252 | document what these functions do so I don't ever have to recuse into |
| 253 | b64_pton/ntop to remember their return values |
Damien Miller | cee8523 | 2009-03-06 00:58:22 +1100 | [diff] [blame] | 254 | |
Damien Miller | 1991384 | 2009-02-23 10:53:58 +1100 | [diff] [blame] | 255 | 20090223 |
| 256 | - (djm) OpenBSD CVS Sync |
| 257 | - djm@cvs.openbsd.org 2009/02/22 23:50:57 |
| 258 | [ssh_config.5 sshd_config.5] |
| 259 | don't advertise experimental options |
Damien Miller | 0296ae8 | 2009-02-23 11:00:24 +1100 | [diff] [blame] | 260 | - djm@cvs.openbsd.org 2009/02/22 23:59:25 |
| 261 | [sshd_config.5] |
| 262 | missing period |
Damien Miller | 582ca6b | 2009-02-23 11:09:25 +1100 | [diff] [blame] | 263 | - djm@cvs.openbsd.org 2009/02/23 00:06:15 |
| 264 | [version.h] |
| 265 | openssh-5.2 |
Damien Miller | 5d0d530 | 2009-02-23 11:11:57 +1100 | [diff] [blame] | 266 | - (djm) [README] update for 5.2 |
Damien Miller | faec50b | 2009-02-23 11:12:29 +1100 | [diff] [blame] | 267 | - (djm) Release openssh-5.2p1 |
Damien Miller | 1991384 | 2009-02-23 10:53:58 +1100 | [diff] [blame] | 268 | |
Damien Miller | 9eab956 | 2009-02-22 08:47:02 +1100 | [diff] [blame] | 269 | 20090222 |
| 270 | - (djm) OpenBSD CVS Sync |
| 271 | - tobias@cvs.openbsd.org 2009/02/21 19:32:04 |
| 272 | [misc.c sftp-server-main.c ssh-keygen.c] |
| 273 | Added missing newlines in error messages. |
| 274 | ok dtucker |
| 275 | |
Damien Miller | e8001d4 | 2009-02-21 12:45:02 +1100 | [diff] [blame] | 276 | 20090221 |
| 277 | - (djm) OpenBSD CVS Sync |
| 278 | - djm@cvs.openbsd.org 2009/02/17 01:28:32 |
| 279 | [ssh_config] |
| 280 | sync with revised default ciphers; pointed out by dkrause@ |
Damien Miller | 2591838 | 2009-02-21 12:45:18 +1100 | [diff] [blame] | 281 | - djm@cvs.openbsd.org 2009/02/18 04:31:21 |
| 282 | [schnorr.c] |
| 283 | signature should hash over the entire group, not just the generator |
| 284 | (this is still disabled code) |
Damien Miller | 7691e5f | 2009-02-21 18:03:04 +1100 | [diff] [blame] | 285 | - (djm) [contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 286 | [contrib/suse/openssh.spec] Prepare for 5.2p1 |
Damien Miller | e8001d4 | 2009-02-21 12:45:02 +1100 | [diff] [blame] | 287 | |
Damien Miller | 3f94aaf | 2009-02-16 15:21:39 +1100 | [diff] [blame] | 288 | 20090216 |
| 289 | - (djm) [regress/conch-ciphers.sh regress/putty-ciphers.sh] |
| 290 | [regress/putty-kex.sh regress/putty-transfer.sh] Downgrade disabled |
| 291 | interop tests from FATAL error to a warning. Allows some interop |
| 292 | tests to proceed if others are missing necessary prerequisites. |
Damien Miller | 9055172 | 2009-02-16 15:37:03 +1100 | [diff] [blame] | 293 | - (djm) [configure.ac] support GNU/kFreeBSD and GNU/kOpensolaris |
| 294 | systems; patch from Aurelien Jarno via rmh AT aybabtu.com |
Damien Miller | 3f94aaf | 2009-02-16 15:21:39 +1100 | [diff] [blame] | 295 | |
Damien Miller | fdd66fc | 2009-02-14 16:26:19 +1100 | [diff] [blame] | 296 | 20090214 |
| 297 | - (djm) OpenBSD CVS Sync |
| 298 | - dtucker@cvs.openbsd.org 2009/02/02 11:15:14 |
| 299 | [sftp.c] |
| 300 | Initialize a few variables to prevent spurious "may be used |
| 301 | uninitialized" warnings from newer gcc's. ok djm@ |
Damien Miller | 4bf648f | 2009-02-14 16:28:21 +1100 | [diff] [blame] | 302 | - djm@cvs.openbsd.org 2009/02/12 03:00:56 |
| 303 | [canohost.c canohost.h channels.c channels.h clientloop.c readconf.c] |
| 304 | [readconf.h serverloop.c ssh.c] |
| 305 | support remote port forwarding with a zero listen port (-R0:...) to |
| 306 | dyamically allocate a listen port at runtime (this is actually |
| 307 | specified in rfc4254); bz#1003 ok markus@ |
Damien Miller | 330d585 | 2009-02-14 16:33:09 +1100 | [diff] [blame] | 308 | - djm@cvs.openbsd.org 2009/02/12 03:16:01 |
| 309 | [serverloop.c] |
| 310 | tighten check for -R0:... forwarding: only allow dynamic allocation |
| 311 | if want_reply is set in the packet |
Damien Miller | 923e8bb | 2009-02-14 16:33:31 +1100 | [diff] [blame] | 312 | - djm@cvs.openbsd.org 2009/02/12 03:26:22 |
| 313 | [monitor.c] |
| 314 | some paranoia: check that the serialised key is really KEY_RSA before |
| 315 | diddling its internals |
Damien Miller | e2f4cc5 | 2009-02-14 16:33:49 +1100 | [diff] [blame] | 316 | - djm@cvs.openbsd.org 2009/02/12 03:42:09 |
| 317 | [ssh.1] |
| 318 | document -R0:... usage |
Damien Miller | 65fa4ca | 2009-02-14 16:34:05 +1100 | [diff] [blame] | 319 | - djm@cvs.openbsd.org 2009/02/12 03:44:25 |
| 320 | [ssh.1] |
| 321 | consistency: Dq => Ql |
Damien Miller | 85c6d8a | 2009-02-14 16:34:21 +1100 | [diff] [blame] | 322 | - djm@cvs.openbsd.org 2009/02/12 03:46:17 |
| 323 | [ssh_config.5] |
| 324 | document RemoteForward usage with 0 listen port |
Damien Miller | e379e10 | 2009-02-14 16:34:39 +1100 | [diff] [blame] | 325 | - jmc@cvs.openbsd.org 2009/02/12 07:34:20 |
| 326 | [ssh_config.5] |
| 327 | kill trailing whitespace; |
Damien Miller | 61433be | 2009-02-14 16:35:01 +1100 | [diff] [blame] | 328 | - markus@cvs.openbsd.org 2009/02/13 11:50:21 |
| 329 | [packet.c] |
| 330 | check for enc !=NULL in packet_start_discard |
Damien Miller | 6385e75 | 2009-02-14 18:00:52 +1100 | [diff] [blame] | 331 | - djm@cvs.openbsd.org 2009/02/14 06:35:49 |
| 332 | [PROTOCOL] |
| 333 | mention that eow and no-more-sessions extensions are sent only to |
| 334 | OpenSSH peers |
Damien Miller | fdd66fc | 2009-02-14 16:26:19 +1100 | [diff] [blame] | 335 | |
| 336 | 20090212 |
Damien Miller | 2de7624 | 2009-02-12 12:19:20 +1100 | [diff] [blame] | 337 | - (djm) [sshpty.c] bz#1419: OSX uses cloning ptys that automagically |
| 338 | set ownership and modes, so avoid explicitly setting them |
Damien Miller | 20e231f | 2009-02-12 13:12:21 +1100 | [diff] [blame] | 339 | - (djm) [configure.ac loginrec.c] bz#1421: fix lastlog support for OSX. |
| 340 | OSX provides a getlastlogxbyname function that automates the reading of |
| 341 | a lastlog file. Also, the pututxline function will update lastlog so |
| 342 | there is no need for loginrec.c to do it explicitly. Collapse some |
| 343 | overly verbose code while I'm in there. |
Damien Miller | 2de7624 | 2009-02-12 12:19:20 +1100 | [diff] [blame] | 344 | |
Darren Tucker | 642ebe5 | 2009-02-01 22:19:54 +1100 | [diff] [blame] | 345 | 20090201 |
| 346 | - (dtucker) [defines.h sshconnect.c] INET6_ADDRSTRLEN is now needed in |
| 347 | channels.c too, so move the definition for non-IP6 platforms to defines.h |
| 348 | where it can be shared. |
| 349 | |
Tim Rice | 6a32534 | 2009-01-29 12:30:01 -0800 | [diff] [blame] | 350 | 20090129 |
| 351 | - (tim) [contrib/cygwin/ssh-host-config] Patch from Corinna Vinschen. |
| 352 | If the CYGWIN environment variable is empty, the installer script |
| 353 | should not install the service with an empty CYGWIN variable, but |
| 354 | rather without setting CYGWNI entirely. |
Tim Rice | 0d8f2f3 | 2009-01-29 12:40:30 -0800 | [diff] [blame] | 355 | - (tim) [contrib/cygwin/ssh-host-config] Whitespace cleanup. No code changes. |
Tim Rice | 6a32534 | 2009-01-29 12:30:01 -0800 | [diff] [blame] | 356 | |
Tim Rice | ca3692d | 2009-01-28 12:50:04 -0800 | [diff] [blame] | 357 | 20090128 |
| 358 | - (tim) [contrib/cygwin/ssh-host-config] Patch from Corinna Vinschen. |
| 359 | Changes to work on Cygwin 1.5.x as well as on the new Cygwin 1.7.x. |
| 360 | The information given for the setting of the CYGWIN environment variable |
| 361 | is wrong for both releases so I just removed it, together with the |
| 362 | unnecessary (Cygwin 1.5.x) or wrong (Cygwin 1.7.x) default setting. |
| 363 | |
Damien Miller | b53d8a1 | 2009-01-28 16:13:04 +1100 | [diff] [blame] | 364 | 20081228 |
| 365 | - (djm) OpenBSD CVS Sync |
| 366 | - stevesk@cvs.openbsd.org 2008/12/09 03:20:42 |
| 367 | [channels.c servconf.c] |
| 368 | channel_print_adm_permitted_opens() should deal with all the printing |
| 369 | for that config option. suggested by markus@; ok markus@ djm@ |
| 370 | dtucker@ |
Damien Miller | c30def9 | 2009-01-28 16:13:39 +1100 | [diff] [blame] | 371 | - djm@cvs.openbsd.org 2008/12/09 04:32:22 |
| 372 | [auth2-chall.c] |
| 373 | replace by-hand string building with xasprinf(); ok deraadt@ |
Damien Miller | 62fd18a | 2009-01-28 16:14:09 +1100 | [diff] [blame] | 374 | - sobrado@cvs.openbsd.org 2008/12/09 15:35:00 |
| 375 | [sftp.1 sftp.c] |
| 376 | update for the synopses displayed by the 'help' command, there are a |
| 377 | few missing flags; add 'bye' to the output of 'help'; sorting and spacing. |
| 378 | jmc@ suggested replacing .Oo/.Oc with a single .Op macro. |
| 379 | ok jmc@ |
Damien Miller | b3f2c9f | 2009-01-28 16:15:30 +1100 | [diff] [blame] | 380 | - stevesk@cvs.openbsd.org 2008/12/09 22:37:33 |
| 381 | [clientloop.c] |
| 382 | fix typo in error message |
Damien Miller | 7375fe2 | 2009-01-28 16:16:00 +1100 | [diff] [blame] | 383 | - stevesk@cvs.openbsd.org 2008/12/10 03:55:20 |
| 384 | [addrmatch.c] |
| 385 | o cannot be NULL here but use xfree() to be consistent; ok djm@ |
Damien Miller | b2c17d4 | 2009-01-28 16:18:03 +1100 | [diff] [blame] | 386 | - stevesk@cvs.openbsd.org 2008/12/29 01:12:36 |
| 387 | [ssh-keyscan.1] |
| 388 | fix example, default key type is rsa for 3+ years; from |
| 389 | frederic.perrin@resel.fr |
Damien Miller | a70ac76 | 2009-01-28 16:19:52 +1100 | [diff] [blame] | 390 | - stevesk@cvs.openbsd.org 2008/12/29 02:23:26 |
| 391 | [pathnames.h] |
| 392 | no need to escape single quotes in comments |
Damien Miller | 1781901 | 2009-01-28 16:20:17 +1100 | [diff] [blame] | 393 | - okan@cvs.openbsd.org 2008/12/30 00:46:56 |
| 394 | [sshd_config.5] |
| 395 | add AllowAgentForwarding to available Match keywords list |
| 396 | ok djm |
Damien Miller | 7a60621 | 2009-01-28 16:22:34 +1100 | [diff] [blame] | 397 | - djm@cvs.openbsd.org 2009/01/01 21:14:35 |
| 398 | [channels.c] |
| 399 | call channel destroy callbacks on receipt of open failure messages. |
| 400 | fixes client hangs when connecting to a server that has MaxSessions=0 |
| 401 | set spotted by imorgan AT nas.nasa.gov; ok markus@ |
Damien Miller | ccf7e22 | 2009-01-28 16:23:06 +1100 | [diff] [blame] | 402 | - djm@cvs.openbsd.org 2009/01/01 21:17:36 |
| 403 | [kexgexs.c] |
| 404 | fix hash calculation for KEXGEX: hash over the original client-supplied |
| 405 | values and not the sanity checked versions that we acutally use; |
| 406 | bz#1540 reported by john.smith AT arrows.demon.co.uk |
| 407 | ok markus@ |
Damien Miller | 1781f53 | 2009-01-28 16:24:41 +1100 | [diff] [blame] | 408 | - djm@cvs.openbsd.org 2009/01/14 01:38:06 |
| 409 | [channels.c] |
| 410 | support SOCKS4A protocol, from dwmw2 AT infradead.org via bz#1482; |
| 411 | "looks ok" markus@ |
Damien Miller | 5bc6aae | 2009-01-28 16:27:31 +1100 | [diff] [blame] | 412 | - stevesk@cvs.openbsd.org 2009/01/15 17:38:43 |
| 413 | [readconf.c] |
| 414 | 1) use obsolete instead of alias for consistency |
| 415 | 2) oUserKnownHostsFile not obsolete but oGlobalKnownHostsFile2 is |
| 416 | so move the comment. |
| 417 | 3) reorder so like options are together |
| 418 | ok djm@ |
Damien Miller | a1c1b6c | 2009-01-28 16:29:49 +1100 | [diff] [blame] | 419 | - djm@cvs.openbsd.org 2009/01/22 09:46:01 |
| 420 | [channels.c channels.h session.c] |
| 421 | make Channel->path an allocated string, saving a few bytes here and |
| 422 | there and fixing bz#1380 in the process; ok markus@ |
Damien Miller | 9576ac4 | 2009-01-28 16:30:33 +1100 | [diff] [blame] | 423 | - djm@cvs.openbsd.org 2009/01/22 09:49:57 |
| 424 | [channels.c] |
| 425 | oops! I committed the wrong version of the Channel->path diff, |
| 426 | it was missing some tweaks suggested by stevesk@ |
Damien Miller | 3dc71ad | 2009-01-28 16:31:22 +1100 | [diff] [blame] | 427 | - djm@cvs.openbsd.org 2009/01/22 10:02:34 |
| 428 | [clientloop.c misc.c readconf.c readconf.h servconf.c servconf.h] |
| 429 | [serverloop.c ssh-keyscan.c ssh.c sshd.c] |
| 430 | make a2port() return -1 when it encounters an invalid port number |
| 431 | rather than 0, which it will now treat as valid (needed for future work) |
| 432 | adjust current consumers of a2port() to check its return value is <= 0, |
| 433 | which in turn required some things to be converted from u_short => int |
| 434 | make use of int vs. u_short consistent in some other places too |
| 435 | feedback & ok markus@ |
Damien Miller | e37dde0 | 2009-01-28 16:33:01 +1100 | [diff] [blame] | 436 | - djm@cvs.openbsd.org 2009/01/22 10:09:16 |
| 437 | [auth-options.c] |
| 438 | another chunk of a2port() diff that got away. wtfdjm?? |
Damien Miller | 67081b5 | 2009-01-28 16:33:31 +1100 | [diff] [blame] | 439 | - djm@cvs.openbsd.org 2009/01/23 07:58:11 |
| 440 | [myproposal.h] |
| 441 | prefer CTR modes and revised arcfour (i.e w/ discard) modes to CBC |
| 442 | modes; ok markus@ |
Damien Miller | 9aa72ba | 2009-01-28 16:34:00 +1100 | [diff] [blame] | 443 | - naddy@cvs.openbsd.org 2009/01/24 17:10:22 |
| 444 | [ssh_config.5 sshd_config.5] |
| 445 | sync list of preferred ciphers; ok djm@ |
Damien Miller | 13ae44c | 2009-01-28 16:38:41 +1100 | [diff] [blame] | 446 | - markus@cvs.openbsd.org 2009/01/26 09:58:15 |
| 447 | [cipher.c cipher.h packet.c] |
| 448 | Work around the CPNI-957037 Plaintext Recovery Attack by always |
| 449 | reading 256K of data on packet size or HMAC errors (in CBC mode only). |
| 450 | Help, feedback and ok djm@ |
| 451 | Feedback from Martin Albrecht and Paterson Kenny |
Damien Miller | b53d8a1 | 2009-01-28 16:13:04 +1100 | [diff] [blame] | 452 | |
Tim Rice | 351529c | 2009-01-07 10:04:12 -0800 | [diff] [blame] | 453 | 20090107 |
Damien Miller | 1598d6b | 2009-01-21 16:04:24 +1100 | [diff] [blame] | 454 | - (djm) [uidswap.c] bz#1412: Support >16 supplemental groups in OS X. |
| 455 | Patch based on one from vgiffin AT apple.com; ok dtucker@ |
Damien Miller | 819dbb6 | 2009-01-21 16:46:26 +1100 | [diff] [blame] | 456 | - (djm) [channels.c] bz#1419: support "on demand" X11 forwarding via |
| 457 | launchd on OS X; patch from vgiffin AT apple.com, slightly tweaked; |
| 458 | ok dtucker@ |
Damien Miller | 0266677 | 2009-01-21 20:29:20 +1100 | [diff] [blame] | 459 | - (djm) [contrib/ssh-copy-id.1 contrib/ssh-copy-id] bz#1492: Make |
| 460 | ssh-copy-id copy id_rsa.pub by default (instead of the legacy "identity" |
| 461 | key). Patch from cjwatson AT debian.org |
Damien Miller | 1598d6b | 2009-01-21 16:04:24 +1100 | [diff] [blame] | 462 | |
| 463 | 20090107 |
Tim Rice | 351529c | 2009-01-07 10:04:12 -0800 | [diff] [blame] | 464 | - (tim) [configure.ac defines.h openbsd-compat/port-uw.c |
| 465 | openbsd-compat/xcrypt.c] Add SECUREWARE support to OpenServer 6 SVR5 ABI. |
| 466 | OK djm@ dtucker@ |
Tim Rice | 2676791 | 2009-01-07 20:50:08 -0800 | [diff] [blame] | 467 | - (tim) [configure.ac] Move check_for_libcrypt_later=1 in *-*-sysv5*) section. |
| 468 | OpenServer 6 doesn't need libcrypt. |
Tim Rice | 351529c | 2009-01-07 10:04:12 -0800 | [diff] [blame] | 469 | |
Damien Miller | 586b005 | 2008-12-09 14:11:32 +1100 | [diff] [blame] | 470 | 20081209 |
| 471 | - (djm) OpenBSD CVS Sync |
| 472 | - djm@cvs.openbsd.org 2008/12/09 02:38:18 |
| 473 | [clientloop.c] |
| 474 | The ~C escape handler does not work correctly for multiplexed sessions - |
| 475 | it opens a commandline on the master session, instead of on the slave |
| 476 | that requested it. Disable it on slave sessions until such time as it |
| 477 | is fixed; bz#1543 report from Adrian Bridgett via Colin Watson |
| 478 | ok markus@ |
Damien Miller | 1be2cc4 | 2008-12-09 14:11:49 +1100 | [diff] [blame] | 479 | - djm@cvs.openbsd.org 2008/12/09 02:39:59 |
| 480 | [sftp.c] |
| 481 | Deal correctly with failures in remote stat() operation in sftp, |
| 482 | correcting fail-on-error behaviour in batchmode. bz#1541 report and |
| 483 | fix from anedvedicky AT gmail.com; ok markus@ |
Damien Miller | 0d772d9 | 2008-12-09 14:12:05 +1100 | [diff] [blame] | 484 | - djm@cvs.openbsd.org 2008/12/09 02:58:16 |
| 485 | [readconf.c] |
| 486 | don't leave junk (free'd) pointers around in Forward *fwd argument on |
| 487 | failure; avoids double-free in ~C -L handler when given an invalid |
| 488 | forwarding specification; bz#1539 report from adejong AT debian.org |
| 489 | via Colin Watson; ok markus@ dtucker@ |
Damien Miller | 7ebfad7 | 2008-12-09 14:12:33 +1100 | [diff] [blame] | 490 | - djm@cvs.openbsd.org 2008/12/09 03:02:37 |
| 491 | [sftp.1 sftp.c] |
| 492 | correct sftp(1) and corresponding usage syntax; |
| 493 | bz#1518 patch from imorgan AT nas.nasa.gov; ok deraadt@ improved diff jmc@ |
Damien Miller | 586b005 | 2008-12-09 14:11:32 +1100 | [diff] [blame] | 494 | |
Damien Miller | 7df2e40 | 2008-12-08 09:35:36 +1100 | [diff] [blame] | 495 | 20081208 |
| 496 | - (djm) [configure.ac] bz#1538: better test for ProPolice/SSP: actually |
| 497 | use some stack in main(). |
| 498 | Report and suggested fix from vapier AT gentoo.org |
Damien Miller | 8533c78 | 2008-12-08 09:54:40 +1100 | [diff] [blame] | 499 | - (djm) OpenBSD CVS Sync |
| 500 | - markus@cvs.openbsd.org 2008/12/02 19:01:07 |
| 501 | [clientloop.c] |
| 502 | we have to use the recipient's channel number (RFC 4254) for |
| 503 | SSH2_MSG_CHANNEL_SUCCESS/SSH2_MSG_CHANNEL_FAILURE messages, |
| 504 | otherwise we trigger 'Non-public channel' error messages on sshd |
| 505 | systems with clientkeepalive enabled; noticed by sturm; ok djm; |
Damien Miller | 5a33ec6 | 2008-12-08 09:55:02 +1100 | [diff] [blame] | 506 | - markus@cvs.openbsd.org 2008/12/02 19:08:59 |
| 507 | [serverloop.c] |
| 508 | backout 1.149, since it's not necessary and openssh clients send |
| 509 | broken CHANNEL_FAILURE/SUCCESS messages since about 2004; ok djm@ |
Damien Miller | 16a7307 | 2008-12-08 09:55:25 +1100 | [diff] [blame] | 510 | - markus@cvs.openbsd.org 2008/12/02 19:09:38 |
| 511 | [channels.c] |
| 512 | s/remote_id/id/ to be more consistent with other code; ok djm@ |
Damien Miller | 7df2e40 | 2008-12-08 09:35:36 +1100 | [diff] [blame] | 513 | |
Darren Tucker | 83795d6 | 2008-12-01 21:34:28 +1100 | [diff] [blame] | 514 | 20081201 |
| 515 | - (dtucker) [contrib/cygwin/{Makefile,ssh-host-config}] Add new doc files |
| 516 | and tweak the is-sshd-running check in ssh-host-config. Patch from |
| 517 | vinschen at redhat com. |
Darren Tucker | 99d11a3 | 2008-12-01 21:40:48 +1100 | [diff] [blame] | 518 | - (dtucker) OpenBSD CVS Sync |
| 519 | - markus@cvs.openbsd.org 2008/11/21 15:47:38 |
| 520 | [packet.c] |
| 521 | packet_disconnect() on padding error, too. should reduce the success |
| 522 | probability for the CPNI-957037 Plaintext Recovery Attack to 2^-18 |
| 523 | ok djm@ |
Darren Tucker | 2364564 | 2008-12-01 21:42:13 +1100 | [diff] [blame] | 524 | - dtucker@cvs.openbsd.org 2008/11/30 11:59:26 |
| 525 | [monitor_fdpass.c] |
| 526 | Retry sendmsg/recvmsg on EAGAIN and EINTR; ok djm@ |
Darren Tucker | 83795d6 | 2008-12-01 21:34:28 +1100 | [diff] [blame] | 527 | |
Darren Tucker | 69087ea | 2008-11-23 14:03:19 +1100 | [diff] [blame] | 528 | 20081123 |
| 529 | - (dtucker) [monitor_fdpass.c] Reduce diff vs OpenBSD by moving some |
| 530 | declarations, removing an unnecessary union member and adding whitespace. |
Darren Tucker | d3782b4 | 2008-11-23 19:05:53 +1100 | [diff] [blame] | 531 | cmsgbuf.tmp thing spotted by des at des no, ok djm some time ago. |
Darren Tucker | 69087ea | 2008-11-23 14:03:19 +1100 | [diff] [blame] | 532 | |
Tim Rice | 0f4d2c0 | 2008-11-18 21:26:41 -0800 | [diff] [blame] | 533 | 20081118 |
| 534 | - (tim) [addrmatch.c configure.ac] Some platforms do not have sin6_scope_id |
| 535 | member of sockaddr_in6. Also reported in Bug 1491 by David Leonard. OK and |
| 536 | feedback by djm@ |
| 537 | |
Darren Tucker | ff4350e | 2008-11-11 16:31:05 +1100 | [diff] [blame] | 538 | 20081111 |
| 539 | - (dtucker) OpenBSD CVS Sync |
| 540 | - jmc@cvs.openbsd.org 2008/11/05 11:22:54 |
| 541 | [servconf.c] |
| 542 | passord -> password; |
| 543 | fixes user/5975 from Rene Maroufi |
Darren Tucker | e15fb09 | 2008-11-11 16:31:43 +1100 | [diff] [blame] | 544 | - stevesk@cvs.openbsd.org 2008/11/07 00:42:12 |
| 545 | [ssh-keygen.c] |
| 546 | spelling/typo in comment |
Darren Tucker | b57fab6 | 2008-11-11 16:32:25 +1100 | [diff] [blame] | 547 | - stevesk@cvs.openbsd.org 2008/11/07 18:50:18 |
| 548 | [nchan.c] |
| 549 | add space to some log/debug messages for readability; ok djm@ markus@ |
Darren Tucker | c6d744e | 2008-11-11 16:33:03 +1100 | [diff] [blame] | 550 | - dtucker@cvs.openbsd.org 2008/11/07 23:34:48 |
| 551 | [auth2-jpake.c] |
| 552 | Move JPAKE define to make life easier for portable. ok djm@ |
Darren Tucker | 63917bd | 2008-11-11 16:33:48 +1100 | [diff] [blame] | 553 | - tobias@cvs.openbsd.org 2008/11/09 12:34:47 |
| 554 | [session.c ssh.1] |
| 555 | typo fixed (overriden -> overridden) |
| 556 | ok espie, jmc |
Darren Tucker | 49c31c4 | 2008-11-11 16:39:44 +1100 | [diff] [blame] | 557 | - stevesk@cvs.openbsd.org 2008/11/11 02:58:09 |
| 558 | [servconf.c] |
| 559 | USE_AFS not referenced so remove #ifdef. fixes sshd -T not printing |
| 560 | kerberosgetafstoken. ok dtucker@ |
| 561 | (Id sync only, we still want the ifdef in portable) |
Darren Tucker | 22662e8 | 2008-11-11 16:40:22 +1100 | [diff] [blame] | 562 | - stevesk@cvs.openbsd.org 2008/11/11 03:55:11 |
| 563 | [channels.c] |
| 564 | for sshd -T print 'permitopen any' vs. 'permitopen' for case of no |
| 565 | permitopen's; ok and input dtucker@ |
Darren Tucker | 4a6f62d | 2008-11-11 16:55:25 +1100 | [diff] [blame] | 566 | - djm@cvs.openbsd.org 2008/11/10 02:06:35 |
| 567 | [regress/putty-ciphers.sh] |
| 568 | PuTTY supports AES CTR modes, so interop test against them too |
Darren Tucker | ff4350e | 2008-11-11 16:31:05 +1100 | [diff] [blame] | 569 | |
Damien Miller | 7fc5c0f | 2008-11-05 16:12:11 +1100 | [diff] [blame] | 570 | 20081105 |
| 571 | - OpenBSD CVS Sync |
| 572 | - djm@cvs.openbsd.org 2008/11/03 08:59:41 |
| 573 | [servconf.c] |
| 574 | include MaxSessions in sshd -T output; patch from imorgan AT nas.nasa.gov |
Damien Miller | 6f66d34 | 2008-11-05 16:12:54 +1100 | [diff] [blame] | 575 | - djm@cvs.openbsd.org 2008/11/04 07:58:09 |
| 576 | [auth.c] |
| 577 | need unistd.h for close() prototype |
| 578 | (ID sync only) |
Damien Miller | 01ed227 | 2008-11-05 16:20:46 +1100 | [diff] [blame] | 579 | - djm@cvs.openbsd.org 2008/11/04 08:22:13 |
| 580 | [auth.h auth2.c monitor.c monitor.h monitor_wrap.c monitor_wrap.h] |
| 581 | [readconf.c readconf.h servconf.c servconf.h ssh2.h ssh_config.5] |
| 582 | [sshconnect2.c sshd_config.5 jpake.c jpake.h schnorr.c auth2-jpake.c] |
| 583 | [Makefile.in] |
| 584 | Add support for an experimental zero-knowledge password authentication |
| 585 | method using the J-PAKE protocol described in F. Hao, P. Ryan, |
| 586 | "Password Authenticated Key Exchange by Juggling", 16th Workshop on |
| 587 | Security Protocols, Cambridge, April 2008. |
| 588 | |
| 589 | This method allows password-based authentication without exposing |
| 590 | the password to the server. Instead, the client and server exchange |
| 591 | cryptographic proofs to demonstrate of knowledge of the password while |
| 592 | revealing nothing useful to an attacker or compromised endpoint. |
| 593 | |
| 594 | This is experimental, work-in-progress code and is presently |
| 595 | compiled-time disabled (turn on -DJPAKE in Makefile.inc). |
| 596 | |
| 597 | "just commit it. It isn't too intrusive." deraadt@ |
Damien Miller | 1a0442f | 2008-11-05 16:30:06 +1100 | [diff] [blame] | 598 | - stevesk@cvs.openbsd.org 2008/11/04 19:18:00 |
| 599 | [readconf.c] |
| 600 | because parse_forward() is now used to parse all forward types (DLR), |
| 601 | and it malloc's space for host variables, we don't need to malloc |
| 602 | here. fixes small memory leaks. |
| 603 | |
| 604 | previously dynamic forwards were not parsed in parse_forward() and |
| 605 | space was not malloc'd in that case. |
| 606 | |
| 607 | ok djm@ |
Damien Miller | 0164cb8 | 2008-11-05 16:30:31 +1100 | [diff] [blame] | 608 | - stevesk@cvs.openbsd.org 2008/11/05 03:23:09 |
| 609 | [clientloop.c ssh.1] |
| 610 | add dynamic forward escape command line; ok djm@ |
Damien Miller | 7fc5c0f | 2008-11-05 16:12:11 +1100 | [diff] [blame] | 611 | |
Damien Miller | 9f6fb56 | 2008-11-03 19:15:44 +1100 | [diff] [blame] | 612 | 20081103 |
| 613 | - OpenBSD CVS Sync |
| 614 | - sthen@cvs.openbsd.org 2008/07/24 23:55:30 |
| 615 | [ssh-keygen.1] |
| 616 | Add "ssh-keygen -F -l" to synopsis (displays fingerprint from |
| 617 | known_hosts). ok djm@ |
| 618 | - grunk@cvs.openbsd.org 2008/07/25 06:56:35 |
| 619 | [ssh_config] |
| 620 | Add VisualHostKey to example file, ok djm@ |
Damien Miller | c6aadd9 | 2008-11-03 19:16:20 +1100 | [diff] [blame] | 621 | - grunk@cvs.openbsd.org 2008/07/25 07:05:16 |
| 622 | [key.c] |
| 623 | In random art visualization, make sure to use the end marker only at the |
| 624 | end. Initial diff by Dirk Loss, tweaks and ok djm@ |
Damien Miller | c674d58 | 2008-11-03 19:16:57 +1100 | [diff] [blame] | 625 | - markus@cvs.openbsd.org 2008/07/31 14:48:28 |
| 626 | [sshconnect2.c] |
| 627 | don't allocate space for empty banners; report t8m at centrum.cz; |
| 628 | ok deraadt |
Damien Miller | fa51b16 | 2008-11-03 19:17:33 +1100 | [diff] [blame] | 629 | - krw@cvs.openbsd.org 2008/08/02 04:29:51 |
| 630 | [ssh_config.5] |
| 631 | whitepsace -> whitespace. From Matthew Clarke via bugs@. |
Damien Miller | ad793d5 | 2008-11-03 19:17:57 +1100 | [diff] [blame] | 632 | - djm@cvs.openbsd.org 2008/08/21 04:09:57 |
| 633 | [session.c] |
| 634 | allow ForceCommand internal-sftp with arguments. based on patch from |
| 635 | michael.barabanov AT gmail.com; ok markus@ |
Damien Miller | 15bce6b | 2008-11-03 19:19:12 +1100 | [diff] [blame] | 636 | - djm@cvs.openbsd.org 2008/09/06 12:24:13 |
| 637 | [kex.c] |
| 638 | OpenSSL 0.9.8h supplies a real EVP_sha256 so we do not need our |
| 639 | replacement anymore |
| 640 | (ID sync only for portable - we still need this) |
Damien Miller | 456e6f0 | 2008-11-03 19:20:10 +1100 | [diff] [blame] | 641 | - markus@cvs.openbsd.org 2008/09/11 14:22:37 |
| 642 | [compat.c compat.h nchan.c ssh.c] |
| 643 | only send eow and no-more-sessions requests to openssh 5 and newer; |
| 644 | fixes interop problems with broken ssh v2 implementations; ok djm@ |
Damien Miller | d58f560 | 2008-11-03 19:20:49 +1100 | [diff] [blame] | 645 | - millert@cvs.openbsd.org 2008/10/02 14:39:35 |
| 646 | [session.c] |
| 647 | Convert an unchecked strdup to xstrdup. OK deraadt@ |
Damien Miller | 49b78d4 | 2008-11-03 19:21:21 +1100 | [diff] [blame] | 648 | - jmc@cvs.openbsd.org 2008/10/03 13:08:12 |
| 649 | [sshd.8] |
| 650 | do not give an example of how to chmod files: we can presume the user |
| 651 | knows that. removes an ambiguity in the permission of authorized_keys; |
| 652 | ok deraadt |
Damien Miller | c4d1b36 | 2008-11-03 19:22:09 +1100 | [diff] [blame] | 653 | - deraadt@cvs.openbsd.org 2008/10/03 23:56:28 |
| 654 | [sshconnect2.c] |
| 655 | Repair strnvis() buffersize of 4*n+1, with termination gauranteed by the |
| 656 | function. |
| 657 | spotted by des@freebsd, who commited an incorrect fix to the freebsd tree |
| 658 | and (as is fairly typical) did not report the problem to us. But this fix |
| 659 | is correct. |
| 660 | ok djm |
Damien Miller | e272a5b | 2008-11-03 19:22:37 +1100 | [diff] [blame] | 661 | - djm@cvs.openbsd.org 2008/10/08 23:34:03 |
| 662 | [ssh.1 ssh.c] |
| 663 | Add -y option to force logging via syslog rather than stderr. |
| 664 | Useful for daemonised ssh connection (ssh -f). Patch originally from |
| 665 | and ok'd by markus@ |
Damien Miller | 51bde60 | 2008-11-03 19:23:10 +1100 | [diff] [blame] | 666 | - djm@cvs.openbsd.org 2008/10/09 03:50:54 |
| 667 | [servconf.c sshd_config.5] |
| 668 | support setting PermitEmptyPasswords in a Match block |
| 669 | requested in PR3891; ok dtucker@ |
Damien Miller | c13c3ee | 2008-11-03 19:23:28 +1100 | [diff] [blame] | 670 | - jmc@cvs.openbsd.org 2008/10/09 06:54:22 |
| 671 | [ssh.c] |
| 672 | add -y to usage(); |
Damien Miller | b4acb47 | 2008-11-03 19:23:45 +1100 | [diff] [blame] | 673 | - stevesk@cvs.openbsd.org 2008/10/10 04:55:16 |
| 674 | [scp.c] |
| 675 | spelling in comment; ok djm@ |
Damien Miller | 2f54ada | 2008-11-03 19:24:16 +1100 | [diff] [blame] | 676 | - stevesk@cvs.openbsd.org 2008/10/10 05:00:12 |
| 677 | [key.c] |
| 678 | typo in error message; ok djm@ |
Damien Miller | ece92c8 | 2008-11-03 19:25:03 +1100 | [diff] [blame] | 679 | - stevesk@cvs.openbsd.org 2008/10/10 16:43:27 |
| 680 | [ssh_config.5] |
| 681 | use 'Privileged ports can be forwarded only when logging in as root on |
| 682 | the remote machine.' for RemoteForward just like ssh.1 -R. |
| 683 | ok djm@ jmc@ |
| 684 | - stevesk@cvs.openbsd.org 2008/10/14 18:11:33 |
| 685 | [sshconnect.c] |
| 686 | use #define ROQUIET here; no binary change. ok dtucker@ |
Damien Miller | a414cd3 | 2008-11-03 19:25:21 +1100 | [diff] [blame] | 687 | - stevesk@cvs.openbsd.org 2008/10/17 18:36:24 |
| 688 | [ssh_config.5] |
| 689 | correct and clarify VisualHostKey; ok jmc@ |
Damien Miller | e7261c7 | 2008-11-03 19:25:40 +1100 | [diff] [blame] | 690 | - stevesk@cvs.openbsd.org 2008/10/30 19:31:16 |
| 691 | [clientloop.c sshd.c] |
| 692 | don't need to #include "monitor_fdpass.h" |
Damien Miller | 2b20a92 | 2008-11-03 19:26:00 +1100 | [diff] [blame] | 693 | - stevesk@cvs.openbsd.org 2008/10/31 15:05:34 |
| 694 | [dispatch.c] |
| 695 | remove unused #define DISPATCH_MIN; ok markus@ |
Damien Miller | f7475d7 | 2008-11-03 19:26:18 +1100 | [diff] [blame] | 696 | - djm@cvs.openbsd.org 2008/11/01 04:50:08 |
| 697 | [sshconnect2.c] |
| 698 | sprinkle ARGSUSED on dispatch handlers |
| 699 | nuke stale unusued prototype |
Damien Miller | a009433 | 2008-11-03 19:26:35 +1100 | [diff] [blame] | 700 | - stevesk@cvs.openbsd.org 2008/11/01 06:43:33 |
| 701 | [channels.c] |
| 702 | fix some typos in log messages; ok djm@ |
Damien Miller | c1719f7 | 2008-11-03 19:27:07 +1100 | [diff] [blame] | 703 | - sobrado@cvs.openbsd.org 2008/11/01 11:14:36 |
| 704 | [ssh-keyscan.1 ssh-keyscan.c] |
| 705 | the ellipsis is not an optional argument; while here, improve spacing. |
Damien Miller | a699d95 | 2008-11-03 19:27:34 +1100 | [diff] [blame] | 706 | - stevesk@cvs.openbsd.org 2008/11/01 17:40:33 |
| 707 | [clientloop.c readconf.c readconf.h ssh.c] |
| 708 | merge dynamic forward parsing into parse_forward(); |
| 709 | 'i think this is OK' djm@ |
Damien Miller | 660d7da | 2008-11-03 19:27:52 +1100 | [diff] [blame] | 710 | - stevesk@cvs.openbsd.org 2008/11/02 00:16:16 |
| 711 | [ttymodes.c] |
| 712 | protocol 2 tty modes support is now 7.5 years old so remove these |
| 713 | debug3()s; ok deraadt@ |
Damien Miller | a279d25 | 2008-11-03 19:28:07 +1100 | [diff] [blame] | 714 | - stevesk@cvs.openbsd.org 2008/11/03 01:07:02 |
| 715 | [readconf.c] |
| 716 | remove valueless comment |
Damien Miller | f4b3953 | 2008-11-03 19:28:21 +1100 | [diff] [blame] | 717 | - stevesk@cvs.openbsd.org 2008/11/03 02:44:41 |
| 718 | [readconf.c] |
| 719 | fix comment |
Damien Miller | 85dec73 | 2008-11-03 20:16:01 +1100 | [diff] [blame] | 720 | - (djm) [contrib/caldera/ssh-host-keygen contrib/suse/rc.sshd] |
| 721 | Make example scripts generate keys with default sizes rather than fixed, |
| 722 | non-default 1024 bits; patch from imorgan AT nas.nasa.gov |
Damien Miller | 250071f | 2008-11-03 20:18:12 +1100 | [diff] [blame] | 723 | - (djm) [contrib/sshd.pam.generic contrib/caldera/sshd.pam] |
| 724 | [contrib/redhat/sshd.pam] Move pam_nologin to account group from |
| 725 | incorrect auth group in example files; |
| 726 | patch from imorgan AT nas.nasa.gov |
Damien Miller | 9f6fb56 | 2008-11-03 19:15:44 +1100 | [diff] [blame] | 727 | |
Darren Tucker | c570ff7 | 2008-09-06 18:20:57 +1000 | [diff] [blame] | 728 | 20080906 |
| 729 | - (dtucker) [config.guess config.sub] Update to latest versions from |
| 730 | http://git.savannah.gnu.org/gitweb/ (2008-04-14 and 2008-06-16 |
| 731 | respectively). |
| 732 | |
Darren Tucker | 661f63b | 2008-08-30 07:32:37 +1000 | [diff] [blame] | 733 | 20080830 |
| 734 | - (dtucker) [openbsd-compat/bsd-poll.c] correctly check for number of FDs |
| 735 | larger than FD_SETSIZE (OpenSSH only ever uses poll with one fd). Patch |
| 736 | from Nicholas Marriott. |
| 737 | |
Damien Miller | aa5f433 | 2008-07-21 18:20:39 +1000 | [diff] [blame] | 738 | 20080721 |
| 739 | - (djm) OpenBSD CVS Sync |
Damien Miller | 6ef430d | 2008-07-23 17:40:04 +1000 | [diff] [blame] | 740 | - djm@cvs.openbsd.org 2008/07/23 07:36:55 |
| 741 | [servconf.c] |
| 742 | do not try to print options that have been compile-time disabled |
| 743 | in config test mode (sshd -T); report from nix-corp AT esperi.org.uk |
| 744 | ok dtucker@ |
Damien Miller | 212f0b0 | 2008-07-23 17:42:29 +1000 | [diff] [blame] | 745 | - (djm) [servconf.c] Print UsePAM option in config test mode (when it |
| 746 | has been compiled in); report from nix-corp AT esperi.org.uk |
| 747 | ok dtucker@ |
Damien Miller | 6ef430d | 2008-07-23 17:40:04 +1000 | [diff] [blame] | 748 | |
| 749 | 20080721 |
| 750 | - (djm) OpenBSD CVS Sync |
Damien Miller | aa5f433 | 2008-07-21 18:20:39 +1000 | [diff] [blame] | 751 | - jmc@cvs.openbsd.org 2008/07/18 22:51:01 |
| 752 | [sftp-server.8] |
| 753 | no need for .Pp before or after .Sh; |
Damien Miller | aaae43e | 2008-07-21 18:21:05 +1000 | [diff] [blame] | 754 | - djm@cvs.openbsd.org 2008/07/21 08:19:07 |
| 755 | [version.h] |
| 756 | openssh-5.1 |
Damien Miller | 1f8909c | 2008-07-21 18:21:52 +1000 | [diff] [blame] | 757 | - (djm) [README contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 758 | [contrib/suse/openssh.spec] Update version number in README and RPM specs |
Damien Miller | 8f42e9b | 2008-07-21 18:22:25 +1000 | [diff] [blame] | 759 | - (djm) Release OpenSSH-5.1 |
Damien Miller | aa5f433 | 2008-07-21 18:20:39 +1000 | [diff] [blame] | 760 | |
Damien Miller | 7ba0ca7 | 2008-07-17 18:57:06 +1000 | [diff] [blame] | 761 | 20080717 |
| 762 | - (djm) OpenBSD CVS Sync |
| 763 | - djm@cvs.openbsd.org 2008/07/17 08:48:00 |
| 764 | [sshconnect2.c] |
| 765 | strnvis preauth banner; pointed out by mpf@ ok markus@ |
Damien Miller | a1d03a5 | 2008-07-17 18:57:19 +1000 | [diff] [blame] | 766 | - djm@cvs.openbsd.org 2008/07/17 08:51:07 |
| 767 | [auth2-hostbased.c] |
| 768 | strip trailing '.' from hostname when HostbasedUsesNameFromPacketOnly=yes |
| 769 | report and patch from res AT qoxp.net (bz#1200); ok markus@ |
Darren Tucker | 9a3f2b4 | 2008-07-17 19:03:49 +1000 | [diff] [blame] | 770 | - (dtucker) [openbsd-compat/bsd-cygwin_util.c] Remove long-unneeded compat |
| 771 | code, replace with equivalent cygwin library call. Patch from vinschen |
Damien Miller | 2e28d86 | 2008-07-17 19:15:43 +1000 | [diff] [blame] | 772 | at redhat.com, ok djm@. |
| 773 | - (djm) [sshconnect2.c] vis.h isn't available everywhere |
Damien Miller | 7ba0ca7 | 2008-07-17 18:57:06 +1000 | [diff] [blame] | 774 | |
Damien Miller | 94717b0 | 2008-07-16 21:17:23 +1000 | [diff] [blame] | 775 | 20080716 |
| 776 | - OpenBSD CVS Sync |
| 777 | - djm@cvs.openbsd.org 2008/07/15 02:23:14 |
| 778 | [sftp.1] |
| 779 | number of pipelined requests is now 64; |
| 780 | prodded by Iain.Morgan AT nasa.gov |
Damien Miller | b9d3bee | 2008-07-16 22:40:52 +1000 | [diff] [blame] | 781 | - djm@cvs.openbsd.org 2008/07/16 11:51:14 |
| 782 | [clientloop.c] |
| 783 | rename variable first_gc -> last_gc (since it is actually the last |
| 784 | in the list). |
Damien Miller | 6ef1749 | 2008-07-16 22:42:06 +1000 | [diff] [blame] | 785 | - djm@cvs.openbsd.org 2008/07/16 11:52:19 |
| 786 | [channels.c] |
| 787 | this loop index should be automatic, not static |
Damien Miller | 94717b0 | 2008-07-16 21:17:23 +1000 | [diff] [blame] | 788 | |
Damien Miller | 81dec05 | 2008-07-14 11:28:29 +1000 | [diff] [blame] | 789 | 20080714 |
| 790 | - (djm) OpenBSD CVS Sync |
| 791 | - sthen@cvs.openbsd.org 2008/07/13 21:22:52 |
| 792 | [ssh-keygen.c] |
| 793 | Change "ssh-keygen -F [host] -l" to not display random art unless |
| 794 | -v is also specified, making it consistent with the manual and other |
| 795 | uses of -l. |
| 796 | ok grunk@ |
Damien Miller | 163886f | 2008-07-14 11:28:58 +1000 | [diff] [blame] | 797 | - djm@cvs.openbsd.org 2008/07/13 22:13:07 |
| 798 | [channels.c] |
| 799 | use struct sockaddr_storage instead of struct sockaddr for accept(2) |
| 800 | address argument. from visibilis AT yahoo.com in bz#1485; ok markus@ |
Damien Miller | 7f980d1 | 2008-07-14 11:29:24 +1000 | [diff] [blame] | 801 | - djm@cvs.openbsd.org 2008/07/13 22:16:03 |
| 802 | [sftp.c] |
| 803 | increase number of piplelined requests so they properly fill the |
| 804 | (recently increased) channel window. prompted by rapier AT psc.edu; |
| 805 | ok markus@ |
Damien Miller | 276571c | 2008-07-14 12:09:57 +1000 | [diff] [blame] | 806 | - djm@cvs.openbsd.org 2008/07/14 01:55:56 |
| 807 | [sftp-server.8] |
| 808 | mention requirement for /dev/log inside chroot when using sftp-server |
| 809 | with ChrootDirectory |
Damien Miller | ce02e5e | 2008-07-14 12:02:24 +1000 | [diff] [blame] | 810 | - (djm) [openbsd-compat/bindresvport.c] Rename variables s/sin/in/ to |
| 811 | avoid clash with sin(3) function; reported by |
| 812 | cristian.ionescu-idbohrn AT axis.com |
Damien Miller | 639ce59 | 2008-07-14 12:03:27 +1000 | [diff] [blame] | 813 | - (djm) [openbsd-compat/rresvport.c] Add unistd.h for missing close() |
| 814 | prototype; reported by cristian.ionescu-idbohrn AT axis.com |
Damien Miller | 36d7056 | 2008-07-14 12:04:43 +1000 | [diff] [blame] | 815 | - (djm) [umac.c] Rename variable s/buffer_ptr/bufp/ to avoid clash; |
| 816 | reported by cristian.ionescu-idbohrn AT axis.com |
Damien Miller | 1fc231c | 2008-07-14 12:12:52 +1000 | [diff] [blame] | 817 | - (djm) [contrib/cygwin/Makefile contrib/cygwin/ssh-host-config] |
| 818 | [contrib/cygwin/ssh-user-config contrib/cygwin/sshd-inetd] |
| 819 | Revamped and simplified Cygwin ssh-host-config script that uses |
| 820 | unified csih configuration tool. Requires recent Cygwin. |
| 821 | Patch from vinschen AT redhat.com |
Damien Miller | 81dec05 | 2008-07-14 11:28:29 +1000 | [diff] [blame] | 822 | |
Damien Miller | 2bcb866 | 2008-07-12 17:12:29 +1000 | [diff] [blame] | 823 | 20080712 |
| 824 | - (djm) OpenBSD CVS Sync |
| 825 | - djm@cvs.openbsd.org 2008/07/12 04:52:50 |
| 826 | [channels.c] |
| 827 | unbreak; move clearing of cctx struct to before first use |
| 828 | reported by dkrause@ |
Damien Miller | a034baf | 2008-07-12 17:12:49 +1000 | [diff] [blame] | 829 | - djm@cvs.openbsd.org 2008/07/12 05:33:41 |
| 830 | [scp.1] |
| 831 | better description for -i flag: |
| 832 | s/RSA authentication/public key authentication/ |
Damien Miller | c4657ef | 2008-07-14 21:37:36 +1000 | [diff] [blame] | 833 | - (djm) [openbsd-compat/fake-rfc2553.c openbsd-compat/fake-rfc2553.h] |
| 834 | return EAI_FAMILY when trying to lookup unsupported address family; |
| 835 | from vinschen AT redhat.com |
Damien Miller | 2bcb866 | 2008-07-12 17:12:29 +1000 | [diff] [blame] | 836 | |
Damien Miller | 2f7faf1 | 2008-07-11 17:34:35 +1000 | [diff] [blame] | 837 | 20080711 |
| 838 | - (djm) OpenBSD CVS Sync |
| 839 | - stevesk@cvs.openbsd.org 2008/07/07 00:31:41 |
| 840 | [ttymodes.c] |
| 841 | we don't need arg after the debug3() was removed. from lint. |
| 842 | ok djm@ |
Damien Miller | 87dd5f2 | 2008-07-11 17:35:09 +1000 | [diff] [blame] | 843 | - stevesk@cvs.openbsd.org 2008/07/07 23:32:51 |
| 844 | [key.c] |
| 845 | /*NOTREACHED*/ for lint warning: |
| 846 | warning: function key_equal falls off bottom without returning value |
| 847 | ok djm@ |
Damien Miller | dda5fff | 2008-07-11 17:35:37 +1000 | [diff] [blame] | 848 | - markus@cvs.openbsd.org 2008/07/10 18:05:58 |
| 849 | [channels.c] |
| 850 | missing bzero; from mickey; ok djm@ |
Damien Miller | b61f3fc | 2008-07-11 17:36:48 +1000 | [diff] [blame] | 851 | - markus@cvs.openbsd.org 2008/07/10 18:08:11 |
| 852 | [clientloop.c monitor.c monitor_wrap.c packet.c packet.h sshd.c] |
| 853 | sync v1 and v2 traffic accounting; add it to sshd, too; |
| 854 | ok djm@, dtucker@ |
Damien Miller | 2f7faf1 | 2008-07-11 17:34:35 +1000 | [diff] [blame] | 855 | |
Damien Miller | d9648ee | 2008-07-09 00:21:12 +1000 | [diff] [blame] | 856 | 20080709 |
| 857 | - (djm) [Makefile.in] Print "all tests passed" when all regress tests pass |
Damien Miller | 773a7b9 | 2008-07-09 20:54:05 +1000 | [diff] [blame] | 858 | - (djm) [auth1.c] Fix format string vulnerability in protocol 1 PAM |
| 859 | account check failure path. The vulnerable format buffer is supplied |
| 860 | from PAM and should not contain attacker-supplied data. |
Damien Miller | 7acf550 | 2008-07-09 20:54:37 +1000 | [diff] [blame] | 861 | - (djm) [auth.c] Missing unistd.h for close() |
Damien Miller | 73193b3 | 2008-07-09 21:07:19 +1000 | [diff] [blame] | 862 | - (djm) [configure.ac] Add -Wformat-security to CFLAGS for gcc 3.x and 4.x |
Damien Miller | d9648ee | 2008-07-09 00:21:12 +1000 | [diff] [blame] | 863 | |
Damien Miller | 22989f1 | 2008-07-05 08:59:43 +1000 | [diff] [blame] | 864 | 20080705 |
| 865 | - (djm) [auth.c] Fixed test for locked account on HP/UX with shadowed |
| 866 | passwords disabled. bz#1083 report & patch from senthilkumar_sen AT |
| 867 | hotpop.com, w/ dtucker@ |
Damien Miller | 20d1694 | 2008-07-05 09:36:58 +1000 | [diff] [blame] | 868 | - (djm) [atomicio.c configure.ac] Disable poll() fallback in atomiciov for |
| 869 | Tru64. readv doesn't seem to be a comparable object there. |
| 870 | bz#1386, patch from dtucker@ ok me |
Damien Miller | 42743cb | 2008-07-05 09:50:23 +1000 | [diff] [blame] | 871 | - (djm) [Makefile.in] Pass though pass to conch for interop tests |
Damien Miller | ff2e492 | 2008-07-05 09:52:03 +1000 | [diff] [blame] | 872 | - (djm) [configure.ac] unbreak: remove extra closing brace |
Damien Miller | d874fa5 | 2008-07-05 09:40:56 +1000 | [diff] [blame] | 873 | - (djm) OpenBSD CVS Sync |
| 874 | - djm@cvs.openbsd.org 2008/07/04 23:08:25 |
| 875 | [packet.c] |
| 876 | handle EINTR in packet_write_poll()l ok dtucker@ |
Damien Miller | 0b4d48b | 2008-07-05 09:44:53 +1000 | [diff] [blame] | 877 | - djm@cvs.openbsd.org 2008/07/04 23:30:16 |
| 878 | [auth1.c auth2.c] |
| 879 | Make protocol 1 MaxAuthTries logic match protocol 2's. |
| 880 | Do not treat the first protocol 2 authentication attempt as |
| 881 | a failure IFF it is for method "none". |
| 882 | Makes MaxAuthTries' user-visible behaviour identical for |
| 883 | protocol 1 vs 2. |
| 884 | ok dtucker@ |
Damien Miller | c9c96f2 | 2008-07-05 15:17:48 +1000 | [diff] [blame] | 885 | - djm@cvs.openbsd.org 2008/07/05 05:16:01 |
| 886 | [PROTOCOL] |
| 887 | grammar |
Damien Miller | 22989f1 | 2008-07-05 08:59:43 +1000 | [diff] [blame] | 888 | |
Darren Tucker | 7c99b1c | 2008-07-04 12:53:23 +1000 | [diff] [blame] | 889 | 20080704 |
| 890 | - (dtucker) OpenBSD CVS Sync |
| 891 | - djm@cvs.openbsd.org 2008/07/02 13:30:34 |
| 892 | [auth2.c] |
| 893 | really really remove the freebie "none" auth try for protocol 2 |
Darren Tucker | 9a2a609 | 2008-07-04 12:53:50 +1000 | [diff] [blame] | 894 | - djm@cvs.openbsd.org 2008/07/02 13:47:39 |
| 895 | [ssh.1 ssh.c] |
| 896 | When forking after authentication ("ssh -f") with ExitOnForwardFailure |
| 897 | enabled, delay the fork until after replies for any -R forwards have |
| 898 | been seen. Allows for robust detection of -R forward failure when |
| 899 | using -f (similar to bz#92); ok dtucker@ |
Darren Tucker | f5cafb0 | 2008-07-04 12:54:25 +1000 | [diff] [blame] | 900 | - otto@cvs.openbsd.org 2008/07/03 21:46:58 |
| 901 | [auth2-pubkey.c] |
| 902 | avoid nasty double free; ok dtucker@ djm@ |
Darren Tucker | b03fd02 | 2008-07-04 13:51:12 +1000 | [diff] [blame] | 903 | - djm@cvs.openbsd.org 2008/07/04 03:44:59 |
| 904 | [servconf.c groupaccess.h groupaccess.c] |
| 905 | support negation of groups in "Match group" block (bz#1315); ok dtucker@ |
Darren Tucker | 2784f1f | 2008-07-04 13:51:45 +1000 | [diff] [blame] | 906 | - dtucker@cvs.openbsd.org 2008/07/04 03:47:02 |
| 907 | [monitor.c] |
| 908 | Make debug a little clearer. ok djm@ |
Darren Tucker | 8c7a14e | 2008-07-04 17:08:58 +1000 | [diff] [blame] | 909 | - djm@cvs.openbsd.org 2008/06/30 08:07:34 |
| 910 | [regress/key-options.sh] |
| 911 | shell portability: use "=" instead of "==" in test(1) expressions, |
| 912 | double-quote string with backslash escaped / |
Darren Tucker | 5d6d70a | 2008-07-04 17:10:30 +1000 | [diff] [blame] | 913 | - djm@cvs.openbsd.org 2008/06/30 10:31:11 |
| 914 | [regress/{putty-transfer,putty-kex,putty-ciphers}.sh] |
| 915 | remove "set -e" left over from debugging |
Darren Tucker | b01bac1 | 2008-07-04 17:11:30 +1000 | [diff] [blame] | 916 | - djm@cvs.openbsd.org 2008/06/30 10:43:03 |
| 917 | [regress/conch-ciphers.sh] |
| 918 | explicitly disable conch options that could interfere with the test |
Darren Tucker | f7fa706 | 2008-07-04 14:10:19 +1000 | [diff] [blame] | 919 | - (dtucker) [sftp-server.c] Bug #1447: fall back to racy rename if link |
| 920 | returns EXDEV. Patch from Mike Garrison, ok djm@ |
Damien Miller | d8968ad | 2008-07-04 23:10:49 +1000 | [diff] [blame] | 921 | - (djm) [atomicio.c channels.c clientloop.c defines.h includes.h] |
| 922 | [packet.c scp.c serverloop.c sftp-client.c ssh-agent.c ssh-keyscan.c] |
| 923 | [sshd.c] Explicitly handle EWOULDBLOCK wherever we handle EAGAIN, on |
| 924 | some platforms (HP nonstop) it is a distinct errno; |
| 925 | bz#1467 reported by sconeu AT yahoo.com; ok dtucker@ |
| 926 | |
Darren Tucker | 00f00f0 | 2008-07-02 22:31:31 +1000 | [diff] [blame] | 927 | 20080702 |
| 928 | - (dtucker) OpenBSD CVS Sync |
| 929 | - djm@cvs.openbsd.org 2008/06/30 08:05:59 |
| 930 | [PROTOCOL.agent] |
| 931 | typo: s/constraint_date/constraint_data/ |
Darren Tucker | 8810dd4 | 2008-07-02 22:32:14 +1000 | [diff] [blame] | 932 | - djm@cvs.openbsd.org 2008/06/30 12:15:39 |
| 933 | [serverloop.c] |
| 934 | only pass channel requests on session channels through to the session |
| 935 | channel handler, avoiding spurious log messages; ok! markus@ |
Darren Tucker | 8748b96 | 2008-07-02 22:32:43 +1000 | [diff] [blame] | 936 | - djm@cvs.openbsd.org 2008/06/30 12:16:02 |
| 937 | [nchan.c] |
| 938 | only send eow@openssh.com notifications for session channels; ok! markus@ |
Darren Tucker | 1f781b1 | 2008-07-02 22:33:16 +1000 | [diff] [blame] | 939 | - djm@cvs.openbsd.org 2008/06/30 12:18:34 |
| 940 | [PROTOCOL] |
| 941 | clarify that eow@openssh.com is only sent on session channels |
Darren Tucker | 068e01f | 2008-07-02 22:33:55 +1000 | [diff] [blame] | 942 | - dtucker@cvs.openbsd.org 2008/07/01 07:20:52 |
| 943 | [sshconnect.c] |
| 944 | Check ExitOnForwardFailure if forwardings are disabled due to a failed |
| 945 | host key check. ok djm@ |
Darren Tucker | d7bdc0c | 2008-07-02 22:34:30 +1000 | [diff] [blame] | 946 | - dtucker@cvs.openbsd.org 2008/07/01 07:24:22 |
| 947 | [sshconnect.c sshd.c] |
| 948 | Send CR LF during protocol banner exchanges, but only for Protocol 2 only, |
| 949 | in order to comply with RFC 4253. bz #1443, ok djm@ |
Darren Tucker | f2e21de | 2008-07-02 22:35:00 +1000 | [diff] [blame] | 950 | - stevesk@cvs.openbsd.org 2008/07/01 23:12:47 |
| 951 | [PROTOCOL.agent] |
| 952 | fix some typos; ok djm@ |
Darren Tucker | 7499b0c | 2008-07-02 22:35:43 +1000 | [diff] [blame] | 953 | - djm@cvs.openbsd.org 2008/07/02 02:24:18 |
| 954 | [sshd_config sshd_config.5 sshd.8 servconf.c] |
| 955 | increase default size of ssh protocol 1 ephemeral key from 768 to 1024 |
| 956 | bits; prodded by & ok dtucker@ ok deraadt@ |
Darren Tucker | 33c787f | 2008-07-02 22:37:30 +1000 | [diff] [blame] | 957 | - dtucker@cvs.openbsd.org 2008/07/02 12:03:51 |
| 958 | [auth-rsa.c auth.c auth2-pubkey.c auth.h] |
| 959 | Merge duplicate host key file checks, based in part on a patch from Rob |
| 960 | Holland via bz #1348 . Also checks for non-regular files during protocol |
| 961 | 1 RSA auth. ok djm@ |
Darren Tucker | 4230a5d | 2008-07-02 22:56:09 +1000 | [diff] [blame] | 962 | - djm@cvs.openbsd.org 2008/07/02 12:36:39 |
| 963 | [auth2-none.c auth2.c] |
| 964 | Make protocol 2 MaxAuthTries behaviour a little more sensible: |
| 965 | Check whether client has exceeded MaxAuthTries before running |
| 966 | an authentication method and skip it if they have, previously it |
| 967 | would always allow one try (for "none" auth). |
| 968 | Preincrement failure count before post-auth test - previously this |
| 969 | checked and postincremented, also to allow one "none" try. |
| 970 | Together, these two changes always count the "none" auth method |
| 971 | which could be skipped by a malicious client (e.g. an SSH worm) |
| 972 | to get an extra attempt at a real auth method. They also make |
| 973 | MaxAuthTries=0 a useful way to block users entirely (esp. in a |
| 974 | sshd_config Match block). |
| 975 | Also, move sending of any preauth banner from "none" auth method |
| 976 | to the first call to input_userauth_request(), so worms that skip |
| 977 | the "none" method get to see it too. |
Darren Tucker | 00f00f0 | 2008-07-02 22:31:31 +1000 | [diff] [blame] | 978 | |
Damien Miller | 2e80cf2 | 2008-06-30 08:06:25 +1000 | [diff] [blame] | 979 | 20080630 |
| 980 | - (djm) OpenBSD CVS Sync |
| 981 | - dtucker@cvs.openbsd.org 2008/06/10 23:13:43 |
| 982 | [regress/Makefile regress/key-options.sh] |
| 983 | Add regress test for key options. ok djm@ |
Damien Miller | d9bfce8 | 2008-06-30 08:06:51 +1000 | [diff] [blame] | 984 | - dtucker@cvs.openbsd.org 2008/06/11 23:11:40 |
Damien Miller | 4268a13 | 2008-06-30 08:07:56 +1000 | [diff] [blame] | 985 | [regress/Makefile] |
Damien Miller | d9bfce8 | 2008-06-30 08:06:51 +1000 | [diff] [blame] | 986 | Don't run cipher-speed test by default; mistakenly enabled by me |
Damien Miller | 4268a13 | 2008-06-30 08:07:56 +1000 | [diff] [blame] | 987 | - djm@cvs.openbsd.org 2008/06/28 13:57:25 |
| 988 | [regress/Makefile regress/test-exec.sh regress/conch-ciphers.sh] |
| 989 | very basic regress test against Twisted Conch in "make interop" |
| 990 | target (conch is available in ports/devel/py-twisted/conch); |
| 991 | ok markus@ |
Damien Miller | a766cea | 2008-06-30 08:12:37 +1000 | [diff] [blame] | 992 | - (djm) [regress/Makefile] search for conch by path, like we do putty |
Damien Miller | 2e80cf2 | 2008-06-30 08:06:25 +1000 | [diff] [blame] | 993 | |
Damien Miller | f184bcf | 2008-06-29 22:45:13 +1000 | [diff] [blame] | 994 | 20080629 |
| 995 | - (djm) OpenBSD CVS Sync |
| 996 | - martynas@cvs.openbsd.org 2008/06/21 07:46:46 |
| 997 | [sftp.c] |
| 998 | use optopt to get invalid flag, instead of return value of getopt, |
| 999 | which is always '?'; ok djm@ |
Damien Miller | 007132a | 2008-06-29 22:45:37 +1000 | [diff] [blame] | 1000 | - otto@cvs.openbsd.org 2008/06/25 11:13:43 |
| 1001 | [key.c] |
| 1002 | add key length to visual fingerprint; zap magical constants; |
| 1003 | ok grunk@ djm@ |
Damien Miller | 9e72028 | 2008-06-29 22:46:35 +1000 | [diff] [blame] | 1004 | - djm@cvs.openbsd.org 2008/06/26 06:10:09 |
| 1005 | [sftp-client.c sftp-server.c] |
| 1006 | allow the sftp chmod(2)-equivalent operation to set set[ug]id/sticky |
| 1007 | bits. Note that this only affects explicit setting of modes (e.g. via |
| 1008 | sftp(1)'s chmod command) and not file transfers. (bz#1310) |
| 1009 | ok deraadt@ at c2k8 |
Damien Miller | 2e9cf49 | 2008-06-29 22:47:04 +1000 | [diff] [blame] | 1010 | - djm@cvs.openbsd.org 2008/06/26 09:19:40 |
| 1011 | [dh.c dh.h moduli.c] |
| 1012 | when loading moduli from /etc/moduli in sshd(8), check that they |
| 1013 | are of the expected "safe prime" structure and have had |
| 1014 | appropriate primality tests performed; |
| 1015 | feedback and ok dtucker@ |
Damien Miller | 1028824 | 2008-06-30 00:04:03 +1000 | [diff] [blame] | 1016 | - grunk@cvs.openbsd.org 2008/06/26 11:46:31 |
| 1017 | [readconf.c readconf.h ssh.1 ssh_config.5 sshconnect.c] |
| 1018 | Move SSH Fingerprint Visualization away from sharing the config option |
| 1019 | CheckHostIP to an own config option named VisualHostKey. |
| 1020 | While there, fix the behaviour that ssh would draw a random art picture |
| 1021 | on every newly seen host even when the option was not enabled. |
| 1022 | prodded by deraadt@, discussions, |
| 1023 | help and ok markus@ djm@ dtucker@ |
Damien Miller | 8639920 | 2008-06-30 00:04:31 +1000 | [diff] [blame] | 1024 | - jmc@cvs.openbsd.org 2008/06/26 21:11:46 |
| 1025 | [ssh.1] |
| 1026 | add VisualHostKey to the list of options listed in -o; |
Damien Miller | bd45afb | 2008-06-30 00:04:57 +1000 | [diff] [blame] | 1027 | - djm@cvs.openbsd.org 2008/06/28 07:25:07 |
| 1028 | [PROTOCOL] |
| 1029 | spelling fixes |
Damien Miller | 1cfadab | 2008-06-30 00:05:21 +1000 | [diff] [blame] | 1030 | - djm@cvs.openbsd.org 2008/06/28 13:58:23 |
| 1031 | [ssh-agent.c] |
| 1032 | refuse to add a key that has unknown constraints specified; |
| 1033 | ok markus |
Damien Miller | 471db5c | 2008-06-30 00:05:48 +1000 | [diff] [blame] | 1034 | - djm@cvs.openbsd.org 2008/06/28 14:05:15 |
| 1035 | [ssh-agent.c] |
| 1036 | reset global compat flag after processing a protocol 2 signature |
| 1037 | request with the legacy DSA encoding flag set; ok markus |
Damien Miller | 1e18beb | 2008-06-30 00:07:00 +1000 | [diff] [blame] | 1038 | - djm@cvs.openbsd.org 2008/06/28 14:08:30 |
| 1039 | [PROTOCOL PROTOCOL.agent] |
| 1040 | document the protocol used by ssh-agent; "looks ok" markus@ |
Damien Miller | f184bcf | 2008-06-29 22:45:13 +1000 | [diff] [blame] | 1041 | |
Damien Miller | 493f032 | 2008-06-28 16:01:35 +1000 | [diff] [blame] | 1042 | 20080628 |
| 1043 | - (djm) [RFC.nroff contrib/cygwin/Makefile contrib/suse/openssh.spec] |
| 1044 | RFC.nroff lacks a license, remove it (it is long gone in OpenBSD). |
| 1045 | |
Damien Miller | 60dcc62 | 2008-06-26 15:59:32 +1000 | [diff] [blame] | 1046 | 20080626 |
| 1047 | - (djm) [Makefile.in moduli.5] Include moduli(5) manpage from OpenBSD. |
| 1048 | (bz#1372) |
Damien Miller | f299ff8 | 2008-06-26 16:01:56 +1000 | [diff] [blame] | 1049 | - (djm) [ contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 1050 | [contrib/suse/openssh.spec] Include moduli.5 in RPM spec files. |
Damien Miller | 60dcc62 | 2008-06-26 15:59:32 +1000 | [diff] [blame] | 1051 | |
Darren Tucker | ed3cdc0 | 2008-06-16 23:29:18 +1000 | [diff] [blame] | 1052 | 20080616 |
| 1053 | - (dtucker) OpenBSD CVS Sync |
| 1054 | - dtucker@cvs.openbsd.org 2008/06/16 13:22:53 |
| 1055 | [session.c channels.c] |
| 1056 | Rename the isatty argument to is_tty so we don't shadow |
| 1057 | isatty(3). ok markus@ |
Darren Tucker | 1a48aec | 2008-06-16 23:35:56 +1000 | [diff] [blame] | 1058 | - (dtucker) [channels.c] isatty -> is_tty here too. |
Darren Tucker | ed3cdc0 | 2008-06-16 23:29:18 +1000 | [diff] [blame] | 1059 | |
Darren Tucker | 330c93f | 2008-06-16 02:27:48 +1000 | [diff] [blame] | 1060 | 20080615 |
| 1061 | - (dtucker) [configure.ac] Enable -fno-builtin-memset when using gcc. |
Damien Miller | 2a62847 | 2008-06-16 07:50:24 +1000 | [diff] [blame] | 1062 | - OpenBSD CVS Sync |
| 1063 | - dtucker@cvs.openbsd.org 2008/06/14 15:49:48 |
| 1064 | [sshd.c] |
| 1065 | wrap long line at 80 chars |
Damien Miller | 6ca16c6 | 2008-06-16 07:50:58 +1000 | [diff] [blame] | 1066 | - dtucker@cvs.openbsd.org 2008/06/14 17:07:11 |
| 1067 | [sshd.c] |
| 1068 | ensure default umask disallows at least group and world write; ok djm@ |
Damien Miller | 6051c94 | 2008-06-16 07:53:16 +1000 | [diff] [blame] | 1069 | - djm@cvs.openbsd.org 2008/06/14 18:33:43 |
| 1070 | [session.c] |
| 1071 | suppress the warning message from chdir(homedir) failures |
| 1072 | when chrooted (bz#1461); ok dtucker |
Damien Miller | c7ce0da | 2008-06-16 07:55:06 +1000 | [diff] [blame] | 1073 | - dtucker@cvs.openbsd.org 2008/06/14 19:42:10 |
| 1074 | [scp.1] |
| 1075 | Mention that scp follows symlinks during -r. bz #1466, |
| 1076 | from nectar at apple |
Damien Miller | c62a5af | 2008-06-16 07:55:46 +1000 | [diff] [blame] | 1077 | - dtucker@cvs.openbsd.org 2008/06/15 16:55:38 |
| 1078 | [sshd_config.5] |
| 1079 | MaxSessions is allowed in a Match block too |
Damien Miller | 307c1d1 | 2008-06-16 07:56:20 +1000 | [diff] [blame] | 1080 | - dtucker@cvs.openbsd.org 2008/06/15 16:58:40 |
| 1081 | [servconf.c sshd_config.5] |
| 1082 | Allow MaxAuthTries within a Match block. ok djm@ |
Damien Miller | d310d51 | 2008-06-16 07:59:23 +1000 | [diff] [blame] | 1083 | - djm@cvs.openbsd.org 2008/06/15 20:06:26 |
| 1084 | [channels.c channels.h session.c] |
| 1085 | don't call isatty() on a pty master, instead pass a flag down to |
| 1086 | channel_set_fds() indicating that te fds refer to a tty. Fixes a |
| 1087 | hang on exit on Solaris (bz#1463) in portable but is actually |
| 1088 | a generic bug; ok dtucker deraadt markus |
Darren Tucker | 330c93f | 2008-06-16 02:27:48 +1000 | [diff] [blame] | 1089 | |
Damien Miller | 8b7ab96 | 2008-06-15 10:55:34 +1000 | [diff] [blame] | 1090 | 20080614 |
| 1091 | - (djm) [openbsd-compat/sigact.c] Avoid NULL derefs in ancient sigaction |
| 1092 | replacement code; patch from ighighi AT gmail.com in bz#1240; |
| 1093 | ok dtucker |
| 1094 | |
Darren Tucker | 99bb761 | 2008-06-13 22:02:50 +1000 | [diff] [blame] | 1095 | 20080613 |
| 1096 | - (dtucker) OpenBSD CVS Sync |
| 1097 | - deraadt@cvs.openbsd.org 2008/06/13 09:44:36 |
| 1098 | [packet.c] |
| 1099 | compile on older gcc; no decl after code |
Darren Tucker | f2c16d3 | 2008-06-14 08:59:49 +1000 | [diff] [blame] | 1100 | - dtucker@cvs.openbsd.org 2008/06/13 13:56:59 |
| 1101 | [monitor.c] |
| 1102 | Clear key options in the monitor on failed authentication, prevents |
| 1103 | applying additional restrictions to non-pubkey authentications in |
| 1104 | the case where pubkey fails but another method subsequently succeeds. |
| 1105 | bz #1472, found by Colin Watson, ok markus@ djm@ |
Darren Tucker | d9526a5 | 2008-06-14 09:01:24 +1000 | [diff] [blame] | 1106 | - dtucker@cvs.openbsd.org 2008/06/13 14:18:51 |
| 1107 | [auth2-pubkey.c auth-rhosts.c] |
| 1108 | Include unistd.h for close(), prevents warnings in -portable |
Darren Tucker | 47e713b | 2008-06-14 09:01:54 +1000 | [diff] [blame] | 1109 | - dtucker@cvs.openbsd.org 2008/06/13 17:21:20 |
| 1110 | [mux.c] |
| 1111 | Friendlier error messages for mux fallback. ok djm@ |
Darren Tucker | 03ccc9b | 2008-06-14 09:02:25 +1000 | [diff] [blame] | 1112 | - dtucker@cvs.openbsd.org 2008/06/13 18:55:22 |
| 1113 | [scp.c] |
| 1114 | Prevent -Wsign-compare warnings on LP64 systems. bz #1192, ok deraadt@ |
Darren Tucker | f6bffb1 | 2008-06-14 09:04:26 +1000 | [diff] [blame] | 1115 | - grunk@cvs.openbsd.org 2008/06/13 20:13:26 |
| 1116 | [ssh.1] |
| 1117 | Explain the use of SSH fpr visualization using random art, and cite the |
| 1118 | original scientific paper inspiring that technique. |
| 1119 | Much help with English and nroff by jmc@, thanks. |
Darren Tucker | 30fd49e | 2008-06-14 09:14:46 +1000 | [diff] [blame] | 1120 | - (dtucker) [configure.ac] Bug #1276: avoid linking against libgssapi, which |
| 1121 | despite its name doesn't seem to implement all of GSSAPI. Patch from |
| 1122 | Jan Engelhardt, sanity checked by Simon Wilkinson. |
Darren Tucker | 99bb761 | 2008-06-13 22:02:50 +1000 | [diff] [blame] | 1123 | |
Darren Tucker | 1199673 | 2008-06-13 04:32:00 +1000 | [diff] [blame] | 1124 | 20080612 |
| 1125 | - (dtucker) OpenBSD CVS Sync |
| 1126 | - jmc@cvs.openbsd.org 2008/06/11 07:30:37 |
| 1127 | [sshd.8] |
| 1128 | kill trailing whitespace; |
Darren Tucker | 9c16ac9 | 2008-06-13 04:40:35 +1000 | [diff] [blame] | 1129 | - grunk@cvs.openbsd.org 2008/06/11 21:01:35 |
| 1130 | [ssh_config.5 key.h readconf.c readconf.h ssh-keygen.1 ssh-keygen.c key.c |
| 1131 | sshconnect.c] |
| 1132 | Introduce SSH Fingerprint ASCII Visualization, a technique inspired by the |
| 1133 | graphical hash visualization schemes known as "random art", and by |
| 1134 | Dan Kaminsky's musings on the subject during a BlackOp talk at the |
| 1135 | 23C3 in Berlin. |
| 1136 | Scientific publication (original paper): |
| 1137 | "Hash Visualization: a New Technique to improve Real-World Security", |
| 1138 | Perrig A. and Song D., 1999, International Workshop on Cryptographic |
| 1139 | Techniques and E-Commerce (CrypTEC '99) |
| 1140 | http://sparrow.ece.cmu.edu/~adrian/projects/validation/validation.pdf |
| 1141 | The algorithm used here is a worm crawling over a discrete plane, |
| 1142 | leaving a trace (augmenting the field) everywhere it goes. |
| 1143 | Movement is taken from dgst_raw 2bit-wise. Bumping into walls |
| 1144 | makes the respective movement vector be ignored for this turn, |
| 1145 | thus switching to the other color of the chessboard. |
| 1146 | Graphs are not unambiguous for now, because circles in graphs can be |
| 1147 | walked in either direction. |
| 1148 | discussions with several people, |
| 1149 | help, corrections and ok markus@ djm@ |
Darren Tucker | a376a32 | 2008-06-13 04:42:14 +1000 | [diff] [blame] | 1150 | - grunk@cvs.openbsd.org 2008/06/11 21:38:25 |
| 1151 | [ssh-keygen.c] |
| 1152 | ssh-keygen -lv -f /etc/ssh/ssh_host_rsa_key.pub |
| 1153 | would not display you the random art as intended, spotted by canacar@ |
Darren Tucker | 35c4553 | 2008-06-13 04:43:15 +1000 | [diff] [blame] | 1154 | - grunk@cvs.openbsd.org 2008/06/11 22:20:46 |
| 1155 | [ssh-keygen.c ssh-keygen.1] |
| 1156 | ssh-keygen would write fingerprints to STDOUT, and random art to STDERR, |
| 1157 | that is not how it was envisioned. |
| 1158 | Also correct manpage saying that -v is needed along with -l for it to work. |
| 1159 | spotted by naddy@ |
Darren Tucker | 014d76f | 2008-06-13 04:43:51 +1000 | [diff] [blame] | 1160 | - otto@cvs.openbsd.org 2008/06/11 23:02:22 |
| 1161 | [key.c] |
| 1162 | simpler way of computing the augmentations; ok grunk@ |
Darren Tucker | dcc1ab5 | 2008-06-13 04:44:25 +1000 | [diff] [blame] | 1163 | - grunk@cvs.openbsd.org 2008/06/11 23:03:56 |
| 1164 | [ssh_config.5] |
| 1165 | CheckHostIP set to ``fingerprint'' will display both hex and random art |
| 1166 | spotted by naddy@ |
Darren Tucker | d32b28a | 2008-06-13 04:45:50 +1000 | [diff] [blame] | 1167 | - grunk@cvs.openbsd.org 2008/06/11 23:51:57 |
| 1168 | [key.c] |
| 1169 | #define statements that are not atoms need braces around them, else they |
| 1170 | will cause trouble in some cases. |
| 1171 | Also do a computation of -1 once, and not in a loop several times. |
| 1172 | spotted by otto@ |
Darren Tucker | 78913e0 | 2008-06-13 04:47:34 +1000 | [diff] [blame] | 1173 | - dtucker@cvs.openbsd.org 2008/06/12 00:03:49 |
| 1174 | [dns.c canohost.c sshconnect.c] |
| 1175 | Do not pass "0" strings as ports to getaddrinfo because the lookups |
| 1176 | can slow things down and we never use the service info anyway. bz |
| 1177 | #859, patch from YOSHIFUJI Hideaki and John Devitofranceschi. ok |
| 1178 | deraadt@ djm@ |
| 1179 | djm belives that the reason for the "0" strings is to ensure that |
| 1180 | it's not possible to call getaddrinfo with both host and port being |
| 1181 | NULL. In the case of canohost.c host is a local array. In the |
| 1182 | case of sshconnect.c, it's checked for null immediately before use. |
| 1183 | In dns.c it ultimately comes from ssh.c:main() and is guaranteed to |
| 1184 | be non-null but it's not obvious, so I added a warning message in |
| 1185 | case it is ever passed a null. |
| 1186 | - grunk@cvs.openbsd.org 2008/06/12 00:13:55 |
| 1187 | [sshconnect.c] |
| 1188 | Make ssh print the random art also when ssh'ing to a host using IP only. |
| 1189 | spotted by naddy@, ok and help djm@ dtucker@ |
Darren Tucker | 267e28b | 2008-06-13 04:48:11 +1000 | [diff] [blame] | 1190 | - otto@cvs.openbsd.org 2008/06/12 00:13:13 |
| 1191 | [key.c] |
| 1192 | use an odd number of rows and columns and a separate start marker, looks |
| 1193 | better; ok grunk@ |
Darren Tucker | 2fb66ca | 2008-06-13 04:49:33 +1000 | [diff] [blame] | 1194 | - djm@cvs.openbsd.org 2008/06/12 03:40:52 |
| 1195 | [clientloop.h mux.c channels.c clientloop.c channels.h] |
| 1196 | Enable ~ escapes for multiplex slave sessions; give each channel |
| 1197 | its own escape state and hook the escape filters up to muxed |
| 1198 | channels. bz #1331 |
| 1199 | Mux slaves do not currently support the ~^Z and ~& escapes. |
| 1200 | NB. this change cranks the mux protocol version, so a new ssh |
| 1201 | mux client will not be able to connect to a running old ssh |
| 1202 | mux master. |
| 1203 | ok dtucker@ |
Darren Tucker | 9f407c4 | 2008-06-13 04:50:27 +1000 | [diff] [blame] | 1204 | - djm@cvs.openbsd.org 2008/06/12 04:06:00 |
| 1205 | [clientloop.h ssh.c clientloop.c] |
| 1206 | maintain an ordered queue of outstanding global requests that we |
| 1207 | expect replies to, similar to the per-channel confirmation queue. |
| 1208 | Use this queue to verify success or failure for remote forward |
| 1209 | establishment in a race free way. |
| 1210 | ok dtucker@ |
Darren Tucker | 4d5cd33 | 2008-06-13 04:51:14 +1000 | [diff] [blame] | 1211 | - djm@cvs.openbsd.org 2008/06/12 04:17:47 |
| 1212 | [clientloop.c] |
| 1213 | thall shalt not code past the eightieth column |
Darren Tucker | d6173c0 | 2008-06-13 04:52:53 +1000 | [diff] [blame] | 1214 | - djm@cvs.openbsd.org 2008/06/12 04:24:06 |
| 1215 | [ssh.c] |
| 1216 | thal shalt not code past the eightieth column |
Darren Tucker | e5d9829 | 2008-06-13 04:53:27 +1000 | [diff] [blame] | 1217 | - djm@cvs.openbsd.org 2008/06/12 05:15:41 |
| 1218 | [PROTOCOL] |
| 1219 | document tun@openssh.com forwarding method |
Darren Tucker | ba69c7a | 2008-06-13 04:54:05 +1000 | [diff] [blame] | 1220 | - djm@cvs.openbsd.org 2008/06/12 05:32:30 |
| 1221 | [mux.c] |
| 1222 | some more TODO for me |
Darren Tucker | 4b3b977 | 2008-06-13 04:55:10 +1000 | [diff] [blame] | 1223 | - grunk@cvs.openbsd.org 2008/06/12 05:42:46 |
| 1224 | [key.c] |
| 1225 | supply the key type (rsa1, rsa, dsa) as a caption in the frame of the |
| 1226 | random art. while there, stress the fact that the field base should at |
| 1227 | least be 8 characters for the pictures to make sense. |
| 1228 | comment and ok djm@ |
| 1229 | - grunk@cvs.openbsd.org 2008/06/12 06:32:59 |
| 1230 | [key.c] |
| 1231 | We already mark the start of the worm, now also mark the end of the worm |
| 1232 | in our random art drawings. |
| 1233 | ok djm@ |
Darren Tucker | 84c56f5 | 2008-06-13 04:55:46 +1000 | [diff] [blame] | 1234 | - djm@cvs.openbsd.org 2008/06/12 15:19:17 |
| 1235 | [clientloop.h channels.h clientloop.c channels.c mux.c] |
| 1236 | The multiplexing escape char handler commit last night introduced a |
| 1237 | small memory leak per session; plug it. |
Darren Tucker | f6b01b7 | 2008-06-13 04:56:37 +1000 | [diff] [blame] | 1238 | - dtucker@cvs.openbsd.org 2008/06/12 16:35:31 |
| 1239 | [ssh_config.5 ssh.c] |
| 1240 | keyword expansion for localcommand. ok djm@ |
Darren Tucker | f09e825 | 2008-06-13 05:18:03 +1000 | [diff] [blame] | 1241 | - jmc@cvs.openbsd.org 2008/06/12 19:10:09 |
| 1242 | [ssh_config.5 ssh-keygen.1] |
| 1243 | tweak the ascii art text; ok grunk |
Darren Tucker | 3fc464e | 2008-06-13 06:42:45 +1000 | [diff] [blame] | 1244 | - dtucker@cvs.openbsd.org 2008/06/12 20:38:28 |
| 1245 | [sshd.c sshconnect.c packet.h misc.c misc.h packet.c] |
| 1246 | Make keepalive timeouts apply while waiting for a packet, particularly |
| 1247 | during key renegotiation (bz #1363). With djm and Matt Day, ok djm@ |
Darren Tucker | a64ab33 | 2008-06-13 07:01:29 +1000 | [diff] [blame] | 1248 | - djm@cvs.openbsd.org 2008/06/12 20:47:04 |
| 1249 | [sftp-client.c] |
| 1250 | print extension revisions for extensions that we understand |
Darren Tucker | babc1d5 | 2008-06-13 08:56:01 +1000 | [diff] [blame] | 1251 | - djm@cvs.openbsd.org 2008/06/12 21:06:25 |
| 1252 | [clientloop.c] |
| 1253 | I was coalescing expected global request confirmation replies at |
| 1254 | the wrong end of the queue - fix; prompted by markus@ |
Darren Tucker | b68fb4a | 2008-06-13 08:57:27 +1000 | [diff] [blame] | 1255 | - grunk@cvs.openbsd.org 2008/06/12 21:14:46 |
| 1256 | [ssh-keygen.c] |
| 1257 | make ssh-keygen -lf show the key type just as ssh-add -l would do it |
| 1258 | ok djm@ markus@ |
Darren Tucker | 0f0ef0a | 2008-06-13 08:58:05 +1000 | [diff] [blame] | 1259 | - grunk@cvs.openbsd.org 2008/06/12 22:03:36 |
| 1260 | [key.c] |
| 1261 | add my copyright, ok djm@ |
Darren Tucker | ff4454d | 2008-06-13 10:21:51 +1000 | [diff] [blame] | 1262 | - ian@cvs.openbsd.org 2008/06/12 23:24:58 |
| 1263 | [sshconnect.c] |
| 1264 | tweak wording in message, ok deraadt@ jmc@ |
Darren Tucker | f8b7eb7 | 2008-06-13 10:22:54 +1000 | [diff] [blame] | 1265 | - dtucker@cvs.openbsd.org 2008/06/13 00:12:02 |
| 1266 | [sftp.h log.h] |
| 1267 | replace __dead with __attribute__((noreturn)), makes things |
| 1268 | a little easier to port. Also, add it to sigdie(). ok djm@ |
Darren Tucker | ca19bfe | 2008-06-13 10:24:03 +1000 | [diff] [blame] | 1269 | - djm@cvs.openbsd.org 2008/06/13 00:16:49 |
| 1270 | [mux.c] |
| 1271 | fall back to creating a new TCP connection on most multiplexing errors |
| 1272 | (socket connect fail, invalid version, refused permittion, corrupted |
| 1273 | messages, etc.); bz #1329 ok dtucker@ |
Darren Tucker | 1adfd36 | 2008-06-13 10:58:10 +1000 | [diff] [blame] | 1274 | - dtucker@cvs.openbsd.org 2008/06/13 00:47:53 |
| 1275 | [mux.c] |
| 1276 | upcast size_t to u_long to match format arg; ok djm@ |
Darren Tucker | c7e030f | 2008-06-13 10:58:50 +1000 | [diff] [blame] | 1277 | - dtucker@cvs.openbsd.org 2008/06/13 00:51:47 |
| 1278 | [mac.c] |
| 1279 | upcast another size_t to u_long to match format |
Darren Tucker | 7517b5b | 2008-06-13 14:48:59 +1000 | [diff] [blame] | 1280 | - dtucker@cvs.openbsd.org 2008/06/13 01:38:23 |
| 1281 | [misc.c] |
| 1282 | upcast uid to long with matching %ld, prevents warnings in portable |
Darren Tucker | 06db584 | 2008-06-13 14:51:28 +1000 | [diff] [blame] | 1283 | - djm@cvs.openbsd.org 2008/06/13 04:40:22 |
| 1284 | [auth2-pubkey.c auth-rhosts.c] |
| 1285 | refuse to read ~/.shosts or ~/.ssh/authorized_keys that are not |
| 1286 | regular files; report from Solar Designer via Colin Watson in bz#1471 |
| 1287 | ok dtucker@ deraadt |
Darren Tucker | 1cf65ae | 2008-06-13 05:09:18 +1000 | [diff] [blame] | 1288 | - (dtucker) [clientloop.c serverloop.c] channel_register_filter now |
| 1289 | takes 2 more args. with djm@ |
Darren Tucker | 61b0695 | 2008-06-13 10:28:57 +1000 | [diff] [blame] | 1290 | - (dtucker) [defines.h] Bug #1112: __dead is, well dead. Based on a patch |
| 1291 | from Todd Vierling. |
Darren Tucker | 2c1eb82 | 2008-06-13 11:13:13 +1000 | [diff] [blame] | 1292 | - (dtucker) [auth-sia.c] Bug #1241: support password expiry on Tru64 SIA |
| 1293 | systems. Patch from R. Scott Bailey. |
Darren Tucker | 2c91b28 | 2008-06-13 12:40:55 +1000 | [diff] [blame] | 1294 | - (dtucker) [umac.c] STORE_UINT32_REVERSED and endian_convert are never used |
| 1295 | on big endian machines, so ifdef them for little-endian only to prevent |
| 1296 | unused function warnings on big-endians. |
Darren Tucker | f387e59 | 2008-06-13 15:03:14 +1000 | [diff] [blame] | 1297 | - (dtucker) [openbsd-compat/setenv.c] Make offsets size_t to prevent |
| 1298 | compiler warnings on some platforms. Based on a discussion with otto@ |
Darren Tucker | 1199673 | 2008-06-13 04:32:00 +1000 | [diff] [blame] | 1299 | |
Damien Miller | 4401e45 | 2008-06-12 06:05:12 +1000 | [diff] [blame] | 1300 | 20080611 |
| 1301 | - (djm) [channels.c configure.ac] |
| 1302 | Do not set SO_REUSEADDR on wildcard X11 listeners (X11UseLocalhost=no) |
| 1303 | bz#1464; ok dtucker |
| 1304 | |
Darren Tucker | 7a3935d | 2008-06-10 22:59:10 +1000 | [diff] [blame] | 1305 | 20080610 |
| 1306 | - (dtucker) OpenBSD CVS Sync |
| 1307 | - djm@cvs.openbsd.org 2008/06/10 03:57:27 |
| 1308 | [servconf.c match.h sshd_config.5] |
| 1309 | support CIDR address matching in sshd_config "Match address" blocks, with |
| 1310 | full support for negation and fall-back to classic wildcard matching. |
| 1311 | For example: |
| 1312 | Match address 192.0.2.0/24,3ffe:ffff::/32,!10.* |
| 1313 | PasswordAuthentication yes |
| 1314 | addrmatch.c code mostly lifted from flowd's addr.c |
| 1315 | feedback and ok dtucker@ |
Darren Tucker | b06cc4a | 2008-06-10 22:59:53 +1000 | [diff] [blame] | 1316 | - djm@cvs.openbsd.org 2008/06/10 04:17:46 |
| 1317 | [sshd_config.5] |
| 1318 | better reference for pattern-list |
Darren Tucker | e7140f2 | 2008-06-10 23:01:51 +1000 | [diff] [blame] | 1319 | - dtucker@cvs.openbsd.org 2008/06/10 04:50:25 |
| 1320 | [sshd.c channels.h channels.c log.c servconf.c log.h servconf.h sshd.8] |
| 1321 | Add extended test mode (-T) and connection parameters for test mode (-C). |
| 1322 | -T causes sshd to write its effective configuration to stdout and exit. |
| 1323 | -C causes any relevant Match rules to be applied before output. The |
| 1324 | combination allows tesing of the parser and config files. ok deraadt djm |
Darren Tucker | 6a2a400 | 2008-06-10 23:03:04 +1000 | [diff] [blame] | 1325 | - jmc@cvs.openbsd.org 2008/06/10 07:12:00 |
| 1326 | [sshd_config.5] |
| 1327 | tweak previous; |
Darren Tucker | e7f3f75 | 2008-06-10 23:06:01 +1000 | [diff] [blame] | 1328 | - jmc@cvs.openbsd.org 2008/06/10 08:17:40 |
| 1329 | [sshd.8 sshd.c] |
| 1330 | - update usage() |
| 1331 | - fix SYNOPSIS, and sort options |
| 1332 | - some minor additional fixes |
Darren Tucker | d788b7c | 2008-06-10 23:15:54 +1000 | [diff] [blame] | 1333 | - dtucker@cvs.openbsd.org 2008/06/09 18:06:32 |
| 1334 | [regress/test-exec.sh] |
| 1335 | Don't generate putty keys if we're not going to use them. ok djm |
Darren Tucker | 10f9242 | 2008-06-10 23:16:46 +1000 | [diff] [blame] | 1336 | - dtucker@cvs.openbsd.org 2008/06/10 05:23:32 |
| 1337 | [regress/addrmatch.sh regress/Makefile] |
| 1338 | Regress test for Match CIDR rules. ok djm@ |
Darren Tucker | 5f34664 | 2008-06-11 01:38:52 +1000 | [diff] [blame] | 1339 | - dtucker@cvs.openbsd.org 2008/06/10 15:21:41 |
| 1340 | [test-exec.sh] |
| 1341 | Use a more portable construct for checking if we're running a putty test |
Darren Tucker | d8bafea | 2008-06-11 01:39:38 +1000 | [diff] [blame] | 1342 | - dtucker@cvs.openbsd.org 2008/06/10 15:28:49 |
| 1343 | [test-exec.sh] |
| 1344 | Add quotes |
Darren Tucker | c9807e8 | 2008-06-11 09:33:01 +1000 | [diff] [blame] | 1345 | - dtucker@cvs.openbsd.org 2008/06/10 18:21:24 |
| 1346 | [ssh_config.5] |
| 1347 | clarify that Host patterns are space-separated. ok deraadt |
Darren Tucker | 8901fa9 | 2008-06-11 09:34:01 +1000 | [diff] [blame] | 1348 | - djm@cvs.openbsd.org 2008/06/10 22:15:23 |
| 1349 | [PROTOCOL ssh.c serverloop.c] |
| 1350 | Add a no-more-sessions@openssh.com global request extension that the |
| 1351 | client sends when it knows that it will never request another session |
| 1352 | (i.e. when session multiplexing is disabled). This allows a server to |
| 1353 | disallow further session requests and terminate the session. |
| 1354 | Why would a non-multiplexing client ever issue additional session |
| 1355 | requests? It could have been attacked with something like SSH'jack: |
| 1356 | http://www.storm.net.nz/projects/7 |
| 1357 | feedback & ok markus |
Darren Tucker | 896ad5a | 2008-06-11 09:34:46 +1000 | [diff] [blame] | 1358 | - djm@cvs.openbsd.org 2008/06/10 23:06:19 |
| 1359 | [auth-options.c match.c servconf.c addrmatch.c sshd.8] |
| 1360 | support CIDR address matching in .ssh/authorized_keys from="..." stanzas |
| 1361 | ok and extensive testing dtucker@ |
Darren Tucker | 2a8b138 | 2008-06-11 09:35:37 +1000 | [diff] [blame] | 1362 | - dtucker@cvs.openbsd.org 2008/06/10 23:21:34 |
| 1363 | [bufaux.c] |
| 1364 | Use '\0' for a nul byte rather than unadorned 0. ok djm@ |
Darren Tucker | e045e0c | 2008-06-11 09:38:12 +1000 | [diff] [blame] | 1365 | - dtucker@cvs.openbsd.org 2008/06/10 23:13:43 |
| 1366 | [Makefile regress/key-options.sh] |
| 1367 | Add regress test for key options. ok djm@ |
Darren Tucker | 5d37690 | 2008-06-11 04:15:05 +1000 | [diff] [blame] | 1368 | - (dtucker) [openbsd-compat/fake-rfc2553.h] Add sin6_scope_id to sockaddr_in6 |
| 1369 | since the new CIDR code in addmatch.c references it. |
| 1370 | - (dtucker) [Makefile.in configure.ac regress/addrmatch.sh] Skip IPv6 |
| 1371 | specific tests on platforms that don't do IPv6. |
Darren Tucker | 34f49c6 | 2008-06-11 05:15:51 +1000 | [diff] [blame] | 1372 | - (dtucker) [Makefile.in] Define TEST_SSH_IPV6 in make's arguments as well |
| 1373 | as environment. |
Darren Tucker | b8e0500 | 2008-06-11 09:47:59 +1000 | [diff] [blame] | 1374 | - (dtucker) [Makefile.in] Move addrmatch.o to libssh.a where it's needed now. |
Darren Tucker | 7a3935d | 2008-06-10 22:59:10 +1000 | [diff] [blame] | 1375 | |
Darren Tucker | 422c34c | 2008-06-09 22:48:31 +1000 | [diff] [blame] | 1376 | 20080609 |
| 1377 | - (dtucker) OpenBSD CVS Sync |
| 1378 | - dtucker@cvs.openbsd.org 2008/06/08 17:04:41 |
| 1379 | [sftp-server.c] |
| 1380 | Add case for ENOSYS in errno_to_portable; ok deraadt |
Darren Tucker | 7b59889 | 2008-06-09 22:49:36 +1000 | [diff] [blame] | 1381 | - dtucker@cvs.openbsd.org 2008/06/08 20:15:29 |
| 1382 | [sftp.c sftp-client.c sftp-client.h] |
| 1383 | Have the sftp client store the statvfs replies in wire format, |
| 1384 | which prevents problems when the server's native sizes exceed the |
| 1385 | client's. |
| 1386 | Also extends the sizes of the remaining 32bit wire format to 64bit, |
| 1387 | they're specified as unsigned long in the standard. |
Darren Tucker | 3463aca | 2008-06-09 23:06:55 +1000 | [diff] [blame] | 1388 | - dtucker@cvs.openbsd.org 2008/06/09 13:02:39 |
Darren Tucker | 588fe0e | 2008-06-09 23:52:22 +1000 | [diff] [blame] | 1389 | [sftp-server.c] |
Darren Tucker | 3463aca | 2008-06-09 23:06:55 +1000 | [diff] [blame] | 1390 | Extend 32bit -> 64bit values for statvfs extension missed in previous |
| 1391 | commit. |
Darren Tucker | 588fe0e | 2008-06-09 23:52:22 +1000 | [diff] [blame] | 1392 | - dtucker@cvs.openbsd.org 2008/06/09 13:38:46 |
| 1393 | [PROTOCOL] |
| 1394 | Use a $OpenBSD tag so our scripts will sync changes. |
Darren Tucker | 422c34c | 2008-06-09 22:48:31 +1000 | [diff] [blame] | 1395 | |
Darren Tucker | 598eaa6 | 2008-06-09 03:32:29 +1000 | [diff] [blame] | 1396 | 20080608 |
| 1397 | - (dtucker) [configure.ac defines.h sftp-client.c sftp-server.c sftp.c |
| 1398 | openbsd-compat/Makefile.in openbsd-compat/openbsd-compat.h |
| 1399 | openbsd-compat/bsd-statvfs.{c,h}] Add a null implementation of statvfs and |
| 1400 | fstatvfs and remove #defines around statvfs code. ok djm@ |
Darren Tucker | 7700138 | 2008-06-09 06:17:53 +1000 | [diff] [blame] | 1401 | - (dtucker) [configure.ac defines.h sftp-client.c M sftp-server.c] Add a |
| 1402 | macro to convert fsid to unsigned long for platforms where fsid is a |
| 1403 | 2-member array. |
Darren Tucker | 598eaa6 | 2008-06-09 03:32:29 +1000 | [diff] [blame] | 1404 | |
Darren Tucker | ce38d82 | 2008-06-07 06:25:15 +1000 | [diff] [blame] | 1405 | 20080607 |
| 1406 | - (dtucker) [mux.c] Include paths.h inside ifdef HAVE_PATHS_H. |
Darren Tucker | 5b2e2ba | 2008-06-08 09:25:28 +1000 | [diff] [blame] | 1407 | - (dtucker) [configure.ac defines.h sftp-client.c sftp-server.c sftp.c] |
| 1408 | Do not enable statvfs extensions on platforms that do not have statvfs. |
Darren Tucker | 136e56f | 2008-06-08 12:49:30 +1000 | [diff] [blame] | 1409 | - (dtucker) OpenBSD CVS Sync |
| 1410 | - djm@cvs.openbsd.org 2008/05/19 06:14:02 |
| 1411 | [packet.c] unbreak protocol keepalive timeouts bz#1465; ok dtucker@ |
Darren Tucker | dd39264 | 2008-06-08 12:53:20 +1000 | [diff] [blame] | 1412 | - djm@cvs.openbsd.org 2008/05/19 15:45:07 |
| 1413 | [sshtty.c ttymodes.c sshpty.h] |
| 1414 | Fix sending tty modes when stdin is not a tty (bz#1199). Previously |
| 1415 | we would send the modes corresponding to a zeroed struct termios, |
| 1416 | whereas we should have been sending an empty list of modes. |
| 1417 | Based on patch from daniel.ritz AT alcatel.ch; ok dtucker@ markus@ |
Darren Tucker | 0f7e910 | 2008-06-08 12:54:29 +1000 | [diff] [blame] | 1418 | - djm@cvs.openbsd.org 2008/05/19 15:46:31 |
| 1419 | [ssh-keygen.c] |
| 1420 | support -l (print fingerprint) in combination with -F (find host) to |
| 1421 | search for a host in ~/.ssh/known_hosts and display its fingerprint; |
| 1422 | ok markus@ |
Darren Tucker | df189fb | 2008-06-08 12:55:32 +1000 | [diff] [blame] | 1423 | - djm@cvs.openbsd.org 2008/05/19 20:53:52 |
| 1424 | [clientloop.c] |
| 1425 | unbreak tree by committing this bit that I missed from: |
| 1426 | Fix sending tty modes when stdin is not a tty (bz#1199). Previously |
| 1427 | we would send the modes corresponding to a zeroed struct termios, |
| 1428 | whereas we should have been sending an empty list of modes. |
| 1429 | Based on patch from daniel.ritz AT alcatel.ch; ok dtucker@ markus@ |
Darren Tucker | ce38d82 | 2008-06-07 06:25:15 +1000 | [diff] [blame] | 1430 | |
Damien Miller | 58ea61b | 2008-06-04 10:54:00 +1000 | [diff] [blame] | 1431 | 20080604 |
| 1432 | - (djm) [openbsd-compat/bsd-arc4random.c] Fix math bug that caused bias |
| 1433 | in arc4random_uniform with upper_bound in (2^30,2*31). Note that |
| 1434 | OpenSSH did not make requests with upper bounds in this range. |
| 1435 | |
Damien Miller | a7058ec | 2008-05-20 08:57:06 +1000 | [diff] [blame] | 1436 | 20080519 |
| 1437 | - (djm) [configure.ac mux.c sftp.c openbsd-compat/Makefile.in] |
| 1438 | [openbsd-compat/fmt_scaled.c openbsd-compat/openbsd-compat.h] |
| 1439 | Fix compilation on Linux, including pulling in fmt_scaled(3) |
| 1440 | implementation from OpenBSD's libutil. |
| 1441 | |
Damien Miller | 797e3d1 | 2008-05-19 14:27:42 +1000 | [diff] [blame] | 1442 | 20080518 |
| 1443 | - (djm) OpenBSD CVS Sync |
| 1444 | - djm@cvs.openbsd.org 2008/04/04 05:14:38 |
| 1445 | [sshd_config.5] |
| 1446 | ChrootDirectory is supported in Match blocks (in fact, it is most useful |
| 1447 | there). Spotted by Minstrel AT minstrel.org.uk |
Damien Miller | 56f41dd | 2008-05-19 14:28:19 +1000 | [diff] [blame] | 1448 | - djm@cvs.openbsd.org 2008/04/04 06:44:26 |
| 1449 | [sshd_config.5] |
| 1450 | oops, some unrelated stuff crept into that commit - backout. |
| 1451 | spotted by jmc@ |
Damien Miller | 25434de | 2008-05-19 14:29:08 +1000 | [diff] [blame] | 1452 | - djm@cvs.openbsd.org 2008/04/05 02:46:02 |
| 1453 | [sshd_config.5] |
| 1454 | HostbasedAuthentication is supported under Match too |
Damien Miller | a4be7c2 | 2008-05-19 14:47:37 +1000 | [diff] [blame] | 1455 | - (djm) [openbsd-compat/bsd-arc4random.c openbsd-compat/openbsd-compat.c] |
| 1456 | [configure.ac] Implement arc4random_buf(), import implementation of |
| 1457 | arc4random_uniform() from OpenBSD |
Damien Miller | caaed01 | 2008-05-19 15:26:54 +1000 | [diff] [blame] | 1458 | - (djm) [openbsd-compat/bsd-arc4random.c] Warning fixes |
Damien Miller | 9417831 | 2008-05-19 15:28:35 +1000 | [diff] [blame] | 1459 | - (djm) [openbsd-compat/port-tun.c] needs sys/queue.h |
Damien Miller | 354c48c | 2008-05-19 14:50:00 +1000 | [diff] [blame] | 1460 | - (djm) OpenBSD CVS Sync |
| 1461 | - djm@cvs.openbsd.org 2008/04/13 00:22:17 |
| 1462 | [dh.c sshd.c] |
| 1463 | Use arc4random_buf() when requesting more than a single word of output |
| 1464 | Use arc4random_uniform() when the desired random number upper bound |
| 1465 | is not a power of two |
| 1466 | ok deraadt@ millert@ |
Damien Miller | d671e5a | 2008-05-19 14:53:33 +1000 | [diff] [blame] | 1467 | - djm@cvs.openbsd.org 2008/04/18 12:32:11 |
| 1468 | [sftp-client.c sftp-client.h sftp-server.c sftp.1 sftp.c sftp.h] |
| 1469 | introduce sftp extension methods statvfs@openssh.com and |
| 1470 | fstatvfs@openssh.com that implement statvfs(2)-like operations, |
| 1471 | based on a patch from miklos AT szeredi.hu (bz#1399) |
| 1472 | also add a "df" command to the sftp client that uses the |
| 1473 | statvfs@openssh.com to produce a df(1)-like display of filesystem |
| 1474 | space and inode utilisation |
| 1475 | ok markus@ |
Damien Miller | 64058cb | 2008-05-19 14:54:25 +1000 | [diff] [blame] | 1476 | - jmc@cvs.openbsd.org 2008/04/18 17:15:47 |
| 1477 | [sftp.1] |
| 1478 | macro fixage; |
Damien Miller | ff0dd88 | 2008-05-19 14:55:02 +1000 | [diff] [blame] | 1479 | - djm@cvs.openbsd.org 2008/04/18 22:01:33 |
| 1480 | [session.c] |
| 1481 | remove unneccessary parentheses |
Damien Miller | 0b4c165 | 2008-05-19 14:55:29 +1000 | [diff] [blame] | 1482 | - otto@cvs.openbsd.org 2008/04/29 11:20:31 |
| 1483 | [monitor_mm.h] |
| 1484 | garbage collect two unused fields in struct mm_master; ok markus@ |
Damien Miller | bacb7fb | 2008-05-19 14:56:33 +1000 | [diff] [blame] | 1485 | - djm@cvs.openbsd.org 2008/04/30 10:14:03 |
| 1486 | [ssh-keyscan.1 ssh-keyscan.c] |
| 1487 | default to rsa (protocol 2) keys, instead of rsa1 keys; spotted by |
| 1488 | larsnooden AT openoffice.org |
Damien Miller | 4f755cd | 2008-05-19 14:57:41 +1000 | [diff] [blame] | 1489 | - pyr@cvs.openbsd.org 2008/05/07 05:49:37 |
| 1490 | [servconf.c servconf.h session.c sshd_config.5] |
| 1491 | Enable the AllowAgentForwarding option in sshd_config (global and match |
| 1492 | context), to specify if agents should be permitted on the server. |
| 1493 | As the man page states: |
| 1494 | ``Note that disabling Agent forwarding does not improve security |
| 1495 | unless users are also denied shell access, as they can always install |
| 1496 | their own forwarders.'' |
| 1497 | ok djm@, ok and a mild frown markus@ |
Damien Miller | ba3a659 | 2008-05-19 14:58:22 +1000 | [diff] [blame] | 1498 | - pyr@cvs.openbsd.org 2008/05/07 06:43:35 |
| 1499 | [sshd_config] |
| 1500 | push the sshd_config bits in, spotted by ajacoutot@ |
Damien Miller | e989019 | 2008-05-19 14:59:02 +1000 | [diff] [blame] | 1501 | - jmc@cvs.openbsd.org 2008/05/07 08:00:14 |
| 1502 | [sshd_config.5] |
| 1503 | sort; |
Damien Miller | db255ca | 2008-05-19 14:59:37 +1000 | [diff] [blame] | 1504 | - markus@cvs.openbsd.org 2008/05/08 06:59:01 |
| 1505 | [bufaux.c buffer.h channels.c packet.c packet.h] |
| 1506 | avoid extra malloc/copy/free when receiving data over the net; |
| 1507 | ~10% speedup for localhost-scp; ok djm@ |
Damien Miller | b84886b | 2008-05-19 15:05:07 +1000 | [diff] [blame] | 1508 | - djm@cvs.openbsd.org 2008/05/08 12:02:23 |
| 1509 | [auth-options.c auth1.c channels.c channels.h clientloop.c gss-serv.c] |
| 1510 | [monitor.c monitor_wrap.c nchan.c servconf.c serverloop.c session.c] |
| 1511 | [ssh.c sshd.c] |
| 1512 | Implement a channel success/failure status confirmation callback |
| 1513 | mechanism. Each channel maintains a queue of callbacks, which will |
| 1514 | be drained in order (RFC4253 guarantees confirm messages are not |
| 1515 | reordered within an channel). |
| 1516 | Also includes a abandonment callback to clean up if a channel is |
| 1517 | closed without sending confirmation messages. This probably |
| 1518 | shouldn't happen in compliant implementations, but it could be |
| 1519 | abused to leak memory. |
| 1520 | ok markus@ (as part of a larger diff) |
Damien Miller | 7207f64 | 2008-05-19 15:34:50 +1000 | [diff] [blame] | 1521 | - djm@cvs.openbsd.org 2008/05/08 12:21:16 |
| 1522 | [monitor.c monitor_wrap.c session.h servconf.c servconf.h session.c] |
| 1523 | [sshd_config sshd_config.5] |
| 1524 | Make the maximum number of sessions run-time controllable via |
| 1525 | a sshd_config MaxSessions knob. This is useful for disabling |
| 1526 | login/shell/subsystem access while leaving port-forwarding working |
| 1527 | (MaxSessions 0), disabling connection multiplexing (MaxSessions 1) or |
| 1528 | simply increasing the number of allows multiplexed sessions. |
| 1529 | Because some bozos are sure to configure MaxSessions in excess of the |
| 1530 | number of available file descriptors in sshd (which, at peak, might be |
| 1531 | as many as 9*MaxSessions), audit sshd to ensure that it doesn't leak fds |
| 1532 | on error paths, and make it fail gracefully on out-of-fd conditions - |
| 1533 | sending channel errors instead of than exiting with fatal(). |
| 1534 | bz#1090; MaxSessions config bits and manpage from junyer AT gmail.com |
| 1535 | ok markus@ |
Damien Miller | 5771ed7 | 2008-05-19 15:35:33 +1000 | [diff] [blame] | 1536 | - djm@cvs.openbsd.org 2008/05/08 13:06:11 |
| 1537 | [clientloop.c clientloop.h ssh.c] |
| 1538 | Use new channel status confirmation callback system to properly deal |
| 1539 | with "important" channel requests that fail, in particular command exec, |
| 1540 | shell and subsystem requests. Previously we would optimistically assume |
| 1541 | that the requests would always succeed, which could cause hangs if they |
| 1542 | did not (e.g. when the server runs out of fds) or were unimplemented by |
| 1543 | the server (bz #1384) |
| 1544 | Also, properly report failing multiplex channel requests via the mux |
| 1545 | client stderr (subject to LogLevel in the mux master) - better than |
| 1546 | silently failing. |
| 1547 | most bits ok markus@ (as part of a larger diff) |
Damien Miller | bd74025 | 2008-05-19 15:37:09 +1000 | [diff] [blame] | 1548 | - djm@cvs.openbsd.org 2008/05/09 04:55:56 |
| 1549 | [channels.c channels.h clientloop.c serverloop.c] |
| 1550 | Try additional addresses when connecting to a port forward destination |
| 1551 | whose DNS name resolves to more than one address. The previous behaviour |
| 1552 | was to try the first address and give up. |
| 1553 | Reported by stig AT venaas.com in bz#343 |
| 1554 | great feedback and ok markus@ |
Damien Miller | b1cbfa2 | 2008-05-19 16:00:08 +1000 | [diff] [blame] | 1555 | - djm@cvs.openbsd.org 2008/05/09 14:18:44 |
| 1556 | [clientloop.c clientloop.h ssh.c mux.c] |
| 1557 | tidy up session multiplexing code, moving it into its own file and |
| 1558 | making the function names more consistent - making ssh.c and |
| 1559 | clientloop.c a fair bit more readable. |
| 1560 | ok markus@ |
Damien Miller | b3da593 | 2008-05-19 16:02:37 +1000 | [diff] [blame] | 1561 | - djm@cvs.openbsd.org 2008/05/09 14:26:08 |
| 1562 | [ssh.c] |
| 1563 | dingo stole my diff hunk |
Damien Miller | 2ff1ca5 | 2008-05-19 16:04:56 +1000 | [diff] [blame] | 1564 | - markus@cvs.openbsd.org 2008/05/09 16:16:06 |
| 1565 | [session.c] |
| 1566 | re-add the USE_PIPES code and enable it. |
| 1567 | without pipes shutdown-read from the sshd does not trigger |
| 1568 | a SIGPIPE when the forked program does a write. |
| 1569 | ok djm@ |
| 1570 | (Id sync only, USE_PIPES never left portable OpenSSH) |
Damien Miller | d654dd2 | 2008-05-19 16:05:41 +1000 | [diff] [blame] | 1571 | - markus@cvs.openbsd.org 2008/05/09 16:17:51 |
| 1572 | [channels.c] |
| 1573 | error-fd race: don't enable the error fd in the select bitmask |
| 1574 | for channels with both in- and output closed, since the channel |
| 1575 | will go away before we call select(); |
| 1576 | report, lots of debugging help and ok djm@ |
Damien Miller | bab9bd4 | 2008-05-19 16:06:47 +1000 | [diff] [blame] | 1577 | - markus@cvs.openbsd.org 2008/05/09 16:21:13 |
| 1578 | [channels.h clientloop.c nchan.c serverloop.c] |
| 1579 | unbreak |
| 1580 | ssh -2 localhost od /bin/ls | true |
| 1581 | ignoring SIGPIPE by adding a new channel message (EOW) that signals |
| 1582 | the peer that we're not interested in any data it might send. |
| 1583 | fixes bz #85; discussion, debugging and ok djm@ |
Damien Miller | 0f30c87 | 2008-05-19 16:07:45 +1000 | [diff] [blame] | 1584 | - pvalchev@cvs.openbsd.org 2008/05/12 20:52:20 |
| 1585 | [umac.c] |
| 1586 | Ensure nh_result lies on a 64-bit boundary (fixes warnings observed |
| 1587 | on Itanium on Linux); from Dale Talcott (bug #1462); ok djm@ |
Damien Miller | 5159bdd | 2008-05-19 16:08:20 +1000 | [diff] [blame] | 1588 | - djm@cvs.openbsd.org 2008/05/15 23:52:24 |
| 1589 | [nchan2.ms] |
| 1590 | document eow message in ssh protocol 2 channel state machine; |
| 1591 | feedback and ok markus@ |
Damien Miller | a7e0d5a | 2008-05-19 16:08:41 +1000 | [diff] [blame] | 1592 | - djm@cvs.openbsd.org 2008/05/18 21:29:05 |
| 1593 | [sftp-server.c] |
| 1594 | comment extension announcement |
Damien Miller | 58a8114 | 2008-05-19 16:11:56 +1000 | [diff] [blame] | 1595 | - djm@cvs.openbsd.org 2008/05/16 08:30:42 |
| 1596 | [PROTOCOL] |
| 1597 | document our protocol extensions and deviations; ok markus@ |
| 1598 | - djm@cvs.openbsd.org 2008/05/17 01:31:56 |
| 1599 | [PROTOCOL] |
| 1600 | grammar and correctness fixes from stevesk@ |
Damien Miller | 797e3d1 | 2008-05-19 14:27:42 +1000 | [diff] [blame] | 1601 | |
Damien Miller | 5f5cd74 | 2008-04-03 08:43:57 +1100 | [diff] [blame] | 1602 | 20080403 |
Damien Miller | 55754fb | 2008-04-04 16:16:35 +1100 | [diff] [blame] | 1603 | - (djm) [openbsd-compat/bsd-poll.c] Include stdlib.h to avoid compile- |
| 1604 | time warnings on LynxOS. Patch from ops AT iki.fi |
Damien Miller | c575022 | 2008-05-16 10:01:54 +1000 | [diff] [blame] | 1605 | - (djm) Force string arguments to replacement setproctitle() though |
| 1606 | strnvis first. Ok dtucker@ |
Damien Miller | 55754fb | 2008-04-04 16:16:35 +1100 | [diff] [blame] | 1607 | |
| 1608 | 20080403 |
Damien Miller | 5f5cd74 | 2008-04-03 08:43:57 +1100 | [diff] [blame] | 1609 | - (djm) OpenBSD CVS sync: |
| 1610 | - markus@cvs.openbsd.org 2008/04/02 15:36:51 |
| 1611 | [channels.c] |
| 1612 | avoid possible hijacking of x11-forwarded connections (back out 1.183) |
| 1613 | CVE-2008-1483; ok djm@ |
Damien Miller | 13ba9c2 | 2008-04-03 20:52:51 +1100 | [diff] [blame] | 1614 | - jmc@cvs.openbsd.org 2008/03/27 22:37:57 |
| 1615 | [sshd.8] |
| 1616 | remove trailing whitespace; |
Damien Miller | a68d31b | 2008-04-03 20:53:08 +1100 | [diff] [blame] | 1617 | - djm@cvs.openbsd.org 2008/04/03 09:50:14 |
| 1618 | [version.h] |
| 1619 | openssh-5.0 |
Damien Miller | 5a4b646 | 2008-04-03 20:55:44 +1100 | [diff] [blame] | 1620 | - (djm) [contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
| 1621 | [contrib/suse/openssh.spec] Crank version numbers in RPM spec files |
Damien Miller | ffa8302 | 2008-04-03 20:56:38 +1100 | [diff] [blame] | 1622 | - (djm) [README] Update link to release notes |
Damien Miller | 79a1bc9 | 2008-04-03 20:57:05 +1100 | [diff] [blame] | 1623 | - (djm) Release 5.0p1 |