]>
Commit | Line | Data |
---|---|---|
19b6c4d5 | 1 | 20091011 |
2 | - (dtucker) [configure.ac sftp-client.c] Remove the gyrations required for | |
3 | dirent d_type and DTTOIF as we've switched OpenBSD to the more portable | |
4 | lstat. | |
21af5fc4 | 5 | - (dtucker) OpenBSD CVS Sync |
6 | - markus@cvs.openbsd.org 2009/10/08 14:03:41 | |
7 | [sshd_config readconf.c ssh_config.5 servconf.c sshd_config.5] | |
8 | disable protocol 1 by default (after a transition period of about 10 years) | |
9 | ok deraadt | |
0dba6d86 | 10 | - jmc@cvs.openbsd.org 2009/10/08 20:42:12 |
11 | [sshd_config.5 ssh_config.5 sshd.8 ssh.1] | |
12 | some tweaks now that protocol 1 is not offered by default; ok markus | |
711fb093 | 13 | - dtucker@cvs.openbsd.org 2009/10/11 10:41:26 |
14 | [sftp-client.c] | |
15 | d_type isn't portable so use lstat to get dirent modes. Suggested by and | |
16 | "looks sane" deraadt@ | |
19b6c4d5 | 17 | |
3496b8d4 | 18 | 20091007 |
19 | - (dtucker) OpenBSD CVS Sync | |
20 | - djm@cvs.openbsd.org 2009/08/12 00:13:00 | |
21 | [sftp.c sftp.1] | |
22 | support most of scp(1)'s commandline arguments in sftp(1), as a first | |
23 | step towards making sftp(1) a drop-in replacement for scp(1). | |
24 | One conflicting option (-P) has not been changed, pending further | |
25 | discussion. | |
26 | Patch from carlosvsilvapt@gmail.com as part of his work in the | |
27 | Google Summer of Code | |
b68241c3 | 28 | - jmc@cvs.openbsd.org 2009/08/12 06:31:42 |
29 | [sftp.1] | |
30 | sort options; | |
97658f13 | 31 | - djm@cvs.openbsd.org 2009/08/13 01:11:19 |
32 | [sftp.1 sftp.c] | |
33 | Swizzle options: "-P sftp_server_path" moves to "-D sftp_server_path", | |
34 | add "-P port" to match scp(1). Fortunately, the -P option is only really | |
35 | used by our regression scripts. | |
36 | part of larger patch from carlosvsilvapt@gmail.com for his Google Summer | |
37 | of Code work; ok deraadt markus | |
5aa0f160 | 38 | - jmc@cvs.openbsd.org 2009/08/13 13:39:54 |
39 | [sftp.1 sftp.c] | |
40 | sync synopsis and usage(); | |
e746280c | 41 | - djm@cvs.openbsd.org 2009/08/14 18:17:49 |
42 | [sftp-client.c] | |
43 | make the "get_handle: ..." error messages vaguely useful by allowing | |
44 | callers to specify their own error message strings. | |
5d799258 | 45 | - fgsch@cvs.openbsd.org 2009/08/15 18:56:34 |
46 | [auth.h] | |
47 | remove unused define. markus@ ok. | |
48 | (Id sync only, Portable still uses this.) | |
7b3a24aa | 49 | - dtucker@cvs.openbsd.org 2009/08/16 23:29:26 |
50 | [sshd_config.5] | |
51 | Add PubkeyAuthentication to the list allowed in a Match block (bz #1577) | |
d141f964 | 52 | - djm@cvs.openbsd.org 2009/08/18 18:36:21 |
53 | [sftp-client.h sftp.1 sftp-client.c sftp.c] | |
54 | recursive transfer support for get/put and on the commandline | |
55 | work mostly by carlosvsilvapt@gmail.com for the Google Summer of Code | |
56 | with some tweaks by me; "go for it" deraadt@ | |
e83f55f9 | 57 | - djm@cvs.openbsd.org 2009/08/18 21:15:59 |
58 | [sftp.1] | |
59 | fix "get" command usage, spotted by jmc@ | |
3829cbca | 60 | - jmc@cvs.openbsd.org 2009/08/19 04:56:03 |
61 | [sftp.1] | |
62 | ether -> either; | |
2e2c33ad | 63 | - dtucker@cvs.openbsd.org 2009/08/20 23:54:28 |
64 | [mux.c] | |
65 | subsystem_flag is defined in ssh.c so it's extern; ok djm | |
99c5cf8e | 66 | - djm@cvs.openbsd.org 2009/08/27 17:28:52 |
67 | [sftp-server.c] | |
68 | allow setting an explicit umask on the commandline to override whatever | |
69 | default the user has. bz#1229; ok dtucker@ deraadt@ markus@ | |
bf3290be | 70 | - djm@cvs.openbsd.org 2009/08/27 17:33:49 |
71 | [ssh-keygen.c] | |
72 | force use of correct hash function for random-art signature display | |
73 | as it was inheriting the wrong one when bubblebabble signatures were | |
74 | activated; bz#1611 report and patch from fwojcik+openssh AT besh.com; | |
75 | ok markus@ | |
62b92bdc | 76 | - djm@cvs.openbsd.org 2009/08/27 17:43:00 |
77 | [sftp-server.8] | |
78 | allow setting an explicit umask on the commandline to override whatever | |
79 | default the user has. bz#1229; ok dtucker@ deraadt@ markus@ | |
b4741f94 | 80 | - djm@cvs.openbsd.org 2009/08/27 17:44:52 |
81 | [authfd.c ssh-add.c authfd.h] | |
82 | Do not fall back to adding keys without contraints (ssh-add -c / -t ...) | |
83 | when the agent refuses the constrained add request. This was a useful | |
84 | migration measure back in 2002 when constraints were new, but just | |
85 | adds risk now. | |
86 | bz #1612, report and patch from dkg AT fifthhorseman.net; ok markus@ | |
57a6b5dd | 87 | - djm@cvs.openbsd.org 2009/08/31 20:56:02 |
88 | [sftp-server.c] | |
89 | check correct variable for error message, spotted by martynas@ | |
b7177174 | 90 | - djm@cvs.openbsd.org 2009/08/31 21:01:29 |
91 | [sftp-server.8] | |
92 | document -e and -h; prodded by jmc@ | |
5561856d | 93 | - djm@cvs.openbsd.org 2009/09/01 14:43:17 |
94 | [ssh-agent.c] | |
95 | fix a race condition in ssh-agent that could result in a wedged or | |
96 | spinning agent: don't read off the end of the allocated fd_sets, and | |
97 | don't issue blocking read/write on agent sockets - just fall back to | |
98 | select() on retriable read/write errors. bz#1633 reported and tested | |
99 | by "noodle10000 AT googlemail.com"; ok dtucker@ markus@ | |
fd8b10fa | 100 | - grunk@cvs.openbsd.org 2009/10/01 11:37:33 |
101 | [dh.c] | |
102 | fix a cast | |
103 | ok djm@ markus@ | |
45bb6142 | 104 | - djm@cvs.openbsd.org 2009/10/06 04:46:40 |
105 | [session.c] | |
106 | bz#1596: fflush(NULL) before exec() to ensure that everying (motd | |
107 | in particular) has made it out before the streams go away. | |
1aeac41e | 108 | - djm@cvs.openbsd.org 2008/12/07 22:17:48 |
109 | [regress/addrmatch.sh] | |
110 | match string "passwordauthentication" only at start of line, not anywhere | |
111 | in sshd -T output | |
cbc2c3e5 | 112 | - dtucker@cvs.openbsd.org 2009/05/05 07:51:36 |
113 | [regress/multiplex.sh] | |
114 | Always specify ssh_config for multiplex tests: prevents breakage caused | |
115 | by options in ~/.ssh/config. From Dan Peterson. | |
95744748 | 116 | - djm@cvs.openbsd.org 2009/08/13 00:57:17 |
117 | [regress/Makefile] | |
118 | regression test for port number parsing. written as part of the a2port | |
119 | change that went into 5.2 but I forgot to commit it at the time... | |
6c8ebe98 | 120 | - djm@cvs.openbsd.org 2009/08/13 01:11:55 |
90fc667e | 121 | [regress/sftp-batch.sh regress/sftp-badcmds.sh regress/sftp.sh |
122 | regress/sftp-cmds.sh regres/sftp-glob.sh] | |
6c8ebe98 | 123 | date: 2009/08/13 01:11:19; author: djm; state: Exp; lines: +10 -7 |
124 | Swizzle options: "-P sftp_server_path" moves to "-D sftp_server_path", | |
125 | add "-P port" to match scp(1). Fortunately, the -P option is only really | |
126 | used by our regression scripts. | |
127 | part of larger patch from carlosvsilvapt@gmail.com for his Google Summer | |
128 | of Code work; ok deraadt markus | |
c16b5840 | 129 | - djm@cvs.openbsd.org 2009/08/20 18:43:07 |
90fc667e | 130 | [regress/ssh-com-sftp.sh] |
c16b5840 | 131 | fix one sftp -D ... => sftp -P ... conversion that I missed; from Carlos |
132 | Silva for Google Summer of Code | |
90fc667e | 133 | - dtucker@cvs.openbsd.org 2009/10/06 23:51:49 |
134 | [regress/ssh2putty.sh] | |
135 | Add OpenBSD tag to make syncs easier | |
deed7126 | 136 | - (dtucker) [regress/portnum.sh] Import new test. |
c7e0fa79 | 137 | - (dtucker) [configure.ac sftp-client.c] DTOTIF is in fs/ffs/dir.h on at |
4b48f754 | 138 | least dragonflybsd. |
c7e0fa79 | 139 | - (dtucker) d_type is not mandated by POSIX, so add fallback code using |
140 | stat(), needed on at least cygwin. | |
3496b8d4 | 141 | |
2391a73c | 142 | 20091002 |
143 | - (djm) [Makefile.in] Mention readconf.o in ssh-keysign's make deps. | |
144 | spotted by des AT des.no | |
145 | ||
018fda87 | 146 | 20090926 |
147 | - (djm) [contrib/caldera/openssh.spec contrib/redhat/openssh.spec] | |
148 | [contrib/suse/openssh.spec] Update for release | |
149 | - (djm) [README] update relnotes URL | |
150 | - (djm) [packet.c] Restore EWOULDBLOCK handling that got lost somewhere | |
151 | - (djm) Release 5.3p1 | |
152 | ||
a37250f4 | 153 | 20090911 |
154 | - (dtucker) [configure.ac] Change the -lresolv check so it works on Mac OS X | |
155 | 10.6 (which doesn't have BIND8_COMPAT and thus uses res_9_query). Patch | |
156 | from jbasney at ncsa uiuc edu. | |
157 | ||
bc33f6d6 | 158 | 20090908 |
159 | - (djm) [serverloop.c] Fix test for server-assigned remote forwarding port | |
160 | (-R 0:...); bz#1578, spotted and fix by gavin AT emf.net; ok dtucker@ | |
161 | ||
3acad382 | 162 | 20090901 |
163 | - (dtucker) [configure.ac] Bug #1639: use AC_PATH_PROG to search the path for | |
164 | krb5-config if it's not in the location specified by --with-kerberos5. | |
165 | Patch from jchadima at redhat. | |
166 | ||
84c645ec | 167 | 20090829 |
168 | - (dtucker) [README.platform] Add text about development packages, based on | |
169 | text from Chris Pepper in bug #1631. | |
170 | ||
7a51ce05 | 171 | 20090828 |
172 | - dtucker [auth-sia.c] Roll back the change for bug #1241 as it apparently | |
173 | causes problems in some Tru64 configurations. | |
d108641a | 174 | - (djm) [sshd_config.5] downgrade mention of login.conf to be an example |
175 | and mention PAM as another provider for ChallengeResponseAuthentication; | |
176 | bz#1408; ok dtucker@ | |
6ecb350f | 177 | - (djm) [sftp-server.c] bz#1535: accept ENOSYS as a fallback error when |
178 | attempting atomic rename(); ok dtucker@ | |
36141cb8 | 179 | - (djm) [Makefile.in] bz#1505: Solaris make(1) doesn't accept make variables |
180 | in argv, so pass them in the environment; ok dtucker@ | |
5e934f78 | 181 | - (dtucker) [channels.c configure.ac] Bug #1528: skip the tcgetattr call on |
182 | the pty master on Solaris, since it never succeeds and can hang if large | |
183 | amounts of data is sent to the slave (eg a copy-paste). Based on a patch | |
184 | originally from Doke Scott, ok djm@ | |
e7ac4a90 | 185 | - (dtucker) [clientloop.c configure.ac defines.h] Make the client's IO buffer |
186 | size a compile-time option and set it to 64k on Cygwin, since Corinna | |
187 | reports that it makes a significant difference to performance. ok djm@ | |
00789f24 | 188 | - (dtucker) [configure.ac] Fix the syntax of the Solaris tcgetattr entry. |
7a51ce05 | 189 | |
bf87c429 | 190 | 20090820 |
191 | - (dtucker) [includes.h] Bug #1634: do not include system glob.h if we're not | |
192 | using it since the type conflicts can cause problems on FreeBSD. Patch | |
193 | from Jonathan Chen. | |
406dc01a | 194 | - (dtucker) [session.c openbsd-compat/port-aix.h] Bugs #1249 and #1567: move |
195 | the setpcred call on AIX to immediately before the permanently_set_uid(). | |
196 | Ensures that we still have privileges when we call chroot and | |
197 | pam_open_sesson. Based on a patch from David Leonard. | |
bf87c429 | 198 | |
8295689f | 199 | 20090817 |
200 | - (dtucker) [configure.ac] Check for headers before libraries for openssl an | |
201 | zlib, which should make the errors slightly more meaningful on platforms | |
202 | where there's separate "-devel" packages for those. | |
e339fa25 | 203 | - (dtucker) [sshlogin.c openbsd-compat/port-aix.{c,h}] Bug #1595: make |
204 | PrintLastLog work on AIX. Based in part on a patch from Miguel Sanders. | |
8295689f | 205 | |
852de6fd | 206 | 20090729 |
207 | - (tim) [contrib/cygwin/ssh-user-config] Change script to call correct error | |
208 | function. Patch from Corinna Vinschen. | |
209 | ||
14a260e8 | 210 | 20090713 |
211 | - (dtucker) [openbsd-compat/getrrsetbyname.c] Reduce answer buffer size so it | |
212 | fits into 16 bits to work around a bug in glibc's resolver where it masks | |
213 | off the buffer size at 16 bits. Patch from Hauke Lampe, ok djm jakob. | |
214 | ||
0a008a4d | 215 | 20090712 |
216 | - (dtucker) [configure.ac] Include sys/param.h for the sys/mount.h test, | |
217 | prevents configure complaining on older BSDs. | |
74973c95 | 218 | - (dtucker [contrib/cygwin/ssh-{host,user}-config] Add license text. Patch |
219 | from Corinna Vinschen. | |
f12c178c | 220 | - (dtucker) [auth-pam.c] Bug #1534: move the deletion of PAM credentials on |
1d5c49e0 | 221 | logout to after the session close. Patch from Anicka Bernathova, |
222 | originally from Andreas Schwab via Novelll ok djm. | |
0a008a4d | 223 | |
78576c54 | 224 | 20090707 |
225 | - (dtucker) [contrib/cygwin/ssh-host-config] better support for automated | |
226 | scripts and fix usage of eval. Patch from Corinna Vinschen. | |
227 | ||
228 | 20090705 | |
dc11a83a | 229 | - (dtucker) OpenBSD CVS Sync |
230 | - andreas@cvs.openbsd.org 2009/06/27 09:29:06 | |
231 | [packet.h packet.c] | |
232 | packet_bacup_state() and packet_restore_state() will be used to | |
233 | temporarily save the current state ren resuming a suspended connection. | |
234 | ok markus@ | |
235 | - andreas@cvs.openbsd.org 2009/06/27 09:32:43 | |
236 | [roaming_common.c roaming.h] | |
237 | It may be necessary to retransmit some data when resuming, so add it | |
238 | to a buffer when roaming is enabled. | |
239 | Most of this code was written by Martin Forssen, maf at appgate dot com. | |
240 | ok markus@ | |
241 | - andreas@cvs.openbsd.org 2009/06/27 09:35:06 | |
242 | [readconf.h readconf.c] | |
243 | Add client option UseRoaming. It doesn't do anything yet but will | |
244 | control whether the client tries to use roaming if enabled on the | |
245 | server. From Martin Forssen. | |
246 | ok markus@ | |
247 | - markus@cvs.openbsd.org 2009/06/30 14:54:40 | |
248 | [version.h] | |
249 | crank version; ok deraadt | |
250 | - dtucker@cvs.openbsd.org 2009/07/02 02:11:47 | |
251 | [ssh.c] | |
252 | allow for long home dir paths (bz #1615). ok deraadt | |
253 | (based in part on a patch from jchadima at redhat) | |
254 | - stevesk@cvs.openbsd.org 2009/07/05 19:28:33 | |
255 | [clientloop.c] | |
256 | only send SSH2_MSG_DISCONNECT if we're in compat20; from dtucker@ | |
257 | ok deraadt@ markus@ | |
258 | ||
127c96db | 259 | 20090622 |
260 | - (dtucker) OpenBSD CVS Sync | |
261 | - dtucker@cvs.openbsd.org 2009/06/22 05:39:28 | |
262 | [monitor_wrap.c monitor_mm.c ssh-keygen.c auth2.c gss-genr.c sftp-client.c] | |
263 | alphabetize includes; reduces diff vs portable and style(9). | |
264 | ok stevesk djm | |
265 | (Id sync only; these were already in order in -portable) | |
266 | ||
f0956980 | 267 | 20090621 |
268 | - (dtucker) OpenBSD CVS Sync | |
269 | - markus@cvs.openbsd.org 2009/03/17 21:37:00 | |
270 | [ssh.c] | |
271 | pass correct argv[0] to openlog(); ok djm@ | |
8fe25329 | 272 | - jmc@cvs.openbsd.org 2009/03/19 15:15:09 |
273 | [ssh.1] | |
274 | for "Ciphers", just point the reader to the keyword in ssh_config(5), just | |
275 | as we do for "MACs": this stops us getting out of sync when the lists | |
276 | change; | |
277 | fixes documentation/6102, submitted by Peter J. Philipp | |
278 | alternative fix proposed by djm | |
279 | ok markus | |
230d03b6 | 280 | - tobias@cvs.openbsd.org 2009/03/23 08:31:19 |
281 | [ssh-agent.c] | |
282 | Fixed a possible out-of-bounds memory access if the environment variable | |
283 | SHELL is shorter than 3 characters. | |
284 | with input by and ok dtucker | |
7027325d | 285 | - tobias@cvs.openbsd.org 2009/03/23 19:38:04 |
286 | [ssh-agent.c] | |
287 | My previous commit didn't fix the problem at all, so stick at my first | |
288 | version of the fix presented to dtucker. | |
289 | Issue notified by Matthias Barkhoff (matthias dot barkhoff at gmx dot de). | |
290 | ok dtucker | |
b31ae930 | 291 | - sobrado@cvs.openbsd.org 2009/03/26 08:38:39 |
292 | [sftp-server.8 sshd.8 ssh-agent.1] | |
293 | fix a few typographical errors found by spell(1). | |
294 | ok dtucker@, jmc@ | |
640f440b | 295 | - stevesk@cvs.openbsd.org 2009/04/13 19:07:44 |
296 | [sshd_config.5] | |
297 | fix possessive; ok djm@ | |
7bd399ce | 298 | - stevesk@cvs.openbsd.org 2009/04/14 16:33:42 |
299 | [sftp-server.c] | |
300 | remove unused option character from getopt() optstring; ok markus@ | |
3e576dfe | 301 | - jj@cvs.openbsd.org 2009/04/14 21:10:54 |
302 | [servconf.c] | |
303 | Fixed a few the-the misspellings in comments. Skipped a bunch in | |
304 | binutils,gcc and so on. ok jmc@ | |
02d56d32 | 305 | - stevesk@cvs.openbsd.org 2009/04/17 19:23:06 |
306 | [session.c] | |
307 | use INTERNAL_SFTP_NAME for setproctitle() of in-process sftp-server; | |
308 | ok djm@ markus@ | |
db1f5925 | 309 | - stevesk@cvs.openbsd.org 2009/04/17 19:40:17 |
310 | [sshd_config.5] | |
311 | clarify that even internal-sftp needs /dev/log for logging to work; ok | |
312 | markus@ | |
47f4188a | 313 | - jmc@cvs.openbsd.org 2009/04/18 18:39:10 |
314 | [sshd_config.5] | |
315 | tweak previous; ok stevesk | |
5df1f0e3 | 316 | - stevesk@cvs.openbsd.org 2009/04/21 15:13:17 |
317 | [sshd_config.5] | |
318 | clarify we cd to user's home after chroot; ok markus@ on | |
319 | earlier version; tweaks and ok jmc@ | |
dc1f1948 | 320 | - andreas@cvs.openbsd.org 2009/05/25 06:48:01 |
321 | [channels.c packet.c clientloop.c packet.h serverloop.c monitor_wrap.c | |
322 | monitor.c] | |
323 | Put the globals in packet.c into a struct and don't access it directly | |
324 | from other files. No functional changes. | |
325 | ok markus@ djm@ | |
326 | - andreas@cvs.openbsd.org 2009/05/27 06:31:25 | |
327 | [canohost.h canohost.c] | |
328 | Add clear_cached_addr(), needed for upcoming changes allowing the peer | |
329 | address to change. | |
330 | ok markus@ | |
f936c5d4 | 331 | - andreas@cvs.openbsd.org 2009/05/27 06:33:39 |
332 | [clientloop.c] | |
333 | Send SSH2_MSG_DISCONNECT when the client disconnects. From a larger | |
334 | change from Martin Forssen, maf at appgate dot com. | |
335 | ok markus@ | |
abdc5018 | 336 | - andreas@cvs.openbsd.org 2009/05/27 06:34:36 |
337 | [kex.c kex.h] | |
338 | Move the KEX_COOKIE_LEN define to kex.h | |
339 | ok markus@ | |
87db7000 | 340 | - andreas@cvs.openbsd.org 2009/05/27 06:36:07 |
341 | [packet.h packet.c] | |
342 | Add packet_put_int64() and packet_get_int64(), part of a larger change | |
343 | from Martin Forssen. | |
c6063ed7 | 344 | ok markus@ |
345 | - andreas@cvs.openbsd.org 2009/05/27 06:38:16 | |
346 | [sshconnect.h sshconnect.c] | |
347 | Un-static ssh_exchange_identification(), part of a larger change from | |
348 | Martin Forssen and needed for upcoming changes. | |
349 | ok markus@ | |
5d4d25cd | 350 | - andreas@cvs.openbsd.org 2009/05/28 16:50:16 |
351 | [sshd.c packet.c serverloop.c monitor_wrap.c clientloop.c sshconnect.c | |
d0137ef8 | 352 | monitor.c Added roaming.h roaming_common.c roaming_dummy.c] |
5d4d25cd | 353 | Keep track of number of bytes read and written. Needed for upcoming |
354 | changes. Most code from Martin Forssen, maf at appgate dot com. | |
355 | ok markus@ | |
d0137ef8 | 356 | Also, applied appropriate changes to Makefile.in |
adb5cc1b | 357 | - andreas@cvs.openbsd.org 2009/06/12 20:43:22 |
358 | [monitor.c packet.c] | |
359 | Fix warnings found by chl@ and djm@ and change roaming_atomicio's | |
360 | return type to match atomicio's | |
361 | Diff from djm@, ok markus@ | |
6a49252d | 362 | - andreas@cvs.openbsd.org 2009/06/12 20:58:32 |
363 | [packet.c] | |
364 | Move some more statics into session_state | |
365 | ok markus@ djm@ | |
ac692f84 | 366 | - dtucker@cvs.openbsd.org 2009/06/21 07:37:15 |
367 | [kexdhs.c kexgexs.c] | |
368 | abort if key_sign fails, preventing possible null deref. Based on report | |
369 | from Paolo Ganci, ok markus@ djm@ | |
370 | - dtucker@cvs.openbsd.org 2009/06/21 09:04:03 | |
371 | [roaming.h roaming_common.c roaming_dummy.c] | |
372 | Add tags for the benefit of the sync scripts | |
373 | Also: pull in the changes for 1.1->1.2 missed in the previous sync. | |
9b9302ea | 374 | - (dtucker) [auth2-jpake.c auth2.c canohost.h session.c] Whitespace and |
375 | header-order changes to reduce diff vs OpenBSD. | |
c8dc0909 | 376 | - (dtucker) [servconf.c sshd.c] More whitespace sync. |
e85016d4 | 377 | - (dtucker) [roaming_common.c roaming_dummy.c] Wrap #include <inttypes.h> in |
378 | ifdef. | |
f0956980 | 379 | |
87562a58 | 380 | 20090616 |
381 | - (dtucker) [configure.ac defines.h] Bug #1607: handle the case where fsid_t | |
382 | is a struct with a __val member. Fixes build on, eg, Redhat 6.2. | |
383 | ||
6ee76eea | 384 | 20090504 |
385 | - (dtucker) [sshlogin.c] Move the NO_SSH_LASTLOG #ifndef line to include | |
386 | variable declarations. Should prevent unused warnings anywhere it's set | |
387 | (only Crays as far as I can tell) and be a no-op everywhere else. | |
388 | ||
bc9a470b | 389 | 20090318 |
390 | - (tim) [configure.ac] Remove setting IP_TOS_IS_BROKEN for Cygwin. The problem | |
391 | that setsockopt(IP_TOS) doesn't work on Cygwin has been fixed since 2005. | |
392 | Based on patch from vinschen at redhat com. | |
393 | ||
5077a5f6 | 394 | 20090308 |
395 | - (dtucker) [auth-passwd.c auth1.c auth2-kbdint.c auth2-none.c auth2-passwd.c | |
396 | auth2-pubkey.c session.c openbsd-compat/bsd-cygwin_util.{c,h} | |
397 | openbsd-compat/daemon.c] Remove support for Windows 95/98/ME and very old | |
398 | version of Cygwin. Patch from vinschen at redhat com. | |
399 | ||
3e566c29 | 400 | 20090307 |
401 | - (dtucker) [contrib/aix/buildbff.sh] Only try to rename ssh_prng_cmds if it | |
402 | exists (it's not created if OpenSSL's PRNG is self-seeded, eg if the OS | |
403 | has a /dev/random). | |
36b68fd5 | 404 | - (dtucker) [schnorr.c openbsd-compat/openssl-compat.{c,h}] Add |
405 | EVP_DigestUpdate to the OLD_EVP compatibility functions and tell schnorr.c | |
406 | to use them. Allows building with older OpenSSL versions. | |
aeed50df | 407 | - (dtucker) [configure.ac defines.h] Check for in_port_t and typedef if needed. |
86783a32 | 408 | - (dtucker) [configure.ac] Missing comma in type list. |
14e380c6 | 409 | - (dtucker) [configure.ac openbsd-compat/openssl-compat.{c,h}] |
410 | EVP_DigestUpdate does not exactly match the other OLD_EVP functions (eg | |
411 | in openssl 0.9.6) so add an explicit test for it. | |
3e566c29 | 412 | |
5b01421b | 413 | 20090306 |
414 | - (djm) OpenBSD CVS Sync | |
415 | - djm@cvs.openbsd.org 2009/03/05 07:18:19 | |
416 | [auth2-jpake.c jpake.c jpake.h monitor_wrap.c monitor_wrap.h schnorr.c] | |
417 | [sshconnect2.c] | |
418 | refactor the (disabled) Schnorr proof code to make it a little more | |
419 | generally useful | |
4f983ff5 | 420 | - djm@cvs.openbsd.org 2009/03/05 11:30:50 |
421 | [uuencode.c] | |
422 | document what these functions do so I don't ever have to recuse into | |
423 | b64_pton/ntop to remember their return values | |
5b01421b | 424 | |
ebf012a2 | 425 | 20090223 |
426 | - (djm) OpenBSD CVS Sync | |
427 | - djm@cvs.openbsd.org 2009/02/22 23:50:57 | |
428 | [ssh_config.5 sshd_config.5] | |
429 | don't advertise experimental options | |
09b37352 | 430 | - djm@cvs.openbsd.org 2009/02/22 23:59:25 |
431 | [sshd_config.5] | |
432 | missing period | |
52d8f3f6 | 433 | - djm@cvs.openbsd.org 2009/02/23 00:06:15 |
434 | [version.h] | |
435 | openssh-5.2 | |
dc336a3b | 436 | - (djm) [README] update for 5.2 |
0e8d25c9 | 437 | - (djm) Release openssh-5.2p1 |
ebf012a2 | 438 | |
7eec82ab | 439 | 20090222 |
440 | - (djm) OpenBSD CVS Sync | |
441 | - tobias@cvs.openbsd.org 2009/02/21 19:32:04 | |
442 | [misc.c sftp-server-main.c ssh-keygen.c] | |
443 | Added missing newlines in error messages. | |
444 | ok dtucker | |
445 | ||
1925d16d | 446 | 20090221 |
447 | - (djm) OpenBSD CVS Sync | |
448 | - djm@cvs.openbsd.org 2009/02/17 01:28:32 | |
449 | [ssh_config] | |
450 | sync with revised default ciphers; pointed out by dkrause@ | |
dca75d4b | 451 | - djm@cvs.openbsd.org 2009/02/18 04:31:21 |
452 | [schnorr.c] | |
453 | signature should hash over the entire group, not just the generator | |
454 | (this is still disabled code) | |
9a4a047b | 455 | - (djm) [contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
456 | [contrib/suse/openssh.spec] Prepare for 5.2p1 | |
1925d16d | 457 | |
aa10bde9 | 458 | 20090216 |
459 | - (djm) [regress/conch-ciphers.sh regress/putty-ciphers.sh] | |
460 | [regress/putty-kex.sh regress/putty-transfer.sh] Downgrade disabled | |
461 | interop tests from FATAL error to a warning. Allows some interop | |
462 | tests to proceed if others are missing necessary prerequisites. | |
4c3b7423 | 463 | - (djm) [configure.ac] support GNU/kFreeBSD and GNU/kOpensolaris |
464 | systems; patch from Aurelien Jarno via rmh AT aybabtu.com | |
aa10bde9 | 465 | |
69354fe2 | 466 | 20090214 |
467 | - (djm) OpenBSD CVS Sync | |
468 | - dtucker@cvs.openbsd.org 2009/02/02 11:15:14 | |
469 | [sftp.c] | |
470 | Initialize a few variables to prevent spurious "may be used | |
471 | uninitialized" warnings from newer gcc's. ok djm@ | |
17525a70 | 472 | - djm@cvs.openbsd.org 2009/02/12 03:00:56 |
473 | [canohost.c canohost.h channels.c channels.h clientloop.c readconf.c] | |
474 | [readconf.h serverloop.c ssh.c] | |
475 | support remote port forwarding with a zero listen port (-R0:...) to | |
476 | dyamically allocate a listen port at runtime (this is actually | |
477 | specified in rfc4254); bz#1003 ok markus@ | |
1d68c50a | 478 | - djm@cvs.openbsd.org 2009/02/12 03:16:01 |
479 | [serverloop.c] | |
480 | tighten check for -R0:... forwarding: only allow dynamic allocation | |
481 | if want_reply is set in the packet | |
28b5d376 | 482 | - djm@cvs.openbsd.org 2009/02/12 03:26:22 |
483 | [monitor.c] | |
484 | some paranoia: check that the serialised key is really KEY_RSA before | |
485 | diddling its internals | |
db9039d0 | 486 | - djm@cvs.openbsd.org 2009/02/12 03:42:09 |
487 | [ssh.1] | |
488 | document -R0:... usage | |
1e709459 | 489 | - djm@cvs.openbsd.org 2009/02/12 03:44:25 |
490 | [ssh.1] | |
491 | consistency: Dq => Ql | |
c6b2c0e0 | 492 | - djm@cvs.openbsd.org 2009/02/12 03:46:17 |
493 | [ssh_config.5] | |
494 | document RemoteForward usage with 0 listen port | |
e12d3e21 | 495 | - jmc@cvs.openbsd.org 2009/02/12 07:34:20 |
496 | [ssh_config.5] | |
497 | kill trailing whitespace; | |
8b773163 | 498 | - markus@cvs.openbsd.org 2009/02/13 11:50:21 |
499 | [packet.c] | |
500 | check for enc !=NULL in packet_start_discard | |
e75a14a1 | 501 | - djm@cvs.openbsd.org 2009/02/14 06:35:49 |
502 | [PROTOCOL] | |
503 | mention that eow and no-more-sessions extensions are sent only to | |
504 | OpenSSH peers | |
69354fe2 | 505 | |
506 | 20090212 | |
f7b8146b | 507 | - (djm) [sshpty.c] bz#1419: OSX uses cloning ptys that automagically |
508 | set ownership and modes, so avoid explicitly setting them | |
295dd642 | 509 | - (djm) [configure.ac loginrec.c] bz#1421: fix lastlog support for OSX. |
510 | OSX provides a getlastlogxbyname function that automates the reading of | |
511 | a lastlog file. Also, the pututxline function will update lastlog so | |
512 | there is no need for loginrec.c to do it explicitly. Collapse some | |
513 | overly verbose code while I'm in there. | |
f7b8146b | 514 | |
b4341d7a | 515 | 20090201 |
516 | - (dtucker) [defines.h sshconnect.c] INET6_ADDRSTRLEN is now needed in | |
517 | channels.c too, so move the definition for non-IP6 platforms to defines.h | |
518 | where it can be shared. | |
519 | ||
e1986e0a | 520 | 20090129 |
521 | - (tim) [contrib/cygwin/ssh-host-config] Patch from Corinna Vinschen. | |
522 | If the CYGWIN environment variable is empty, the installer script | |
523 | should not install the service with an empty CYGWIN variable, but | |
524 | rather without setting CYGWNI entirely. | |
863ba23a | 525 | - (tim) [contrib/cygwin/ssh-host-config] Whitespace cleanup. No code changes. |
e1986e0a | 526 | |
7f24626b | 527 | 20090128 |
528 | - (tim) [contrib/cygwin/ssh-host-config] Patch from Corinna Vinschen. | |
529 | Changes to work on Cygwin 1.5.x as well as on the new Cygwin 1.7.x. | |
530 | The information given for the setting of the CYGWIN environment variable | |
531 | is wrong for both releases so I just removed it, together with the | |
532 | unnecessary (Cygwin 1.5.x) or wrong (Cygwin 1.7.x) default setting. | |
533 | ||
68405671 | 534 | 20081228 |
535 | - (djm) OpenBSD CVS Sync | |
536 | - stevesk@cvs.openbsd.org 2008/12/09 03:20:42 | |
537 | [channels.c servconf.c] | |
538 | channel_print_adm_permitted_opens() should deal with all the printing | |
539 | for that config option. suggested by markus@; ok markus@ djm@ | |
540 | dtucker@ | |
7efff8ce | 541 | - djm@cvs.openbsd.org 2008/12/09 04:32:22 |
542 | [auth2-chall.c] | |
543 | replace by-hand string building with xasprinf(); ok deraadt@ | |
d3cd4016 | 544 | - sobrado@cvs.openbsd.org 2008/12/09 15:35:00 |
545 | [sftp.1 sftp.c] | |
546 | update for the synopses displayed by the 'help' command, there are a | |
547 | few missing flags; add 'bye' to the output of 'help'; sorting and spacing. | |
548 | jmc@ suggested replacing .Oo/.Oc with a single .Op macro. | |
549 | ok jmc@ | |
6c20a13f | 550 | - stevesk@cvs.openbsd.org 2008/12/09 22:37:33 |
551 | [clientloop.c] | |
552 | fix typo in error message | |
fd2ce9c6 | 553 | - stevesk@cvs.openbsd.org 2008/12/10 03:55:20 |
554 | [addrmatch.c] | |
555 | o cannot be NULL here but use xfree() to be consistent; ok djm@ | |
8647612c | 556 | - stevesk@cvs.openbsd.org 2008/12/29 01:12:36 |
557 | [ssh-keyscan.1] | |
558 | fix example, default key type is rsa for 3+ years; from | |
559 | frederic.perrin@resel.fr | |
040d6b1f | 560 | - stevesk@cvs.openbsd.org 2008/12/29 02:23:26 |
561 | [pathnames.h] | |
562 | no need to escape single quotes in comments | |
d4bfdc62 | 563 | - okan@cvs.openbsd.org 2008/12/30 00:46:56 |
564 | [sshd_config.5] | |
565 | add AllowAgentForwarding to available Match keywords list | |
566 | ok djm | |
6cf44b6a | 567 | - djm@cvs.openbsd.org 2009/01/01 21:14:35 |
568 | [channels.c] | |
569 | call channel destroy callbacks on receipt of open failure messages. | |
570 | fixes client hangs when connecting to a server that has MaxSessions=0 | |
571 | set spotted by imorgan AT nas.nasa.gov; ok markus@ | |
546202d0 | 572 | - djm@cvs.openbsd.org 2009/01/01 21:17:36 |
573 | [kexgexs.c] | |
574 | fix hash calculation for KEXGEX: hash over the original client-supplied | |
575 | values and not the sanity checked versions that we acutally use; | |
576 | bz#1540 reported by john.smith AT arrows.demon.co.uk | |
577 | ok markus@ | |
4866a6d6 | 578 | - djm@cvs.openbsd.org 2009/01/14 01:38:06 |
579 | [channels.c] | |
580 | support SOCKS4A protocol, from dwmw2 AT infradead.org via bz#1482; | |
581 | "looks ok" markus@ | |
9b4b86c2 | 582 | - stevesk@cvs.openbsd.org 2009/01/15 17:38:43 |
583 | [readconf.c] | |
584 | 1) use obsolete instead of alias for consistency | |
585 | 2) oUserKnownHostsFile not obsolete but oGlobalKnownHostsFile2 is | |
586 | so move the comment. | |
587 | 3) reorder so like options are together | |
588 | ok djm@ | |
1338ba77 | 589 | - djm@cvs.openbsd.org 2009/01/22 09:46:01 |
590 | [channels.c channels.h session.c] | |
591 | make Channel->path an allocated string, saving a few bytes here and | |
592 | there and fixing bz#1380 in the process; ok markus@ | |
920706fd | 593 | - djm@cvs.openbsd.org 2009/01/22 09:49:57 |
594 | [channels.c] | |
595 | oops! I committed the wrong version of the Channel->path diff, | |
596 | it was missing some tweaks suggested by stevesk@ | |
5134115d | 597 | - djm@cvs.openbsd.org 2009/01/22 10:02:34 |
598 | [clientloop.c misc.c readconf.c readconf.h servconf.c servconf.h] | |
599 | [serverloop.c ssh-keyscan.c ssh.c sshd.c] | |
600 | make a2port() return -1 when it encounters an invalid port number | |
601 | rather than 0, which it will now treat as valid (needed for future work) | |
602 | adjust current consumers of a2port() to check its return value is <= 0, | |
603 | which in turn required some things to be converted from u_short => int | |
604 | make use of int vs. u_short consistent in some other places too | |
605 | feedback & ok markus@ | |
368e246f | 606 | - djm@cvs.openbsd.org 2009/01/22 10:09:16 |
607 | [auth-options.c] | |
608 | another chunk of a2port() diff that got away. wtfdjm?? | |
700fd7e7 | 609 | - djm@cvs.openbsd.org 2009/01/23 07:58:11 |
610 | [myproposal.h] | |
611 | prefer CTR modes and revised arcfour (i.e w/ discard) modes to CBC | |
612 | modes; ok markus@ | |
29ec8eb3 | 613 | - naddy@cvs.openbsd.org 2009/01/24 17:10:22 |
614 | [ssh_config.5 sshd_config.5] | |
615 | sync list of preferred ciphers; ok djm@ | |
608bcf58 | 616 | - markus@cvs.openbsd.org 2009/01/26 09:58:15 |
617 | [cipher.c cipher.h packet.c] | |
618 | Work around the CPNI-957037 Plaintext Recovery Attack by always | |
619 | reading 256K of data on packet size or HMAC errors (in CBC mode only). | |
620 | Help, feedback and ok djm@ | |
621 | Feedback from Martin Albrecht and Paterson Kenny | |
68405671 | 622 | |
a25d08b3 | 623 | 20090107 |
624 | - (djm) [uidswap.c] bz#1412: Support >16 supplemental groups in OS X. | |
625 | Patch based on one from vgiffin AT apple.com; ok dtucker@ | |
b5a1596f | 626 | - (djm) [channels.c] bz#1419: support "on demand" X11 forwarding via |
627 | launchd on OS X; patch from vgiffin AT apple.com, slightly tweaked; | |
628 | ok dtucker@ | |
23b3ed0b | 629 | - (djm) [contrib/ssh-copy-id.1 contrib/ssh-copy-id] bz#1492: Make |
630 | ssh-copy-id copy id_rsa.pub by default (instead of the legacy "identity" | |
631 | key). Patch from cjwatson AT debian.org | |
a25d08b3 | 632 | |
16076ac9 | 633 | 20090107 |
634 | - (tim) [configure.ac defines.h openbsd-compat/port-uw.c | |
635 | openbsd-compat/xcrypt.c] Add SECUREWARE support to OpenServer 6 SVR5 ABI. | |
636 | OK djm@ dtucker@ | |
44a71983 | 637 | - (tim) [configure.ac] Move check_for_libcrypt_later=1 in *-*-sysv5*) section. |
638 | OpenServer 6 doesn't need libcrypt. | |
16076ac9 | 639 | |
09925c00 | 640 | 20081209 |
641 | - (djm) OpenBSD CVS Sync | |
642 | - djm@cvs.openbsd.org 2008/12/09 02:38:18 | |
643 | [clientloop.c] | |
644 | The ~C escape handler does not work correctly for multiplexed sessions - | |
645 | it opens a commandline on the master session, instead of on the slave | |
646 | that requested it. Disable it on slave sessions until such time as it | |
647 | is fixed; bz#1543 report from Adrian Bridgett via Colin Watson | |
648 | ok markus@ | |
ddb5e00f | 649 | - djm@cvs.openbsd.org 2008/12/09 02:39:59 |
650 | [sftp.c] | |
651 | Deal correctly with failures in remote stat() operation in sftp, | |
652 | correcting fail-on-error behaviour in batchmode. bz#1541 report and | |
653 | fix from anedvedicky AT gmail.com; ok markus@ | |
bab3d903 | 654 | - djm@cvs.openbsd.org 2008/12/09 02:58:16 |
655 | [readconf.c] | |
656 | don't leave junk (free'd) pointers around in Forward *fwd argument on | |
657 | failure; avoids double-free in ~C -L handler when given an invalid | |
658 | forwarding specification; bz#1539 report from adejong AT debian.org | |
659 | via Colin Watson; ok markus@ dtucker@ | |
83cd8c39 | 660 | - djm@cvs.openbsd.org 2008/12/09 03:02:37 |
661 | [sftp.1 sftp.c] | |
662 | correct sftp(1) and corresponding usage syntax; | |
663 | bz#1518 patch from imorgan AT nas.nasa.gov; ok deraadt@ improved diff jmc@ | |
09925c00 | 664 | |
53e2660a | 665 | 20081208 |
666 | - (djm) [configure.ac] bz#1538: better test for ProPolice/SSP: actually | |
667 | use some stack in main(). | |
668 | Report and suggested fix from vapier AT gentoo.org | |
66e16767 | 669 | - (djm) OpenBSD CVS Sync |
670 | - markus@cvs.openbsd.org 2008/12/02 19:01:07 | |
671 | [clientloop.c] | |
672 | we have to use the recipient's channel number (RFC 4254) for | |
673 | SSH2_MSG_CHANNEL_SUCCESS/SSH2_MSG_CHANNEL_FAILURE messages, | |
674 | otherwise we trigger 'Non-public channel' error messages on sshd | |
675 | systems with clientkeepalive enabled; noticed by sturm; ok djm; | |
7ec2b275 | 676 | - markus@cvs.openbsd.org 2008/12/02 19:08:59 |
677 | [serverloop.c] | |
678 | backout 1.149, since it's not necessary and openssh clients send | |
679 | broken CHANNEL_FAILURE/SUCCESS messages since about 2004; ok djm@ | |
b09b559d | 680 | - markus@cvs.openbsd.org 2008/12/02 19:09:38 |
681 | [channels.c] | |
682 | s/remote_id/id/ to be more consistent with other code; ok djm@ | |
53e2660a | 683 | |
de470c82 | 684 | 20081201 |
685 | - (dtucker) [contrib/cygwin/{Makefile,ssh-host-config}] Add new doc files | |
686 | and tweak the is-sshd-running check in ssh-host-config. Patch from | |
687 | vinschen at redhat com. | |
66af1c21 | 688 | - (dtucker) OpenBSD CVS Sync |
689 | - markus@cvs.openbsd.org 2008/11/21 15:47:38 | |
690 | [packet.c] | |
691 | packet_disconnect() on padding error, too. should reduce the success | |
692 | probability for the CPNI-957037 Plaintext Recovery Attack to 2^-18 | |
693 | ok djm@ | |
8e10da10 | 694 | - dtucker@cvs.openbsd.org 2008/11/30 11:59:26 |
695 | [monitor_fdpass.c] | |
696 | Retry sendmsg/recvmsg on EAGAIN and EINTR; ok djm@ | |
de470c82 | 697 | |
d2aa725a | 698 | 20081123 |
699 | - (dtucker) [monitor_fdpass.c] Reduce diff vs OpenBSD by moving some | |
700 | declarations, removing an unnecessary union member and adding whitespace. | |
c037a517 | 701 | cmsgbuf.tmp thing spotted by des at des no, ok djm some time ago. |
d2aa725a | 702 | |
95e16084 | 703 | 20081118 |
704 | - (tim) [addrmatch.c configure.ac] Some platforms do not have sin6_scope_id | |
705 | member of sockaddr_in6. Also reported in Bug 1491 by David Leonard. OK and | |
706 | feedback by djm@ | |
707 | ||
0bd3332c | 708 | 20081111 |
709 | - (dtucker) OpenBSD CVS Sync | |
710 | - jmc@cvs.openbsd.org 2008/11/05 11:22:54 | |
711 | [servconf.c] | |
712 | passord -> password; | |
713 | fixes user/5975 from Rene Maroufi | |
1890bf8b | 714 | - stevesk@cvs.openbsd.org 2008/11/07 00:42:12 |
715 | [ssh-keygen.c] | |
716 | spelling/typo in comment | |
92d0164c | 717 | - stevesk@cvs.openbsd.org 2008/11/07 18:50:18 |
718 | [nchan.c] | |
719 | add space to some log/debug messages for readability; ok djm@ markus@ | |
3d7f6c3d | 720 | - dtucker@cvs.openbsd.org 2008/11/07 23:34:48 |
721 | [auth2-jpake.c] | |
722 | Move JPAKE define to make life easier for portable. ok djm@ | |
94087553 | 723 | - tobias@cvs.openbsd.org 2008/11/09 12:34:47 |
724 | [session.c ssh.1] | |
725 | typo fixed (overriden -> overridden) | |
726 | ok espie, jmc | |
2505b891 | 727 | - stevesk@cvs.openbsd.org 2008/11/11 02:58:09 |
728 | [servconf.c] | |
729 | USE_AFS not referenced so remove #ifdef. fixes sshd -T not printing | |
730 | kerberosgetafstoken. ok dtucker@ | |
731 | (Id sync only, we still want the ifdef in portable) | |
861e9e53 | 732 | - stevesk@cvs.openbsd.org 2008/11/11 03:55:11 |
733 | [channels.c] | |
734 | for sshd -T print 'permitopen any' vs. 'permitopen' for case of no | |
735 | permitopen's; ok and input dtucker@ | |
0771f5dd | 736 | - djm@cvs.openbsd.org 2008/11/10 02:06:35 |
737 | [regress/putty-ciphers.sh] | |
738 | PuTTY supports AES CTR modes, so interop test against them too | |
0bd3332c | 739 | |
39aa8698 | 740 | 20081105 |
741 | - OpenBSD CVS Sync | |
742 | - djm@cvs.openbsd.org 2008/11/03 08:59:41 | |
743 | [servconf.c] | |
744 | include MaxSessions in sshd -T output; patch from imorgan AT nas.nasa.gov | |
a28625a6 | 745 | - djm@cvs.openbsd.org 2008/11/04 07:58:09 |
746 | [auth.c] | |
747 | need unistd.h for close() prototype | |
748 | (ID sync only) | |
5adf6b9a | 749 | - djm@cvs.openbsd.org 2008/11/04 08:22:13 |
750 | [auth.h auth2.c monitor.c monitor.h monitor_wrap.c monitor_wrap.h] | |
751 | [readconf.c readconf.h servconf.c servconf.h ssh2.h ssh_config.5] | |
752 | [sshconnect2.c sshd_config.5 jpake.c jpake.h schnorr.c auth2-jpake.c] | |
753 | [Makefile.in] | |
754 | Add support for an experimental zero-knowledge password authentication | |
755 | method using the J-PAKE protocol described in F. Hao, P. Ryan, | |
756 | "Password Authenticated Key Exchange by Juggling", 16th Workshop on | |
757 | Security Protocols, Cambridge, April 2008. | |
758 | ||
759 | This method allows password-based authentication without exposing | |
760 | the password to the server. Instead, the client and server exchange | |
761 | cryptographic proofs to demonstrate of knowledge of the password while | |
762 | revealing nothing useful to an attacker or compromised endpoint. | |
763 | ||
764 | This is experimental, work-in-progress code and is presently | |
765 | compiled-time disabled (turn on -DJPAKE in Makefile.inc). | |
766 | ||
767 | "just commit it. It isn't too intrusive." deraadt@ | |
d35f707e | 768 | - stevesk@cvs.openbsd.org 2008/11/04 19:18:00 |
769 | [readconf.c] | |
770 | because parse_forward() is now used to parse all forward types (DLR), | |
771 | and it malloc's space for host variables, we don't need to malloc | |
772 | here. fixes small memory leaks. | |
773 | ||
774 | previously dynamic forwards were not parsed in parse_forward() and | |
775 | space was not malloc'd in that case. | |
776 | ||
777 | ok djm@ | |
10cf2ac3 | 778 | - stevesk@cvs.openbsd.org 2008/11/05 03:23:09 |
779 | [clientloop.c ssh.1] | |
780 | add dynamic forward escape command line; ok djm@ | |
39aa8698 | 781 | |
94f36816 | 782 | 20081103 |
783 | - OpenBSD CVS Sync | |
784 | - sthen@cvs.openbsd.org 2008/07/24 23:55:30 | |
785 | [ssh-keygen.1] | |
786 | Add "ssh-keygen -F -l" to synopsis (displays fingerprint from | |
787 | known_hosts). ok djm@ | |
788 | - grunk@cvs.openbsd.org 2008/07/25 06:56:35 | |
789 | [ssh_config] | |
790 | Add VisualHostKey to example file, ok djm@ | |
5ca42ddb | 791 | - grunk@cvs.openbsd.org 2008/07/25 07:05:16 |
792 | [key.c] | |
793 | In random art visualization, make sure to use the end marker only at the | |
794 | end. Initial diff by Dirk Loss, tweaks and ok djm@ | |
341cb46b | 795 | - markus@cvs.openbsd.org 2008/07/31 14:48:28 |
796 | [sshconnect2.c] | |
797 | don't allocate space for empty banners; report t8m at centrum.cz; | |
798 | ok deraadt | |
686bdcbd | 799 | - krw@cvs.openbsd.org 2008/08/02 04:29:51 |
800 | [ssh_config.5] | |
801 | whitepsace -> whitespace. From Matthew Clarke via bugs@. | |
e3ef5245 | 802 | - djm@cvs.openbsd.org 2008/08/21 04:09:57 |
803 | [session.c] | |
804 | allow ForceCommand internal-sftp with arguments. based on patch from | |
805 | michael.barabanov AT gmail.com; ok markus@ | |
1975fb98 | 806 | - djm@cvs.openbsd.org 2008/09/06 12:24:13 |
807 | [kex.c] | |
808 | OpenSSL 0.9.8h supplies a real EVP_sha256 so we do not need our | |
809 | replacement anymore | |
810 | (ID sync only for portable - we still need this) | |
72bd2fca | 811 | - markus@cvs.openbsd.org 2008/09/11 14:22:37 |
812 | [compat.c compat.h nchan.c ssh.c] | |
813 | only send eow and no-more-sessions requests to openssh 5 and newer; | |
814 | fixes interop problems with broken ssh v2 implementations; ok djm@ | |
2e96832c | 815 | - millert@cvs.openbsd.org 2008/10/02 14:39:35 |
816 | [session.c] | |
817 | Convert an unchecked strdup to xstrdup. OK deraadt@ | |
dc94d57e | 818 | - jmc@cvs.openbsd.org 2008/10/03 13:08:12 |
819 | [sshd.8] | |
820 | do not give an example of how to chmod files: we can presume the user | |
821 | knows that. removes an ambiguity in the permission of authorized_keys; | |
822 | ok deraadt | |
90d5350e | 823 | - deraadt@cvs.openbsd.org 2008/10/03 23:56:28 |
824 | [sshconnect2.c] | |
825 | Repair strnvis() buffersize of 4*n+1, with termination gauranteed by the | |
826 | function. | |
827 | spotted by des@freebsd, who commited an incorrect fix to the freebsd tree | |
828 | and (as is fairly typical) did not report the problem to us. But this fix | |
829 | is correct. | |
830 | ok djm | |
bf793210 | 831 | - djm@cvs.openbsd.org 2008/10/08 23:34:03 |
832 | [ssh.1 ssh.c] | |
833 | Add -y option to force logging via syslog rather than stderr. | |
834 | Useful for daemonised ssh connection (ssh -f). Patch originally from | |
835 | and ok'd by markus@ | |
e68868a1 | 836 | - djm@cvs.openbsd.org 2008/10/09 03:50:54 |
837 | [servconf.c sshd_config.5] | |
838 | support setting PermitEmptyPasswords in a Match block | |
839 | requested in PR3891; ok dtucker@ | |
f3a4d0d0 | 840 | - jmc@cvs.openbsd.org 2008/10/09 06:54:22 |
841 | [ssh.c] | |
842 | add -y to usage(); | |
6503dc91 | 843 | - stevesk@cvs.openbsd.org 2008/10/10 04:55:16 |
844 | [scp.c] | |
845 | spelling in comment; ok djm@ | |
260bf88a | 846 | - stevesk@cvs.openbsd.org 2008/10/10 05:00:12 |
847 | [key.c] | |
848 | typo in error message; ok djm@ | |
96a00a9d | 849 | - stevesk@cvs.openbsd.org 2008/10/10 16:43:27 |
850 | [ssh_config.5] | |
851 | use 'Privileged ports can be forwarded only when logging in as root on | |
852 | the remote machine.' for RemoteForward just like ssh.1 -R. | |
853 | ok djm@ jmc@ | |
854 | - stevesk@cvs.openbsd.org 2008/10/14 18:11:33 | |
855 | [sshconnect.c] | |
856 | use #define ROQUIET here; no binary change. ok dtucker@ | |
8d20b087 | 857 | - stevesk@cvs.openbsd.org 2008/10/17 18:36:24 |
858 | [ssh_config.5] | |
859 | correct and clarify VisualHostKey; ok jmc@ | |
25f93f2c | 860 | - stevesk@cvs.openbsd.org 2008/10/30 19:31:16 |
861 | [clientloop.c sshd.c] | |
862 | don't need to #include "monitor_fdpass.h" | |
b8974c94 | 863 | - stevesk@cvs.openbsd.org 2008/10/31 15:05:34 |
864 | [dispatch.c] | |
865 | remove unused #define DISPATCH_MIN; ok markus@ | |
e64399cc | 866 | - djm@cvs.openbsd.org 2008/11/01 04:50:08 |
867 | [sshconnect2.c] | |
868 | sprinkle ARGSUSED on dispatch handlers | |
869 | nuke stale unusued prototype | |
2ea438c2 | 870 | - stevesk@cvs.openbsd.org 2008/11/01 06:43:33 |
871 | [channels.c] | |
872 | fix some typos in log messages; ok djm@ | |
9995aaa3 | 873 | - sobrado@cvs.openbsd.org 2008/11/01 11:14:36 |
874 | [ssh-keyscan.1 ssh-keyscan.c] | |
875 | the ellipsis is not an optional argument; while here, improve spacing. | |
30573fea | 876 | - stevesk@cvs.openbsd.org 2008/11/01 17:40:33 |
877 | [clientloop.c readconf.c readconf.h ssh.c] | |
878 | merge dynamic forward parsing into parse_forward(); | |
879 | 'i think this is OK' djm@ | |
9bbba34b | 880 | - stevesk@cvs.openbsd.org 2008/11/02 00:16:16 |
881 | [ttymodes.c] | |
882 | protocol 2 tty modes support is now 7.5 years old so remove these | |
883 | debug3()s; ok deraadt@ | |
b626b7ae | 884 | - stevesk@cvs.openbsd.org 2008/11/03 01:07:02 |
885 | [readconf.c] | |
886 | remove valueless comment | |
c8eaf0ec | 887 | - stevesk@cvs.openbsd.org 2008/11/03 02:44:41 |
888 | [readconf.c] | |
889 | fix comment | |
fce91335 | 890 | - (djm) [contrib/caldera/ssh-host-keygen contrib/suse/rc.sshd] |
891 | Make example scripts generate keys with default sizes rather than fixed, | |
892 | non-default 1024 bits; patch from imorgan AT nas.nasa.gov | |
933e2f91 | 893 | - (djm) [contrib/sshd.pam.generic contrib/caldera/sshd.pam] |
894 | [contrib/redhat/sshd.pam] Move pam_nologin to account group from | |
895 | incorrect auth group in example files; | |
896 | patch from imorgan AT nas.nasa.gov | |
94f36816 | 897 | |
d6339843 | 898 | 20080906 |
899 | - (dtucker) [config.guess config.sub] Update to latest versions from | |
900 | http://git.savannah.gnu.org/gitweb/ (2008-04-14 and 2008-06-16 | |
901 | respectively). | |
902 | ||
974ce4a0 | 903 | 20080830 |
904 | - (dtucker) [openbsd-compat/bsd-poll.c] correctly check for number of FDs | |
905 | larger than FD_SETSIZE (OpenSSH only ever uses poll with one fd). Patch | |
906 | from Nicholas Marriott. | |
907 | ||
e888d981 | 908 | 20080721 |
909 | - (djm) OpenBSD CVS Sync | |
910 | - djm@cvs.openbsd.org 2008/07/23 07:36:55 | |
911 | [servconf.c] | |
912 | do not try to print options that have been compile-time disabled | |
913 | in config test mode (sshd -T); report from nix-corp AT esperi.org.uk | |
914 | ok dtucker@ | |
cee47c9f | 915 | - (djm) [servconf.c] Print UsePAM option in config test mode (when it |
916 | has been compiled in); report from nix-corp AT esperi.org.uk | |
917 | ok dtucker@ | |
e888d981 | 918 | |
b14e719f | 919 | 20080721 |
920 | - (djm) OpenBSD CVS Sync | |
921 | - jmc@cvs.openbsd.org 2008/07/18 22:51:01 | |
922 | [sftp-server.8] | |
923 | no need for .Pp before or after .Sh; | |
f3b93df3 | 924 | - djm@cvs.openbsd.org 2008/07/21 08:19:07 |
925 | [version.h] | |
926 | openssh-5.1 | |
11368183 | 927 | - (djm) [README contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
928 | [contrib/suse/openssh.spec] Update version number in README and RPM specs | |
55d5db1c | 929 | - (djm) Release OpenSSH-5.1 |
b14e719f | 930 | |
e5df5ff2 | 931 | 20080717 |
932 | - (djm) OpenBSD CVS Sync | |
933 | - djm@cvs.openbsd.org 2008/07/17 08:48:00 | |
934 | [sshconnect2.c] | |
935 | strnvis preauth banner; pointed out by mpf@ ok markus@ | |
2800468d | 936 | - djm@cvs.openbsd.org 2008/07/17 08:51:07 |
937 | [auth2-hostbased.c] | |
938 | strip trailing '.' from hostname when HostbasedUsesNameFromPacketOnly=yes | |
939 | report and patch from res AT qoxp.net (bz#1200); ok markus@ | |
d9d96f7a | 940 | - (dtucker) [openbsd-compat/bsd-cygwin_util.c] Remove long-unneeded compat |
941 | code, replace with equivalent cygwin library call. Patch from vinschen | |
3a69fb58 | 942 | at redhat.com, ok djm@. |
943 | - (djm) [sshconnect2.c] vis.h isn't available everywhere | |
e5df5ff2 | 944 | |
b8c9ea19 | 945 | 20080716 |
946 | - OpenBSD CVS Sync | |
947 | - djm@cvs.openbsd.org 2008/07/15 02:23:14 | |
948 | [sftp.1] | |
949 | number of pipelined requests is now 64; | |
950 | prodded by Iain.Morgan AT nasa.gov | |
dfe666f6 | 951 | - djm@cvs.openbsd.org 2008/07/16 11:51:14 |
952 | [clientloop.c] | |
953 | rename variable first_gc -> last_gc (since it is actually the last | |
954 | in the list). | |
cdfbc829 | 955 | - djm@cvs.openbsd.org 2008/07/16 11:52:19 |
956 | [channels.c] | |
957 | this loop index should be automatic, not static | |
b8c9ea19 | 958 | |
322b3f02 | 959 | 20080714 |
960 | - (djm) OpenBSD CVS Sync | |
961 | - sthen@cvs.openbsd.org 2008/07/13 21:22:52 | |
962 | [ssh-keygen.c] | |
963 | Change "ssh-keygen -F [host] -l" to not display random art unless | |
964 | -v is also specified, making it consistent with the manual and other | |
965 | uses of -l. | |
966 | ok grunk@ | |
9fb764ab | 967 | - djm@cvs.openbsd.org 2008/07/13 22:13:07 |
968 | [channels.c] | |
969 | use struct sockaddr_storage instead of struct sockaddr for accept(2) | |
970 | address argument. from visibilis AT yahoo.com in bz#1485; ok markus@ | |
873722cc | 971 | - djm@cvs.openbsd.org 2008/07/13 22:16:03 |
972 | [sftp.c] | |
973 | increase number of piplelined requests so they properly fill the | |
974 | (recently increased) channel window. prompted by rapier AT psc.edu; | |
975 | ok markus@ | |
66fba053 | 976 | - djm@cvs.openbsd.org 2008/07/14 01:55:56 |
977 | [sftp-server.8] | |
978 | mention requirement for /dev/log inside chroot when using sftp-server | |
979 | with ChrootDirectory | |
6c6bb9a6 | 980 | - (djm) [openbsd-compat/bindresvport.c] Rename variables s/sin/in/ to |
981 | avoid clash with sin(3) function; reported by | |
982 | cristian.ionescu-idbohrn AT axis.com | |
7be182d4 | 983 | - (djm) [openbsd-compat/rresvport.c] Add unistd.h for missing close() |
984 | prototype; reported by cristian.ionescu-idbohrn AT axis.com | |
6a9c22a5 | 985 | - (djm) [umac.c] Rename variable s/buffer_ptr/bufp/ to avoid clash; |
986 | reported by cristian.ionescu-idbohrn AT axis.com | |
7ea1abf7 | 987 | - (djm) [contrib/cygwin/Makefile contrib/cygwin/ssh-host-config] |
988 | [contrib/cygwin/ssh-user-config contrib/cygwin/sshd-inetd] | |
989 | Revamped and simplified Cygwin ssh-host-config script that uses | |
990 | unified csih configuration tool. Requires recent Cygwin. | |
991 | Patch from vinschen AT redhat.com | |
322b3f02 | 992 | |
267d5589 | 993 | 20080712 |
994 | - (djm) OpenBSD CVS Sync | |
995 | - djm@cvs.openbsd.org 2008/07/12 04:52:50 | |
996 | [channels.c] | |
997 | unbreak; move clearing of cctx struct to before first use | |
998 | reported by dkrause@ | |
da9a823d | 999 | - djm@cvs.openbsd.org 2008/07/12 05:33:41 |
1000 | [scp.1] | |
1001 | better description for -i flag: | |
1002 | s/RSA authentication/public key authentication/ | |
2ade01eb | 1003 | - (djm) [openbsd-compat/fake-rfc2553.c openbsd-compat/fake-rfc2553.h] |
1004 | return EAI_FAMILY when trying to lookup unsupported address family; | |
1005 | from vinschen AT redhat.com | |
267d5589 | 1006 | |
971deff8 | 1007 | 20080711 |
1008 | - (djm) OpenBSD CVS Sync | |
1009 | - stevesk@cvs.openbsd.org 2008/07/07 00:31:41 | |
1010 | [ttymodes.c] | |
1011 | we don't need arg after the debug3() was removed. from lint. | |
1012 | ok djm@ | |
d5b5b8f6 | 1013 | - stevesk@cvs.openbsd.org 2008/07/07 23:32:51 |
1014 | [key.c] | |
1015 | /*NOTREACHED*/ for lint warning: | |
1016 | warning: function key_equal falls off bottom without returning value | |
1017 | ok djm@ | |
6eb3f18c | 1018 | - markus@cvs.openbsd.org 2008/07/10 18:05:58 |
1019 | [channels.c] | |
1020 | missing bzero; from mickey; ok djm@ | |
e8e08a80 | 1021 | - markus@cvs.openbsd.org 2008/07/10 18:08:11 |
1022 | [clientloop.c monitor.c monitor_wrap.c packet.c packet.h sshd.c] | |
1023 | sync v1 and v2 traffic accounting; add it to sshd, too; | |
1024 | ok djm@, dtucker@ | |
971deff8 | 1025 | |
17969fcc | 1026 | 20080709 |
1027 | - (djm) [Makefile.in] Print "all tests passed" when all regress tests pass | |
b5fc5d94 | 1028 | - (djm) [auth1.c] Fix format string vulnerability in protocol 1 PAM |
1029 | account check failure path. The vulnerable format buffer is supplied | |
1030 | from PAM and should not contain attacker-supplied data. | |
78cb4705 | 1031 | - (djm) [auth.c] Missing unistd.h for close() |
b5902374 | 1032 | - (djm) [configure.ac] Add -Wformat-security to CFLAGS for gcc 3.x and 4.x |
17969fcc | 1033 | |
3fde0623 | 1034 | 20080705 |
1035 | - (djm) [auth.c] Fixed test for locked account on HP/UX with shadowed | |
1036 | passwords disabled. bz#1083 report & patch from senthilkumar_sen AT | |
1037 | hotpop.com, w/ dtucker@ | |
e8983917 | 1038 | - (djm) [atomicio.c configure.ac] Disable poll() fallback in atomiciov for |
1039 | Tru64. readv doesn't seem to be a comparable object there. | |
1040 | bz#1386, patch from dtucker@ ok me | |
b8d635d0 | 1041 | - (djm) [Makefile.in] Pass though pass to conch for interop tests |
6ed8a3ae | 1042 | - (djm) [configure.ac] unbreak: remove extra closing brace |
d38d9a80 | 1043 | - (djm) OpenBSD CVS Sync |
1044 | - djm@cvs.openbsd.org 2008/07/04 23:08:25 | |
1045 | [packet.c] | |
1046 | handle EINTR in packet_write_poll()l ok dtucker@ | |
71709bcd | 1047 | - djm@cvs.openbsd.org 2008/07/04 23:30:16 |
1048 | [auth1.c auth2.c] | |
1049 | Make protocol 1 MaxAuthTries logic match protocol 2's. | |
1050 | Do not treat the first protocol 2 authentication attempt as | |
1051 | a failure IFF it is for method "none". | |
1052 | Makes MaxAuthTries' user-visible behaviour identical for | |
1053 | protocol 1 vs 2. | |
1054 | ok dtucker@ | |
3086db6e | 1055 | - djm@cvs.openbsd.org 2008/07/05 05:16:01 |
1056 | [PROTOCOL] | |
1057 | grammar | |
3fde0623 | 1058 | |
a0d38609 | 1059 | 20080704 |
1060 | - (dtucker) OpenBSD CVS Sync | |
1061 | - djm@cvs.openbsd.org 2008/07/02 13:30:34 | |
1062 | [auth2.c] | |
1063 | really really remove the freebie "none" auth try for protocol 2 | |
6c777090 | 1064 | - djm@cvs.openbsd.org 2008/07/02 13:47:39 |
1065 | [ssh.1 ssh.c] | |
1066 | When forking after authentication ("ssh -f") with ExitOnForwardFailure | |
1067 | enabled, delay the fork until after replies for any -R forwards have | |
1068 | been seen. Allows for robust detection of -R forward failure when | |
1069 | using -f (similar to bz#92); ok dtucker@ | |
f0b9fde3 | 1070 | - otto@cvs.openbsd.org 2008/07/03 21:46:58 |
1071 | [auth2-pubkey.c] | |
1072 | avoid nasty double free; ok dtucker@ djm@ | |
cece208b | 1073 | - djm@cvs.openbsd.org 2008/07/04 03:44:59 |
1074 | [servconf.c groupaccess.h groupaccess.c] | |
1075 | support negation of groups in "Match group" block (bz#1315); ok dtucker@ | |
c54d3d1c | 1076 | - dtucker@cvs.openbsd.org 2008/07/04 03:47:02 |
1077 | [monitor.c] | |
1078 | Make debug a little clearer. ok djm@ | |
c7cbf377 | 1079 | - djm@cvs.openbsd.org 2008/06/30 08:07:34 |
1080 | [regress/key-options.sh] | |
1081 | shell portability: use "=" instead of "==" in test(1) expressions, | |
1082 | double-quote string with backslash escaped / | |
8a972082 | 1083 | - djm@cvs.openbsd.org 2008/06/30 10:31:11 |
1084 | [regress/{putty-transfer,putty-kex,putty-ciphers}.sh] | |
1085 | remove "set -e" left over from debugging | |
9b0c87d9 | 1086 | - djm@cvs.openbsd.org 2008/06/30 10:43:03 |
1087 | [regress/conch-ciphers.sh] | |
1088 | explicitly disable conch options that could interfere with the test | |
97e61398 | 1089 | - (dtucker) [sftp-server.c] Bug #1447: fall back to racy rename if link |
1090 | returns EXDEV. Patch from Mike Garrison, ok djm@ | |
5a0c8771 | 1091 | - (djm) [atomicio.c channels.c clientloop.c defines.h includes.h] |
1092 | [packet.c scp.c serverloop.c sftp-client.c ssh-agent.c ssh-keyscan.c] | |
1093 | [sshd.c] Explicitly handle EWOULDBLOCK wherever we handle EAGAIN, on | |
1094 | some platforms (HP nonstop) it is a distinct errno; | |
1095 | bz#1467 reported by sconeu AT yahoo.com; ok dtucker@ | |
1096 | ||
8f02e0be | 1097 | 20080702 |
1098 | - (dtucker) OpenBSD CVS Sync | |
1099 | - djm@cvs.openbsd.org 2008/06/30 08:05:59 | |
1100 | [PROTOCOL.agent] | |
1101 | typo: s/constraint_date/constraint_data/ | |
fb5582f7 | 1102 | - djm@cvs.openbsd.org 2008/06/30 12:15:39 |
1103 | [serverloop.c] | |
1104 | only pass channel requests on session channels through to the session | |
1105 | channel handler, avoiding spurious log messages; ok! markus@ | |
4d92dbc1 | 1106 | - djm@cvs.openbsd.org 2008/06/30 12:16:02 |
1107 | [nchan.c] | |
1108 | only send eow@openssh.com notifications for session channels; ok! markus@ | |
8fb1ddc9 | 1109 | - djm@cvs.openbsd.org 2008/06/30 12:18:34 |
1110 | [PROTOCOL] | |
1111 | clarify that eow@openssh.com is only sent on session channels | |
979b31ed | 1112 | - dtucker@cvs.openbsd.org 2008/07/01 07:20:52 |
1113 | [sshconnect.c] | |
1114 | Check ExitOnForwardFailure if forwardings are disabled due to a failed | |
1115 | host key check. ok djm@ | |
f9b45eaf | 1116 | - dtucker@cvs.openbsd.org 2008/07/01 07:24:22 |
1117 | [sshconnect.c sshd.c] | |
1118 | Send CR LF during protocol banner exchanges, but only for Protocol 2 only, | |
1119 | in order to comply with RFC 4253. bz #1443, ok djm@ | |
5ebed98d | 1120 | - stevesk@cvs.openbsd.org 2008/07/01 23:12:47 |
1121 | [PROTOCOL.agent] | |
1122 | fix some typos; ok djm@ | |
39ceddb7 | 1123 | - djm@cvs.openbsd.org 2008/07/02 02:24:18 |
1124 | [sshd_config sshd_config.5 sshd.8 servconf.c] | |
1125 | increase default size of ssh protocol 1 ephemeral key from 768 to 1024 | |
1126 | bits; prodded by & ok dtucker@ ok deraadt@ | |
f7c2a004 | 1127 | - dtucker@cvs.openbsd.org 2008/07/02 12:03:51 |
1128 | [auth-rsa.c auth.c auth2-pubkey.c auth.h] | |
1129 | Merge duplicate host key file checks, based in part on a patch from Rob | |
1130 | Holland via bz #1348 . Also checks for non-regular files during protocol | |
1131 | 1 RSA auth. ok djm@ | |
221fc73c | 1132 | - djm@cvs.openbsd.org 2008/07/02 12:36:39 |
1133 | [auth2-none.c auth2.c] | |
1134 | Make protocol 2 MaxAuthTries behaviour a little more sensible: | |
1135 | Check whether client has exceeded MaxAuthTries before running | |
1136 | an authentication method and skip it if they have, previously it | |
1137 | would always allow one try (for "none" auth). | |
1138 | Preincrement failure count before post-auth test - previously this | |
1139 | checked and postincremented, also to allow one "none" try. | |
1140 | Together, these two changes always count the "none" auth method | |
1141 | which could be skipped by a malicious client (e.g. an SSH worm) | |
1142 | to get an extra attempt at a real auth method. They also make | |
1143 | MaxAuthTries=0 a useful way to block users entirely (esp. in a | |
1144 | sshd_config Match block). | |
1145 | Also, move sending of any preauth banner from "none" auth method | |
1146 | to the first call to input_userauth_request(), so worms that skip | |
1147 | the "none" method get to see it too. | |
8f02e0be | 1148 | |
00b7389d | 1149 | 20080630 |
1150 | - (djm) OpenBSD CVS Sync | |
1151 | - dtucker@cvs.openbsd.org 2008/06/10 23:13:43 | |
1152 | [regress/Makefile regress/key-options.sh] | |
1153 | Add regress test for key options. ok djm@ | |
86d745dc | 1154 | - dtucker@cvs.openbsd.org 2008/06/11 23:11:40 |
014f1b23 | 1155 | [regress/Makefile] |
86d745dc | 1156 | Don't run cipher-speed test by default; mistakenly enabled by me |
014f1b23 | 1157 | - djm@cvs.openbsd.org 2008/06/28 13:57:25 |
1158 | [regress/Makefile regress/test-exec.sh regress/conch-ciphers.sh] | |
1159 | very basic regress test against Twisted Conch in "make interop" | |
1160 | target (conch is available in ports/devel/py-twisted/conch); | |
1161 | ok markus@ | |
8476b024 | 1162 | - (djm) [regress/Makefile] search for conch by path, like we do putty |
00b7389d | 1163 | |
aa47edcc | 1164 | 20080629 |
1165 | - (djm) OpenBSD CVS Sync | |
1166 | - martynas@cvs.openbsd.org 2008/06/21 07:46:46 | |
1167 | [sftp.c] | |
1168 | use optopt to get invalid flag, instead of return value of getopt, | |
1169 | which is always '?'; ok djm@ | |
ccf0fcb6 | 1170 | - otto@cvs.openbsd.org 2008/06/25 11:13:43 |
1171 | [key.c] | |
1172 | add key length to visual fingerprint; zap magical constants; | |
1173 | ok grunk@ djm@ | |
681efe9f | 1174 | - djm@cvs.openbsd.org 2008/06/26 06:10:09 |
1175 | [sftp-client.c sftp-server.c] | |
1176 | allow the sftp chmod(2)-equivalent operation to set set[ug]id/sticky | |
1177 | bits. Note that this only affects explicit setting of modes (e.g. via | |
1178 | sftp(1)'s chmod command) and not file transfers. (bz#1310) | |
1179 | ok deraadt@ at c2k8 | |
b080d398 | 1180 | - djm@cvs.openbsd.org 2008/06/26 09:19:40 |
1181 | [dh.c dh.h moduli.c] | |
1182 | when loading moduli from /etc/moduli in sshd(8), check that they | |
1183 | are of the expected "safe prime" structure and have had | |
1184 | appropriate primality tests performed; | |
1185 | feedback and ok dtucker@ | |
7b3999b8 | 1186 | - grunk@cvs.openbsd.org 2008/06/26 11:46:31 |
1187 | [readconf.c readconf.h ssh.1 ssh_config.5 sshconnect.c] | |
1188 | Move SSH Fingerprint Visualization away from sharing the config option | |
1189 | CheckHostIP to an own config option named VisualHostKey. | |
1190 | While there, fix the behaviour that ssh would draw a random art picture | |
1191 | on every newly seen host even when the option was not enabled. | |
1192 | prodded by deraadt@, discussions, | |
1193 | help and ok markus@ djm@ dtucker@ | |
2e8d3306 | 1194 | - jmc@cvs.openbsd.org 2008/06/26 21:11:46 |
1195 | [ssh.1] | |
1196 | add VisualHostKey to the list of options listed in -o; | |
cda43f66 | 1197 | - djm@cvs.openbsd.org 2008/06/28 07:25:07 |
1198 | [PROTOCOL] | |
1199 | spelling fixes | |
c525650a | 1200 | - djm@cvs.openbsd.org 2008/06/28 13:58:23 |
1201 | [ssh-agent.c] | |
1202 | refuse to add a key that has unknown constraints specified; | |
1203 | ok markus | |
9ee2fb0e | 1204 | - djm@cvs.openbsd.org 2008/06/28 14:05:15 |
1205 | [ssh-agent.c] | |
1206 | reset global compat flag after processing a protocol 2 signature | |
1207 | request with the legacy DSA encoding flag set; ok markus | |
ab3eb078 | 1208 | - djm@cvs.openbsd.org 2008/06/28 14:08:30 |
1209 | [PROTOCOL PROTOCOL.agent] | |
1210 | document the protocol used by ssh-agent; "looks ok" markus@ | |
aa47edcc | 1211 | |
f6351d4d | 1212 | 20080628 |
1213 | - (djm) [RFC.nroff contrib/cygwin/Makefile contrib/suse/openssh.spec] | |
1214 | RFC.nroff lacks a license, remove it (it is long gone in OpenBSD). | |
1215 | ||
bd6b3feb | 1216 | 20080626 |
1217 | - (djm) [Makefile.in moduli.5] Include moduli(5) manpage from OpenBSD. | |
1218 | (bz#1372) | |
a32d8b38 | 1219 | - (djm) [ contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
1220 | [contrib/suse/openssh.spec] Include moduli.5 in RPM spec files. | |
bd6b3feb | 1221 | |
b3784859 | 1222 | 20080616 |
1223 | - (dtucker) OpenBSD CVS Sync | |
1224 | - dtucker@cvs.openbsd.org 2008/06/16 13:22:53 | |
1225 | [session.c channels.c] | |
1226 | Rename the isatty argument to is_tty so we don't shadow | |
1227 | isatty(3). ok markus@ | |
245f4d36 | 1228 | - (dtucker) [channels.c] isatty -> is_tty here too. |
b3784859 | 1229 | |
b55b0285 | 1230 | 20080615 |
1231 | - (dtucker) [configure.ac] Enable -fno-builtin-memset when using gcc. | |
081573fe | 1232 | - OpenBSD CVS Sync |
1233 | - dtucker@cvs.openbsd.org 2008/06/14 15:49:48 | |
1234 | [sshd.c] | |
1235 | wrap long line at 80 chars | |
26512357 | 1236 | - dtucker@cvs.openbsd.org 2008/06/14 17:07:11 |
1237 | [sshd.c] | |
1238 | ensure default umask disallows at least group and world write; ok djm@ | |
2608aa2b | 1239 | - djm@cvs.openbsd.org 2008/06/14 18:33:43 |
1240 | [session.c] | |
1241 | suppress the warning message from chdir(homedir) failures | |
1242 | when chrooted (bz#1461); ok dtucker | |
49c5f262 | 1243 | - dtucker@cvs.openbsd.org 2008/06/14 19:42:10 |
1244 | [scp.1] | |
1245 | Mention that scp follows symlinks during -r. bz #1466, | |
1246 | from nectar at apple | |
d97287d3 | 1247 | - dtucker@cvs.openbsd.org 2008/06/15 16:55:38 |
1248 | [sshd_config.5] | |
1249 | MaxSessions is allowed in a Match block too | |
8086aeb2 | 1250 | - dtucker@cvs.openbsd.org 2008/06/15 16:58:40 |
1251 | [servconf.c sshd_config.5] | |
1252 | Allow MaxAuthTries within a Match block. ok djm@ | |
c9478090 | 1253 | - djm@cvs.openbsd.org 2008/06/15 20:06:26 |
1254 | [channels.c channels.h session.c] | |
1255 | don't call isatty() on a pty master, instead pass a flag down to | |
1256 | channel_set_fds() indicating that te fds refer to a tty. Fixes a | |
1257 | hang on exit on Solaris (bz#1463) in portable but is actually | |
1258 | a generic bug; ok dtucker deraadt markus | |
b55b0285 | 1259 | |
add357c6 | 1260 | 20080614 |
1261 | - (djm) [openbsd-compat/sigact.c] Avoid NULL derefs in ancient sigaction | |
1262 | replacement code; patch from ighighi AT gmail.com in bz#1240; | |
1263 | ok dtucker | |
1264 | ||
849d3ceb | 1265 | 20080613 |
1266 | - (dtucker) OpenBSD CVS Sync | |
1267 | - deraadt@cvs.openbsd.org 2008/06/13 09:44:36 | |
1268 | [packet.c] | |
1269 | compile on older gcc; no decl after code | |
52ad6b9a | 1270 | - dtucker@cvs.openbsd.org 2008/06/13 13:56:59 |
1271 | [monitor.c] | |
1272 | Clear key options in the monitor on failed authentication, prevents | |
1273 | applying additional restrictions to non-pubkey authentications in | |
1274 | the case where pubkey fails but another method subsequently succeeds. | |
1275 | bz #1472, found by Colin Watson, ok markus@ djm@ | |
1d0b7aaa | 1276 | - dtucker@cvs.openbsd.org 2008/06/13 14:18:51 |
1277 | [auth2-pubkey.c auth-rhosts.c] | |
1278 | Include unistd.h for close(), prevents warnings in -portable | |
a3f13d60 | 1279 | - dtucker@cvs.openbsd.org 2008/06/13 17:21:20 |
1280 | [mux.c] | |
1281 | Friendlier error messages for mux fallback. ok djm@ | |
a15e7da1 | 1282 | - dtucker@cvs.openbsd.org 2008/06/13 18:55:22 |
1283 | [scp.c] | |
1284 | Prevent -Wsign-compare warnings on LP64 systems. bz #1192, ok deraadt@ | |
990ada29 | 1285 | - grunk@cvs.openbsd.org 2008/06/13 20:13:26 |
1286 | [ssh.1] | |
1287 | Explain the use of SSH fpr visualization using random art, and cite the | |
1288 | original scientific paper inspiring that technique. | |
1289 | Much help with English and nroff by jmc@, thanks. | |
596a825b | 1290 | - (dtucker) [configure.ac] Bug #1276: avoid linking against libgssapi, which |
1291 | despite its name doesn't seem to implement all of GSSAPI. Patch from | |
1292 | Jan Engelhardt, sanity checked by Simon Wilkinson. | |
849d3ceb | 1293 | |
9754b94b | 1294 | 20080612 |
1295 | - (dtucker) OpenBSD CVS Sync | |
1296 | - jmc@cvs.openbsd.org 2008/06/11 07:30:37 | |
1297 | [sshd.8] | |
1298 | kill trailing whitespace; | |
aff73c5f | 1299 | - grunk@cvs.openbsd.org 2008/06/11 21:01:35 |
1300 | [ssh_config.5 key.h readconf.c readconf.h ssh-keygen.1 ssh-keygen.c key.c | |
1301 | sshconnect.c] | |
1302 | Introduce SSH Fingerprint ASCII Visualization, a technique inspired by the | |
1303 | graphical hash visualization schemes known as "random art", and by | |
1304 | Dan Kaminsky's musings on the subject during a BlackOp talk at the | |
1305 | 23C3 in Berlin. | |
1306 | Scientific publication (original paper): | |
1307 | "Hash Visualization: a New Technique to improve Real-World Security", | |
1308 | Perrig A. and Song D., 1999, International Workshop on Cryptographic | |
1309 | Techniques and E-Commerce (CrypTEC '99) | |
1310 | http://sparrow.ece.cmu.edu/~adrian/projects/validation/validation.pdf | |
1311 | The algorithm used here is a worm crawling over a discrete plane, | |
1312 | leaving a trace (augmenting the field) everywhere it goes. | |
1313 | Movement is taken from dgst_raw 2bit-wise. Bumping into walls | |
1314 | makes the respective movement vector be ignored for this turn, | |
1315 | thus switching to the other color of the chessboard. | |
1316 | Graphs are not unambiguous for now, because circles in graphs can be | |
1317 | walked in either direction. | |
1318 | discussions with several people, | |
1319 | help, corrections and ok markus@ djm@ | |
93778882 | 1320 | - grunk@cvs.openbsd.org 2008/06/11 21:38:25 |
1321 | [ssh-keygen.c] | |
1322 | ssh-keygen -lv -f /etc/ssh/ssh_host_rsa_key.pub | |
1323 | would not display you the random art as intended, spotted by canacar@ | |
639211b7 | 1324 | - grunk@cvs.openbsd.org 2008/06/11 22:20:46 |
1325 | [ssh-keygen.c ssh-keygen.1] | |
1326 | ssh-keygen would write fingerprints to STDOUT, and random art to STDERR, | |
1327 | that is not how it was envisioned. | |
1328 | Also correct manpage saying that -v is needed along with -l for it to work. | |
1329 | spotted by naddy@ | |
e3115002 | 1330 | - otto@cvs.openbsd.org 2008/06/11 23:02:22 |
1331 | [key.c] | |
1332 | simpler way of computing the augmentations; ok grunk@ | |
fe88400f | 1333 | - grunk@cvs.openbsd.org 2008/06/11 23:03:56 |
1334 | [ssh_config.5] | |
1335 | CheckHostIP set to ``fingerprint'' will display both hex and random art | |
1336 | spotted by naddy@ | |
97841001 | 1337 | - grunk@cvs.openbsd.org 2008/06/11 23:51:57 |
1338 | [key.c] | |
1339 | #define statements that are not atoms need braces around them, else they | |
1340 | will cause trouble in some cases. | |
1341 | Also do a computation of -1 once, and not in a loop several times. | |
1342 | spotted by otto@ | |
e907df41 | 1343 | - dtucker@cvs.openbsd.org 2008/06/12 00:03:49 |
1344 | [dns.c canohost.c sshconnect.c] | |
1345 | Do not pass "0" strings as ports to getaddrinfo because the lookups | |
1346 | can slow things down and we never use the service info anyway. bz | |
1347 | #859, patch from YOSHIFUJI Hideaki and John Devitofranceschi. ok | |
1348 | deraadt@ djm@ | |
1349 | djm belives that the reason for the "0" strings is to ensure that | |
1350 | it's not possible to call getaddrinfo with both host and port being | |
1351 | NULL. In the case of canohost.c host is a local array. In the | |
1352 | case of sshconnect.c, it's checked for null immediately before use. | |
1353 | In dns.c it ultimately comes from ssh.c:main() and is guaranteed to | |
1354 | be non-null but it's not obvious, so I added a warning message in | |
1355 | case it is ever passed a null. | |
1356 | - grunk@cvs.openbsd.org 2008/06/12 00:13:55 | |
1357 | [sshconnect.c] | |
1358 | Make ssh print the random art also when ssh'ing to a host using IP only. | |
1359 | spotted by naddy@, ok and help djm@ dtucker@ | |
208cc0ee | 1360 | - otto@cvs.openbsd.org 2008/06/12 00:13:13 |
1361 | [key.c] | |
1362 | use an odd number of rows and columns and a separate start marker, looks | |
1363 | better; ok grunk@ | |
f17f705b | 1364 | - djm@cvs.openbsd.org 2008/06/12 03:40:52 |
1365 | [clientloop.h mux.c channels.c clientloop.c channels.h] | |
1366 | Enable ~ escapes for multiplex slave sessions; give each channel | |
1367 | its own escape state and hook the escape filters up to muxed | |
1368 | channels. bz #1331 | |
1369 | Mux slaves do not currently support the ~^Z and ~& escapes. | |
1370 | NB. this change cranks the mux protocol version, so a new ssh | |
1371 | mux client will not be able to connect to a running old ssh | |
1372 | mux master. | |
1373 | ok dtucker@ | |
72becb62 | 1374 | - djm@cvs.openbsd.org 2008/06/12 04:06:00 |
1375 | [clientloop.h ssh.c clientloop.c] | |
1376 | maintain an ordered queue of outstanding global requests that we | |
1377 | expect replies to, similar to the per-channel confirmation queue. | |
1378 | Use this queue to verify success or failure for remote forward | |
1379 | establishment in a race free way. | |
1380 | ok dtucker@ | |
344f1d3d | 1381 | - djm@cvs.openbsd.org 2008/06/12 04:17:47 |
1382 | [clientloop.c] | |
1383 | thall shalt not code past the eightieth column | |
e8097dc9 | 1384 | - djm@cvs.openbsd.org 2008/06/12 04:24:06 |
1385 | [ssh.c] | |
1386 | thal shalt not code past the eightieth column | |
9bcf03ce | 1387 | - djm@cvs.openbsd.org 2008/06/12 05:15:41 |
1388 | [PROTOCOL] | |
1389 | document tun@openssh.com forwarding method | |
aacab402 | 1390 | - djm@cvs.openbsd.org 2008/06/12 05:32:30 |
1391 | [mux.c] | |
1392 | some more TODO for me | |
2bb50d23 | 1393 | - grunk@cvs.openbsd.org 2008/06/12 05:42:46 |
1394 | [key.c] | |
1395 | supply the key type (rsa1, rsa, dsa) as a caption in the frame of the | |
1396 | random art. while there, stress the fact that the field base should at | |
1397 | least be 8 characters for the pictures to make sense. | |
1398 | comment and ok djm@ | |
1399 | - grunk@cvs.openbsd.org 2008/06/12 06:32:59 | |
1400 | [key.c] | |
1401 | We already mark the start of the worm, now also mark the end of the worm | |
1402 | in our random art drawings. | |
1403 | ok djm@ | |
e74caf1e | 1404 | - djm@cvs.openbsd.org 2008/06/12 15:19:17 |
1405 | [clientloop.h channels.h clientloop.c channels.c mux.c] | |
1406 | The multiplexing escape char handler commit last night introduced a | |
1407 | small memory leak per session; plug it. | |
e9d0b573 | 1408 | - dtucker@cvs.openbsd.org 2008/06/12 16:35:31 |
1409 | [ssh_config.5 ssh.c] | |
1410 | keyword expansion for localcommand. ok djm@ | |
a64f8307 | 1411 | - jmc@cvs.openbsd.org 2008/06/12 19:10:09 |
1412 | [ssh_config.5 ssh-keygen.1] | |
1413 | tweak the ascii art text; ok grunk | |
bc2d97c8 | 1414 | - dtucker@cvs.openbsd.org 2008/06/12 20:38:28 |
1415 | [sshd.c sshconnect.c packet.h misc.c misc.h packet.c] | |
1416 | Make keepalive timeouts apply while waiting for a packet, particularly | |
1417 | during key renegotiation (bz #1363). With djm and Matt Day, ok djm@ | |
ad39a852 | 1418 | - djm@cvs.openbsd.org 2008/06/12 20:47:04 |
1419 | [sftp-client.c] | |
1420 | print extension revisions for extensions that we understand | |
07d8d480 | 1421 | - djm@cvs.openbsd.org 2008/06/12 21:06:25 |
1422 | [clientloop.c] | |
1423 | I was coalescing expected global request confirmation replies at | |
1424 | the wrong end of the queue - fix; prompted by markus@ | |
31de76cc | 1425 | - grunk@cvs.openbsd.org 2008/06/12 21:14:46 |
1426 | [ssh-keygen.c] | |
1427 | make ssh-keygen -lf show the key type just as ssh-add -l would do it | |
1428 | ok djm@ markus@ | |
f97fb6ca | 1429 | - grunk@cvs.openbsd.org 2008/06/12 22:03:36 |
1430 | [key.c] | |
1431 | add my copyright, ok djm@ | |
6d8216ff | 1432 | - ian@cvs.openbsd.org 2008/06/12 23:24:58 |
1433 | [sshconnect.c] | |
1434 | tweak wording in message, ok deraadt@ jmc@ | |
2c83cd01 | 1435 | - dtucker@cvs.openbsd.org 2008/06/13 00:12:02 |
1436 | [sftp.h log.h] | |
1437 | replace __dead with __attribute__((noreturn)), makes things | |
1438 | a little easier to port. Also, add it to sigdie(). ok djm@ | |
b97ea6eb | 1439 | - djm@cvs.openbsd.org 2008/06/13 00:16:49 |
1440 | [mux.c] | |
1441 | fall back to creating a new TCP connection on most multiplexing errors | |
1442 | (socket connect fail, invalid version, refused permittion, corrupted | |
1443 | messages, etc.); bz #1329 ok dtucker@ | |
243cc316 | 1444 | - dtucker@cvs.openbsd.org 2008/06/13 00:47:53 |
1445 | [mux.c] | |
1446 | upcast size_t to u_long to match format arg; ok djm@ | |
041f11dc | 1447 | - dtucker@cvs.openbsd.org 2008/06/13 00:51:47 |
1448 | [mac.c] | |
1449 | upcast another size_t to u_long to match format | |
852eb76b | 1450 | - dtucker@cvs.openbsd.org 2008/06/13 01:38:23 |
1451 | [misc.c] | |
1452 | upcast uid to long with matching %ld, prevents warnings in portable | |
632f2669 | 1453 | - djm@cvs.openbsd.org 2008/06/13 04:40:22 |
1454 | [auth2-pubkey.c auth-rhosts.c] | |
1455 | refuse to read ~/.shosts or ~/.ssh/authorized_keys that are not | |
1456 | regular files; report from Solar Designer via Colin Watson in bz#1471 | |
1457 | ok dtucker@ deraadt | |
136d0181 | 1458 | - (dtucker) [clientloop.c serverloop.c] channel_register_filter now |
1459 | takes 2 more args. with djm@ | |
49190c3d | 1460 | - (dtucker) [defines.h] Bug #1112: __dead is, well dead. Based on a patch |
1461 | from Todd Vierling. | |
02e605ed | 1462 | - (dtucker) [auth-sia.c] Bug #1241: support password expiry on Tru64 SIA |
1463 | systems. Patch from R. Scott Bailey. | |
c694c610 | 1464 | - (dtucker) [umac.c] STORE_UINT32_REVERSED and endian_convert are never used |
1465 | on big endian machines, so ifdef them for little-endian only to prevent | |
1466 | unused function warnings on big-endians. | |
56f77432 | 1467 | - (dtucker) [openbsd-compat/setenv.c] Make offsets size_t to prevent |
1468 | compiler warnings on some platforms. Based on a discussion with otto@ | |
9754b94b | 1469 | |
554ebbed | 1470 | 20080611 |
1471 | - (djm) [channels.c configure.ac] | |
1472 | Do not set SO_REUSEADDR on wildcard X11 listeners (X11UseLocalhost=no) | |
1473 | bz#1464; ok dtucker | |
1474 | ||
15b5fa9b | 1475 | 20080610 |
1476 | - (dtucker) OpenBSD CVS Sync | |
1477 | - djm@cvs.openbsd.org 2008/06/10 03:57:27 | |
1478 | [servconf.c match.h sshd_config.5] | |
1479 | support CIDR address matching in sshd_config "Match address" blocks, with | |
1480 | full support for negation and fall-back to classic wildcard matching. | |
1481 | For example: | |
1482 | Match address 192.0.2.0/24,3ffe:ffff::/32,!10.* | |
1483 | PasswordAuthentication yes | |
1484 | addrmatch.c code mostly lifted from flowd's addr.c | |
1485 | feedback and ok dtucker@ | |
8b671558 | 1486 | - djm@cvs.openbsd.org 2008/06/10 04:17:46 |
1487 | [sshd_config.5] | |
1488 | better reference for pattern-list | |
1760c982 | 1489 | - dtucker@cvs.openbsd.org 2008/06/10 04:50:25 |
1490 | [sshd.c channels.h channels.c log.c servconf.c log.h servconf.h sshd.8] | |
1491 | Add extended test mode (-T) and connection parameters for test mode (-C). | |
1492 | -T causes sshd to write its effective configuration to stdout and exit. | |
1493 | -C causes any relevant Match rules to be applied before output. The | |
1494 | combination allows tesing of the parser and config files. ok deraadt djm | |
01e9e424 | 1495 | - jmc@cvs.openbsd.org 2008/06/10 07:12:00 |
1496 | [sshd_config.5] | |
1497 | tweak previous; | |
3b42e3ac | 1498 | - jmc@cvs.openbsd.org 2008/06/10 08:17:40 |
1499 | [sshd.8 sshd.c] | |
1500 | - update usage() | |
1501 | - fix SYNOPSIS, and sort options | |
1502 | - some minor additional fixes | |
f0528444 | 1503 | - dtucker@cvs.openbsd.org 2008/06/09 18:06:32 |
1504 | [regress/test-exec.sh] | |
1505 | Don't generate putty keys if we're not going to use them. ok djm | |
16d46c30 | 1506 | - dtucker@cvs.openbsd.org 2008/06/10 05:23:32 |
1507 | [regress/addrmatch.sh regress/Makefile] | |
1508 | Regress test for Match CIDR rules. ok djm@ | |
94edc013 | 1509 | - dtucker@cvs.openbsd.org 2008/06/10 15:21:41 |
1510 | [test-exec.sh] | |
1511 | Use a more portable construct for checking if we're running a putty test | |
64c576e9 | 1512 | - dtucker@cvs.openbsd.org 2008/06/10 15:28:49 |
1513 | [test-exec.sh] | |
1514 | Add quotes | |
f6748d7b | 1515 | - dtucker@cvs.openbsd.org 2008/06/10 18:21:24 |
1516 | [ssh_config.5] | |
1517 | clarify that Host patterns are space-separated. ok deraadt | |
3f0444ca | 1518 | - djm@cvs.openbsd.org 2008/06/10 22:15:23 |
1519 | [PROTOCOL ssh.c serverloop.c] | |
1520 | Add a no-more-sessions@openssh.com global request extension that the | |
1521 | client sends when it knows that it will never request another session | |
1522 | (i.e. when session multiplexing is disabled). This allows a server to | |
1523 | disallow further session requests and terminate the session. | |
1524 | Why would a non-multiplexing client ever issue additional session | |
1525 | requests? It could have been attacked with something like SSH'jack: | |
1526 | http://www.storm.net.nz/projects/7 | |
1527 | feedback & ok markus | |
b3b048d6 | 1528 | - djm@cvs.openbsd.org 2008/06/10 23:06:19 |
1529 | [auth-options.c match.c servconf.c addrmatch.c sshd.8] | |
1530 | support CIDR address matching in .ssh/authorized_keys from="..." stanzas | |
1531 | ok and extensive testing dtucker@ | |
8fb12ef0 | 1532 | - dtucker@cvs.openbsd.org 2008/06/10 23:21:34 |
1533 | [bufaux.c] | |
1534 | Use '\0' for a nul byte rather than unadorned 0. ok djm@ | |
a6d05adf | 1535 | - dtucker@cvs.openbsd.org 2008/06/10 23:13:43 |
1536 | [Makefile regress/key-options.sh] | |
1537 | Add regress test for key options. ok djm@ | |
edee47f5 | 1538 | - (dtucker) [openbsd-compat/fake-rfc2553.h] Add sin6_scope_id to sockaddr_in6 |
1539 | since the new CIDR code in addmatch.c references it. | |
1540 | - (dtucker) [Makefile.in configure.ac regress/addrmatch.sh] Skip IPv6 | |
1541 | specific tests on platforms that don't do IPv6. | |
8ac1d2eb | 1542 | - (dtucker) [Makefile.in] Define TEST_SSH_IPV6 in make's arguments as well |
1543 | as environment. | |
0694c78f | 1544 | - (dtucker) [Makefile.in] Move addrmatch.o to libssh.a where it's needed now. |
15b5fa9b | 1545 | |
10e804f4 | 1546 | 20080609 |
1547 | - (dtucker) OpenBSD CVS Sync | |
1548 | - dtucker@cvs.openbsd.org 2008/06/08 17:04:41 | |
1549 | [sftp-server.c] | |
1550 | Add case for ENOSYS in errno_to_portable; ok deraadt | |
5a3cde15 | 1551 | - dtucker@cvs.openbsd.org 2008/06/08 20:15:29 |
1552 | [sftp.c sftp-client.c sftp-client.h] | |
1553 | Have the sftp client store the statvfs replies in wire format, | |
1554 | which prevents problems when the server's native sizes exceed the | |
1555 | client's. | |
1556 | Also extends the sizes of the remaining 32bit wire format to 64bit, | |
1557 | they're specified as unsigned long in the standard. | |
7290afcb | 1558 | - dtucker@cvs.openbsd.org 2008/06/09 13:02:39 |
2626070f | 1559 | [sftp-server.c] |
7290afcb | 1560 | Extend 32bit -> 64bit values for statvfs extension missed in previous |
1561 | commit. | |
2626070f | 1562 | - dtucker@cvs.openbsd.org 2008/06/09 13:38:46 |
1563 | [PROTOCOL] | |
1564 | Use a $OpenBSD tag so our scripts will sync changes. | |
10e804f4 | 1565 | |
22f5e872 | 1566 | 20080608 |
1567 | - (dtucker) [configure.ac defines.h sftp-client.c sftp-server.c sftp.c | |
1568 | openbsd-compat/Makefile.in openbsd-compat/openbsd-compat.h | |
1569 | openbsd-compat/bsd-statvfs.{c,h}] Add a null implementation of statvfs and | |
1570 | fstatvfs and remove #defines around statvfs code. ok djm@ | |
7a4f468b | 1571 | - (dtucker) [configure.ac defines.h sftp-client.c M sftp-server.c] Add a |
1572 | macro to convert fsid to unsigned long for platforms where fsid is a | |
1573 | 2-member array. | |
22f5e872 | 1574 | |
0894bbed | 1575 | 20080607 |
1576 | - (dtucker) [mux.c] Include paths.h inside ifdef HAVE_PATHS_H. | |
4538e135 | 1577 | - (dtucker) [configure.ac defines.h sftp-client.c sftp-server.c sftp.c] |
1578 | Do not enable statvfs extensions on platforms that do not have statvfs. | |
2abb1ef5 | 1579 | - (dtucker) OpenBSD CVS Sync |
1580 | - djm@cvs.openbsd.org 2008/05/19 06:14:02 | |
1581 | [packet.c] unbreak protocol keepalive timeouts bz#1465; ok dtucker@ | |
82bb6f20 | 1582 | - djm@cvs.openbsd.org 2008/05/19 15:45:07 |
1583 | [sshtty.c ttymodes.c sshpty.h] | |
1584 | Fix sending tty modes when stdin is not a tty (bz#1199). Previously | |
1585 | we would send the modes corresponding to a zeroed struct termios, | |
1586 | whereas we should have been sending an empty list of modes. | |
1587 | Based on patch from daniel.ritz AT alcatel.ch; ok dtucker@ markus@ | |
048acbeb | 1588 | - djm@cvs.openbsd.org 2008/05/19 15:46:31 |
1589 | [ssh-keygen.c] | |
1590 | support -l (print fingerprint) in combination with -F (find host) to | |
1591 | search for a host in ~/.ssh/known_hosts and display its fingerprint; | |
1592 | ok markus@ | |
4651c790 | 1593 | - djm@cvs.openbsd.org 2008/05/19 20:53:52 |
1594 | [clientloop.c] | |
1595 | unbreak tree by committing this bit that I missed from: | |
1596 | Fix sending tty modes when stdin is not a tty (bz#1199). Previously | |
1597 | we would send the modes corresponding to a zeroed struct termios, | |
1598 | whereas we should have been sending an empty list of modes. | |
1599 | Based on patch from daniel.ritz AT alcatel.ch; ok dtucker@ markus@ | |
0894bbed | 1600 | |
07e61b8a | 1601 | 20080604 |
1602 | - (djm) [openbsd-compat/bsd-arc4random.c] Fix math bug that caused bias | |
1603 | in arc4random_uniform with upper_bound in (2^30,2*31). Note that | |
1604 | OpenSSH did not make requests with upper bounds in this range. | |
1605 | ||
b3ef88dc | 1606 | 20080519 |
1607 | - (djm) [configure.ac mux.c sftp.c openbsd-compat/Makefile.in] | |
1608 | [openbsd-compat/fmt_scaled.c openbsd-compat/openbsd-compat.h] | |
1609 | Fix compilation on Linux, including pulling in fmt_scaled(3) | |
1610 | implementation from OpenBSD's libutil. | |
1611 | ||
9b04dbaa | 1612 | 20080518 |
1613 | - (djm) OpenBSD CVS Sync | |
1614 | - djm@cvs.openbsd.org 2008/04/04 05:14:38 | |
1615 | [sshd_config.5] | |
1616 | ChrootDirectory is supported in Match blocks (in fact, it is most useful | |
1617 | there). Spotted by Minstrel AT minstrel.org.uk | |
5b76e3ef | 1618 | - djm@cvs.openbsd.org 2008/04/04 06:44:26 |
1619 | [sshd_config.5] | |
1620 | oops, some unrelated stuff crept into that commit - backout. | |
1621 | spotted by jmc@ | |
ade21243 | 1622 | - djm@cvs.openbsd.org 2008/04/05 02:46:02 |
1623 | [sshd_config.5] | |
1624 | HostbasedAuthentication is supported under Match too | |
185adaf8 | 1625 | - (djm) [openbsd-compat/bsd-arc4random.c openbsd-compat/openbsd-compat.c] |
1626 | [configure.ac] Implement arc4random_buf(), import implementation of | |
1627 | arc4random_uniform() from OpenBSD | |
936e7c8c | 1628 | - (djm) [openbsd-compat/bsd-arc4random.c] Warning fixes |
c49ce62e | 1629 | - (djm) [openbsd-compat/port-tun.c] needs sys/queue.h |
c1d152b8 | 1630 | - (djm) OpenBSD CVS Sync |
1631 | - djm@cvs.openbsd.org 2008/04/13 00:22:17 | |
1632 | [dh.c sshd.c] | |
1633 | Use arc4random_buf() when requesting more than a single word of output | |
1634 | Use arc4random_uniform() when the desired random number upper bound | |
1635 | is not a power of two | |
1636 | ok deraadt@ millert@ | |
360b43ab | 1637 | - djm@cvs.openbsd.org 2008/04/18 12:32:11 |
1638 | [sftp-client.c sftp-client.h sftp-server.c sftp.1 sftp.c sftp.h] | |
1639 | introduce sftp extension methods statvfs@openssh.com and | |
1640 | fstatvfs@openssh.com that implement statvfs(2)-like operations, | |
1641 | based on a patch from miklos AT szeredi.hu (bz#1399) | |
1642 | also add a "df" command to the sftp client that uses the | |
1643 | statvfs@openssh.com to produce a df(1)-like display of filesystem | |
1644 | space and inode utilisation | |
1645 | ok markus@ | |
ea530517 | 1646 | - jmc@cvs.openbsd.org 2008/04/18 17:15:47 |
1647 | [sftp.1] | |
1648 | macro fixage; | |
48fbfda0 | 1649 | - djm@cvs.openbsd.org 2008/04/18 22:01:33 |
1650 | [session.c] | |
1651 | remove unneccessary parentheses | |
0bb7755b | 1652 | - otto@cvs.openbsd.org 2008/04/29 11:20:31 |
1653 | [monitor_mm.h] | |
1654 | garbage collect two unused fields in struct mm_master; ok markus@ | |
c47ff7a6 | 1655 | - djm@cvs.openbsd.org 2008/04/30 10:14:03 |
1656 | [ssh-keyscan.1 ssh-keyscan.c] | |
1657 | default to rsa (protocol 2) keys, instead of rsa1 keys; spotted by | |
1658 | larsnooden AT openoffice.org | |
43c3f85c | 1659 | - pyr@cvs.openbsd.org 2008/05/07 05:49:37 |
1660 | [servconf.c servconf.h session.c sshd_config.5] | |
1661 | Enable the AllowAgentForwarding option in sshd_config (global and match | |
1662 | context), to specify if agents should be permitted on the server. | |
1663 | As the man page states: | |
1664 | ``Note that disabling Agent forwarding does not improve security | |
1665 | unless users are also denied shell access, as they can always install | |
1666 | their own forwarders.'' | |
1667 | ok djm@, ok and a mild frown markus@ | |
5c7e2b47 | 1668 | - pyr@cvs.openbsd.org 2008/05/07 06:43:35 |
1669 | [sshd_config] | |
1670 | push the sshd_config bits in, spotted by ajacoutot@ | |
94569631 | 1671 | - jmc@cvs.openbsd.org 2008/05/07 08:00:14 |
1672 | [sshd_config.5] | |
1673 | sort; | |
17f02f0a | 1674 | - markus@cvs.openbsd.org 2008/05/08 06:59:01 |
1675 | [bufaux.c buffer.h channels.c packet.c packet.h] | |
1676 | avoid extra malloc/copy/free when receiving data over the net; | |
1677 | ~10% speedup for localhost-scp; ok djm@ | |
3593bdc0 | 1678 | - djm@cvs.openbsd.org 2008/05/08 12:02:23 |
1679 | [auth-options.c auth1.c channels.c channels.h clientloop.c gss-serv.c] | |
1680 | [monitor.c monitor_wrap.c nchan.c servconf.c serverloop.c session.c] | |
1681 | [ssh.c sshd.c] | |
1682 | Implement a channel success/failure status confirmation callback | |
1683 | mechanism. Each channel maintains a queue of callbacks, which will | |
1684 | be drained in order (RFC4253 guarantees confirm messages are not | |
1685 | reordered within an channel). | |
1686 | Also includes a abandonment callback to clean up if a channel is | |
1687 | closed without sending confirmation messages. This probably | |
1688 | shouldn't happen in compliant implementations, but it could be | |
1689 | abused to leak memory. | |
1690 | ok markus@ (as part of a larger diff) | |
c6dca55e | 1691 | - djm@cvs.openbsd.org 2008/05/08 12:21:16 |
1692 | [monitor.c monitor_wrap.c session.h servconf.c servconf.h session.c] | |
1693 | [sshd_config sshd_config.5] | |
1694 | Make the maximum number of sessions run-time controllable via | |
1695 | a sshd_config MaxSessions knob. This is useful for disabling | |
1696 | login/shell/subsystem access while leaving port-forwarding working | |
1697 | (MaxSessions 0), disabling connection multiplexing (MaxSessions 1) or | |
1698 | simply increasing the number of allows multiplexed sessions. | |
1699 | Because some bozos are sure to configure MaxSessions in excess of the | |
1700 | number of available file descriptors in sshd (which, at peak, might be | |
1701 | as many as 9*MaxSessions), audit sshd to ensure that it doesn't leak fds | |
1702 | on error paths, and make it fail gracefully on out-of-fd conditions - | |
1703 | sending channel errors instead of than exiting with fatal(). | |
1704 | bz#1090; MaxSessions config bits and manpage from junyer AT gmail.com | |
1705 | ok markus@ | |
95d3c124 | 1706 | - djm@cvs.openbsd.org 2008/05/08 13:06:11 |
1707 | [clientloop.c clientloop.h ssh.c] | |
1708 | Use new channel status confirmation callback system to properly deal | |
1709 | with "important" channel requests that fail, in particular command exec, | |
1710 | shell and subsystem requests. Previously we would optimistically assume | |
1711 | that the requests would always succeed, which could cause hangs if they | |
1712 | did not (e.g. when the server runs out of fds) or were unimplemented by | |
1713 | the server (bz #1384) | |
1714 | Also, properly report failing multiplex channel requests via the mux | |
1715 | client stderr (subject to LogLevel in the mux master) - better than | |
1716 | silently failing. | |
1717 | most bits ok markus@ (as part of a larger diff) | |
e07e21ad | 1718 | - djm@cvs.openbsd.org 2008/05/09 04:55:56 |
1719 | [channels.c channels.h clientloop.c serverloop.c] | |
1720 | Try additional addresses when connecting to a port forward destination | |
1721 | whose DNS name resolves to more than one address. The previous behaviour | |
1722 | was to try the first address and give up. | |
1723 | Reported by stig AT venaas.com in bz#343 | |
1724 | great feedback and ok markus@ | |
3bcced4c | 1725 | - djm@cvs.openbsd.org 2008/05/09 14:18:44 |
1726 | [clientloop.c clientloop.h ssh.c mux.c] | |
1727 | tidy up session multiplexing code, moving it into its own file and | |
1728 | making the function names more consistent - making ssh.c and | |
1729 | clientloop.c a fair bit more readable. | |
1730 | ok markus@ | |
6cd3e678 | 1731 | - djm@cvs.openbsd.org 2008/05/09 14:26:08 |
1732 | [ssh.c] | |
1733 | dingo stole my diff hunk | |
ee7c3e92 | 1734 | - markus@cvs.openbsd.org 2008/05/09 16:16:06 |
1735 | [session.c] | |
1736 | re-add the USE_PIPES code and enable it. | |
1737 | without pipes shutdown-read from the sshd does not trigger | |
1738 | a SIGPIPE when the forked program does a write. | |
1739 | ok djm@ | |
1740 | (Id sync only, USE_PIPES never left portable OpenSSH) | |
271f4a13 | 1741 | - markus@cvs.openbsd.org 2008/05/09 16:17:51 |
1742 | [channels.c] | |
1743 | error-fd race: don't enable the error fd in the select bitmask | |
1744 | for channels with both in- and output closed, since the channel | |
1745 | will go away before we call select(); | |
1746 | report, lots of debugging help and ok djm@ | |
50c96367 | 1747 | - markus@cvs.openbsd.org 2008/05/09 16:21:13 |
1748 | [channels.h clientloop.c nchan.c serverloop.c] | |
1749 | unbreak | |
1750 | ssh -2 localhost od /bin/ls | true | |
1751 | ignoring SIGPIPE by adding a new channel message (EOW) that signals | |
1752 | the peer that we're not interested in any data it might send. | |
1753 | fixes bz #85; discussion, debugging and ok djm@ | |
d5820099 | 1754 | - pvalchev@cvs.openbsd.org 2008/05/12 20:52:20 |
1755 | [umac.c] | |
1756 | Ensure nh_result lies on a 64-bit boundary (fixes warnings observed | |
1757 | on Itanium on Linux); from Dale Talcott (bug #1462); ok djm@ | |
56b12440 | 1758 | - djm@cvs.openbsd.org 2008/05/15 23:52:24 |
1759 | [nchan2.ms] | |
1760 | document eow message in ssh protocol 2 channel state machine; | |
1761 | feedback and ok markus@ | |
f8db3345 | 1762 | - djm@cvs.openbsd.org 2008/05/18 21:29:05 |
1763 | [sftp-server.c] | |
1764 | comment extension announcement | |
8be03682 | 1765 | - djm@cvs.openbsd.org 2008/05/16 08:30:42 |
1766 | [PROTOCOL] | |
1767 | document our protocol extensions and deviations; ok markus@ | |
1768 | - djm@cvs.openbsd.org 2008/05/17 01:31:56 | |
1769 | [PROTOCOL] | |
1770 | grammar and correctness fixes from stevesk@ | |
9b04dbaa | 1771 | |
490c3105 | 1772 | 20080403 |
1773 | - (djm) [openbsd-compat/bsd-poll.c] Include stdlib.h to avoid compile- | |
1774 | time warnings on LynxOS. Patch from ops AT iki.fi | |
1ebb73e4 | 1775 | - (djm) Force string arguments to replacement setproctitle() though |
1776 | strnvis first. Ok dtucker@ | |
490c3105 | 1777 | |
2b363e83 | 1778 | 20080403 |
1779 | - (djm) OpenBSD CVS sync: | |
1780 | - markus@cvs.openbsd.org 2008/04/02 15:36:51 | |
1781 | [channels.c] | |
1782 | avoid possible hijacking of x11-forwarded connections (back out 1.183) | |
1783 | CVE-2008-1483; ok djm@ | |
adb7acbc | 1784 | - jmc@cvs.openbsd.org 2008/03/27 22:37:57 |
1785 | [sshd.8] | |
1786 | remove trailing whitespace; | |
53e0dc70 | 1787 | - djm@cvs.openbsd.org 2008/04/03 09:50:14 |
1788 | [version.h] | |
1789 | openssh-5.0 | |
31b1b2c8 | 1790 | - (djm) [contrib/caldera/openssh.spec contrib/redhat/openssh.spec] |
1791 | [contrib/suse/openssh.spec] Crank version numbers in RPM spec files | |
dd052df9 | 1792 | - (djm) [README] Update link to release notes |
098ebea7 | 1793 | - (djm) Release 5.0p1 |