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