Oops, forgot to credit Ahmad Khalifa of arabeyes.org for the bidi/shaping work.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5612 cda61777-01e9-0310-a592-d414129be87e

Add keyboard accelerators for IPv4/IPv6 selections options.
(Also for `window_border'.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5610 cda61777-01e9-0310-a592-d414129be87e

If a new session was saved from Change Settings, a side-effect on Windows was
that the global `sesslist' got out of sync with the saved-sessions submenu,
causing the latter to launch the wrong sessions.

Also, Change Settings wasn't getting a fresh session list, so if the set of
sessions had changed since session startup it wouldn't reflect that (at least
until a session was saved). Fixed (on all platforms).

Therefore, since the global sesslist didn't seem to be useful, I've got rid
of it; config.c creates one as needed, as do the frontends. (Not tried
compiling Mac changes.)

Also, we now build the saved-sessions submenu on demand on Windows and Unix.
(This should probably also be done on the Mac.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5609 cda61777-01e9-0310-a592-d414129be87e

In get_sesslist(), when freeing, set freed members to NULL on general
principles.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5608 cda61777-01e9-0310-a592-d414129be87e

Remove support for the "rijndael256-cbc", "rijndael192-cbc", and
"rijndael128-cbc" names for AES.  These are in the IANA namespace, but
never appeared in any secsh-transport draft, and no version of OpenSSH
has supported them without also supporting the aes*-cbc names.

"rijndael-cbc@lysator.liu.se" gets to live because it's in the private
namespace.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5607 cda61777-01e9-0310-a592-d414129be87e

Implement SDCTR modes, as defined in the newmodes draft.  This adds
aes128-ctr, aes192-ctr, and aes256-ctr.  blowfish-ctr and 3des-ctr are
present but disabled, since I haven't tested them yet.

In addition, change the user-visible names of ciphers (as displayed in the
Event Log) to include the mode name and, in Blowfish's case, the key size.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5605 cda61777-01e9-0310-a592-d414129be87e

`xp-wont-run': Apparently my application-manifest trims of r5534 aren't
acceptable on all versions of XP. Bah. Revert to pre-r5534 format (but
keep version number as 0.0.0.0). People who've had this problem have
reported putty.mft to make it go away.

NB, putting these updated manifests alongside the executable (e.g. as
`putty.exe.manifest') is also reported to work.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5604 cda61777-01e9-0310-a592-d414129be87e

Version number and other tweakings prior to the 0.58 release.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5598 cda61777-01e9-0310-a592-d414129be87e

tweak wording in "pscp -ls"

git-svn-id: svn://svn.tartarus.org/sgt/putty@5597 cda61777-01e9-0310-a592-d414129be87e

add XXX-REMOVE-BEFORE-RELEASE re uninstaller

git-svn-id: svn://svn.tartarus.org/sgt/putty@5596 cda61777-01e9-0310-a592-d414129be87e

Add target to build `info' file.
We probably already require a new enough version of Halibut that this isn't
a problem; nevertheless, I've put it in a separate target for now.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5595 cda61777-01e9-0310-a592-d414129be87e

Add versionid for index.
Also plumb in pgpkeys.but versionid.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5594 cda61777-01e9-0310-a592-d414129be87e

Index the complete PuTTY manual.
This was a bit rushed, and could doubtless be improved.
Also fix a couple of things I noted on the way, including:
 - "pscp -ls" wasn't documented
 - Windows XP wasn't mentioned enough

git-svn-id: svn://svn.tartarus.org/sgt/putty@5593 cda61777-01e9-0310-a592-d414129be87e

NULL needs to be cast to void * when passed to a variadic function like
execl().  Spotted by Damien Miller.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5592 cda61777-01e9-0310-a592-d414129be87e

#ifdef out expensive combining-character/scrollback diagnostics (for now).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5589 cda61777-01e9-0310-a592-d414129be87e

sk_address_is_local() was ignoring the possibility that a SockAddr
might have an IPv4 address in `ai' rather than in `addresses'.
Thanks to Martin Prikryl for pointing this out.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5587 cda61777-01e9-0310-a592-d414129be87e

Add text to the troubleshooting chapter about the errors that can be
caused when an active connection times out due to outgoing data
exceeding its maximum number of retries, and mention that this can
occur even when you didn't think you'd sent anything due to rekeys
and/or keepalives.

Unix generates ETIMEDOUT in this situation. Windows, it turns out
after doing an actual experiment by disabling my firewall, generates
ECONNABORTED! So _that's_ what it means under Windows. I wish I'd
done this experiment years ago now.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5585 cda61777-01e9-0310-a592-d414129be87e

FAQ entry on rekeys: Jacob would like to pre-emptively try to avoid
the possibility of people falling back to SSH-1 just because it gets
rid of the irritating delays.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5584 cda61777-01e9-0310-a592-d414129be87e

Beef up the `unfinished and experimental' warnings for the two Mac
ports, mentioning in particular that even if you downloaded a
_release_ source archive these particular ports are not considered
to be of release quality.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5583 cda61777-01e9-0310-a592-d414129be87e

XXX-REMOVE-BEFORE-RELEASE: check docs too

git-svn-id: svn://svn.tartarus.org/sgt/putty@5574 cda61777-01e9-0310-a592-d414129be87e

Further fix for lcc-win32. The PuTTY suite now compiles fine for me
using lcc-win32 v3.8 (compilation date Mar 2 2005 18:40:17) provided
I pass COMPAT="-DNO_IPV6 -DNO_MULTIMON" on the command line.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5573 cda61777-01e9-0310-a592-d414129be87e

faq-timeout: update for XP

git-svn-id: svn://svn.tartarus.org/sgt/putty@5565 cda61777-01e9-0310-a592-d414129be87e

<1112163342.018492.101240@f14g2000cwb.googlegroups.com> on
comp.security.ssh contains a Dr Watson log which looks to me as if
`unclean-close-crash' occurred due to a rekey timer going off after
the session had closed. Hence, ssh2_timer() now avoids doing
anything if the session is already closed, and also ssh_do_close()
proactively calls expire_timer_context(). Between those I think they
ought to solve the problem.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5564 cda61777-01e9-0310-a592-d414129be87e

Small markup fixes I just noticed, mostly use of literal double
quotes where \q{} would be better.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5562 cda61777-01e9-0310-a592-d414129be87e

After we receive EOF on stdin, we should clear ssh->send_ok so that
we stop trying to read anything further from stdin. Otherwise we
send a continuous stream of SSH2_MSG_CHANNEL_EOF.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5561 cda61777-01e9-0310-a592-d414129be87e

Render timing.c robust in the face of strangeness. The strangenesses
in question vary per OS: on Windows the problem is that WM_TIMER
sometimes goes off too early, so that GetTickCount() is right and
the callback time is wrong, whereas on Unix the problem is that my
GETTICKCOUNT implementation comes from the system clock which means
it can change suddenly and non-monotonically if the sysadmin is
messing about (meaning that the timing of callbacks from GTK or
select timeouts is _more_ likely to be right than GETTICKCOUNT).
This checkin provides band-aid workarounds for both problems, which
aren't pretty but ought to at least prevent catastrophic assertion
failure.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5556 cda61777-01e9-0310-a592-d414129be87e

In fact, I'll document the wrinkle with "plink -load", rather than just
wittering about it on a mailing list.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5550 cda61777-01e9-0310-a592-d414129be87e

Fix comment re dmemdumpl()

git-svn-id: svn://svn.tartarus.org/sgt/putty@5549 cda61777-01e9-0310-a592-d414129be87e

Since Markus Kuhn no longer explicitly places wcwidth.c in the public domain,
it'd probably be generous to list him in the copyrights.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5548 cda61777-01e9-0310-a592-d414129be87e

`wcwidth-upgrade': upgrade to latest wcwidth.c from Markus Kuhn
<http://www.cl.cam.ac.uk/~mgk25/ucs/wcwidth.c>.
This is identified both internally and in HTTP headers as 2003-05-20,
for Unicode 4.0.

Only changes from upstream are to make mk_wcwidth_cjk() non-static and to
#include "putty.h" for prototypes.

The status of some code points has changed; see the wishlist item. We've
had some feedback from the CJK and Arabic communities that upgrading is
probably the right thing to do.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5547 cda61777-01e9-0310-a592-d414129be87e

Another fix from Hung-Te Lin; apparently in some IMEs (such as
"MS NewPhonetics"), move events (arrow keys) were being doubled up,
apparently because we turned both KEYDOWN and KEYUP events into new
KEYDOWN events.

I don't claim to understand the precise effect of this patch :( but
I'm reasonably confident that it only affects IME users, and experimentally
it doesn't seem to break anything obvious, so if piaip says it makes
things better that's good enough for me :)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5545 cda61777-01e9-0310-a592-d414129be87e

Fix my fix (r5539) to the `multi-changesettings-crash' fix. Ahem.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5543 cda61777-01e9-0310-a592-d414129be87e

Add an option to use wcwidth_cjk() instead of wcwidth(), as several people
have asked for it.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5542 cda61777-01e9-0310-a592-d414129be87e

Hung-Te Lin spotted that the `multi-changesettings-crash' workaround for Windows
would prevent a user opening Change Settings if they'd cancelled a previous
Change Settings.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5539 cda61777-01e9-0310-a592-d414129be87e

Mention PUTTY.RND by name

git-svn-id: svn://svn.tartarus.org/sgt/putty@5538 cda61777-01e9-0310-a592-d414129be87e

Martin Trautmann spotted a bare char being passed to isspace.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5536 cda61777-01e9-0310-a592-d414129be87e

Trim application manifests as much as MS' documentation and Owen's experiments
on XP allow while still having the desired effect -- this allows removal of
some fibs.
Also, change version number to 0.0.0.0 in preparation for `win-versioninfo'
(not that we found anything that took any notice of the version number
declared here).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5534 cda61777-01e9-0310-a592-d414129be87e

Improvement for IME font display from Hung-Te Lin.
Not tested, but it appears only to affect Glenn Maynard's r1406 code from
<20011006170741.A23470@zewt.org> and nothing else, so seems harmless enough.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5533 cda61777-01e9-0310-a592-d414129be87e

Add comments about default processing in DialogProc/WindowProc, since I
often forget the rules.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5532 cda61777-01e9-0310-a592-d414129be87e

Revert last change: Some versions of the GNU C Library (notably SUSE
glibc-2.3.3-118 and Debian libc6 2.3.2.ds1-20) have clock_gettime() and
CLOCK_MONOTONIC in their headers, but not in libc itself, which we can't
detect easily.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5529 cda61777-01e9-0310-a592-d414129be87e

If it's available, try to use clock_gettime(CLOCK_MONOTONIC) rather than
gettimeofday(), since the former shouldn't warp when the user resets the clock.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5528 cda61777-01e9-0310-a592-d414129be87e

Work around lcc's annoying (but, even more annoyingly, legitimate)
refusal to allow comparison of function pointers. Unfortunately this
still doesn't cause PuTTY to compile on my lcc installation, because
the GetCharacterPlacement stuff in exact_textout() is missing from
its header files. This may have been fixed in a future version (I'm
using lcc-win32 version 3.8 from December 2003), but I haven't
checked.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5527 cda61777-01e9-0310-a592-d414129be87e

More compact display for the key links/fingerprints. Still not ideal, but
it'll probably do.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5526 cda61777-01e9-0310-a592-d414129be87e

Set SVN properties on udp.but the same as everything else

git-svn-id: svn://svn.tartarus.org/sgt/putty@5525 cda61777-01e9-0310-a592-d414129be87e

Try to make our PGP signing more useful:

 * All the PuTTY tools for Windows and Unix now contain the fingerprints of
   the Master Keys. The method for accessing them is crude but universal:
   a new "-pgpfp" command-line option. (Except Unix PuTTYgen, which takes
   "--pgpfp" just to be awkward.)

 * Move the key policy discussion from putty-website/keys.html to
   putty/doc/pgpkeys.but, and autogenerate the former from the latter.
   Also tweak the text somewhat and include the fingerprints of the
   Master Keys themselves.
   (I've merged the existing autogeneration scripts into a single new
   one; I've left the old scripts and keys.html around until such time
   as the webmonster reviews the changes and plumbs in the new script;
   he should remove the old files then.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5524 cda61777-01e9-0310-a592-d414129be87e

After we thaw a frozen socket, we apparently need to restart the
WSAAsyncSelect or else Windows loses read events.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5521 cda61777-01e9-0310-a592-d414129be87e

I believe MAC/CRC errors can be caused by network corruption, in principle.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5520 cda61777-01e9-0310-a592-d414129be87e

Hung-Te Lin's fix for intermittent WM_PAINT problems.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5518 cda61777-01e9-0310-a592-d414129be87e

The structural reorganisation of ssh.c (r4909) caused
ssh2_try_send() to no longer be run after receiving WINDOW_ADJUSTs.
I believe this is likely to have been the cause of recent PSCP
hanging issues.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5517 cda61777-01e9-0310-a592-d414129be87e

faq-savedsettings: mention Default Settings

git-svn-id: svn://svn.tartarus.org/sgt/putty@5514 cda61777-01e9-0310-a592-d414129be87e

Fix `telnet-option-loop', I think. (Untested except to check it
compiles, since I don't have a suitably awkward server to run it
against; but Ben reviewed the patch before checkin so we can share
the blame if it doesn't work.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5512 cda61777-01e9-0310-a592-d414129be87e

Add protection against multiple Change Settings dialogues.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5508 cda61777-01e9-0310-a592-d414129be87e

Fix pterm-logtype-crash

git-svn-id: svn://svn.tartarus.org/sgt/putty@5505 cda61777-01e9-0310-a592-d414129be87e

Consistent indenting in option specs

git-svn-id: svn://svn.tartarus.org/sgt/putty@5502 cda61777-01e9-0310-a592-d414129be87e

punctuation nit

git-svn-id: svn://svn.tartarus.org/sgt/putty@5495 cda61777-01e9-0310-a592-d414129be87e

Retire MINGW32_FIX. It was a fix for a blatant bug in MinGW's windres, which
was fixed in CVS in 2000 (I think); and we now depend on MinGW much more
recent than that for various other reasons. I've tested with my current
MinGW (around 2.0.0 vintage) and the original symptoms (dodgy characters in
edit boxes) don't appear to show up.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5491 cda61777-01e9-0310-a592-d414129be87e

Make it clearer which protocol version various bugs apply to.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5490 cda61777-01e9-0310-a592-d414129be87e

Fiddle with source archive for new OS X files.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5489 cda61777-01e9-0310-a592-d414129be87e

Thanks to D H Becker for sending in an icon. I'd have liked to have
written a script which would generate the various graphical
components of the PuTTY icon suite at any given resolution and then
used that to generate the OS X icon as well as all the others, but I
can always do that later; this'll do for now.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5487 cda61777-01e9-0310-a592-d414129be87e

Expand "v" to "version" when discussing SSH protocol versions.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5486 cda61777-01e9-0310-a592-d414129be87e

When we're quoting user-interface text from PuTTY 0.51, we probably shouldn't
correct its spelling of "SSH-2".

git-svn-id: svn://svn.tartarus.org/sgt/putty@5485 cda61777-01e9-0310-a592-d414129be87e

Fix an "SSH v1" that Jacob missed (because it spanned a line break).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5484 cda61777-01e9-0310-a592-d414129be87e

Make it clear that even when using SSH 2, you only need the SSH-2 pubkey format
if you're actually using the SSH-2 protocol.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5483 cda61777-01e9-0310-a592-d414129be87e

Consistently use a single notation to refer to SSH protocol versions, as
discussed. Use Barrett and Silverman's convention of "SSH-1" for SSH protocol
version 1 and "SSH-2" for protocol 2 ("SSH1"/"SSH2" refer to ssh.com
implementations in this scheme). <http://www.snailbook.com/terms.html>

git-svn-id: svn://svn.tartarus.org/sgt/putty@5480 cda61777-01e9-0310-a592-d414129be87e

Support multiple anchor name types now that Jacob has enhanced
Halibut to be able to do so.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5479 cda61777-01e9-0310-a592-d414129be87e

Protect against multiple Change Settings dialogues.  We should probably also
arrange to switch to an existing Change Settings if the user selects the
menu item and we already have a Change Settings.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5475 cda61777-01e9-0310-a592-d414129be87e

Ben Hutchings reports that new PuTTY instances created from the saved
sessions menu (etc) can inherit listening sockets, and that this sometimes
causes trouble. Can't reproduce any problems myself, but let's only allow
inheritance when absolutely necessary -- Duplicate Session -- in which
case there's already going to be trouble with two processes trying to
listen on the same port.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5468 cda61777-01e9-0310-a592-d414129be87e

Miscellaneous tweaks.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5461 cda61777-01e9-0310-a592-d414129be87e

Charlie at work points out that the screen FAQ contains a different
- and probably better - solution to the screen-vs-alt-screen issue.
Reference it and describe the fix.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5452 cda61777-01e9-0310-a592-d414129be87e

`What commands can I type into my PuTTY session' is probably the
single most frequently asked thing which isn't in the FAQ (it's in
feedback.but instead), so let's add it. I'm uncertain that the
people who mail us asking things like `how do I read my email' and
`how do I access $database' will successfully recognise this more
general question as one which includes their specific one, but it's
worth a try.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5451 cda61777-01e9-0310-a592-d414129be87e

Add notification when using the various auth methods with server-supplied
prompts, to make it more obvious if a server is attempting to spoof a local
passphrase prompt.

I believe an alert user could have spotted this in all cases in SSH-2,
although perhaps not in SSH-1. (But they'd have to have enabled
TIS/CryptoCard.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5450 cda61777-01e9-0310-a592-d414129be87e

Fix sense of test in previous checkin :)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5448 cda61777-01e9-0310-a592-d414129be87e

I'm sick and tired of the persistent reports of lineptr() failing
its `line != NULL' assertion, so I've replaced the assertion with a
call to fatalbox() giving oodles of information. I may still not be
able to reproduce it, but at least next time it happens we should
see a decent amount of debugging data!

git-svn-id: svn://svn.tartarus.org/sgt/putty@5447 cda61777-01e9-0310-a592-d414129be87e

Over-enthusiastic assertions in the port forwarding code removed.
Port forwardings are set up before initialising the last few details
of the main shell session, so ssh->state can reasonably hold values
other than SSH_STATE_SESSION and SSH_STATE_CLOSED during calls to
sshfwd_*.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5446 cda61777-01e9-0310-a592-d414129be87e

Make sure we do notify_remote_exit(ssh->frontend) _before_
connection_fatal(), since the latter is entitled to destroy the
backend so `ssh' may no longer be valid once it returns.

For the Unix port, switch exit(0) to gtk_main_quit() in
notify_remote_exit(), so that we don't exit before the subsequent
connection_fatal()!

git-svn-id: svn://svn.tartarus.org/sgt/putty@5445 cda61777-01e9-0310-a592-d414129be87e

Platform-independent fixes for NO_IPV6 compilation (prevent
segfaults on trying to access pfd->addressfamily which doesn't exist).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5444 cda61777-01e9-0310-a592-d414129be87e

Fixes for NO_IPV6 compilation under Unix.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5443 cda61777-01e9-0310-a592-d414129be87e

Fix potential fault where -L/-R/-D could accept a string too long for our
portfwd[] array. (Not sure what would happen in this case, mind.)
Also modify -L/-R/-D code to cope with IPv4/IPv6 tunnels in saved settings.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5440 cda61777-01e9-0310-a592-d414129be87e

rm some magic numbers in ssh_setup_portfwd()

git-svn-id: svn://svn.tartarus.org/sgt/putty@5439 cda61777-01e9-0310-a592-d414129be87e

Fix minor imprecisions in cfg->portfwd[] bounds checking (which erred on the
side of rejecting input, not buffer overflow).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5438 cda61777-01e9-0310-a592-d414129be87e

Explicitly set MIN and TIME when in raw mode to request single characters
with no timeout.  This seems to make Plink work better on Solaris.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5436 cda61777-01e9-0310-a592-d414129be87e

Draglists, implemented as up-down preference lists as in GTK.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5434 cda61777-01e9-0310-a592-d414129be87e

More confusing "(BYTE *) & val" style punctuation.
I blame GNU indent, although its confusion is understandable.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5432 cda61777-01e9-0310-a592-d414129be87e

nit: confusing punctuation

git-svn-id: svn://svn.tartarus.org/sgt/putty@5430 cda61777-01e9-0310-a592-d414129be87e

rm incorrect comment

git-svn-id: svn://svn.tartarus.org/sgt/putty@5429 cda61777-01e9-0310-a592-d414129be87e

Fix off-by-one in memory management of PPK reading routine, which could cause
1-byte buffer overflow when reading .PPK files with long lines (>=128 bytes
in header value -- probably only happened in practice in the comment field).

git-svn-id: svn://svn.tartarus.org/sgt/putty@5427 cda61777-01e9-0310-a592-d414129be87e

Close on exit.  I'm not entirely happy with the distribution of this
across mac_closeterm() and notify_remote_exit() but it will do for now.
Also, "PuTTY (inactive)" looks strange as a Mac window title in a way it
doesn't on Unix or Windows.  Perhaps we should find another way of
indicating that a window contains a dead session?

git-svn-id: svn://svn.tartarus.org/sgt/putty@5424 cda61777-01e9-0310-a592-d414129be87e

Update Inno Setup script. Tested with IS 5.0.8 on Win98SE; I think there are
a few things that will faze whatever we're using currently (2.0.19 or
thereabouts?), but nothing desperately modern. (NB, the 0.57 putty.iss works
fine with 5.0.8 and the installer is even 40k smaller.)

Notable changes:
 - Uninstallation now runs a variant of `putty -cleanup'. The variance is
   only in the text displayed; the user is still prompted, and the default
   action is (now) "keep" in both cases.
 - Optionally add an icon in the Quick Launch bar.
 - Make desktop item optionally for all users. (not tested)
 - "Create a Start Menu group" now handled via IS' own mechanism.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5423 cda61777-01e9-0310-a592-d414129be87e

Mention `-cleanup' caveats on multi-user systems.
Mention installer in faq-cleanup.
Fix indexing snafu.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5422 cda61777-01e9-0310-a592-d414129be87e

VC didn't like PATH_MAX. Use MAX_PATH instead. (This macro is mentioned in
MSDN's SetCurrentDirectory() documentation, although I haven't found a
statement of where it's supposed to be defined.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5420 cda61777-01e9-0310-a592-d414129be87e

Add context help to a couple of message boxes. Unfortunately the ones
I wanted to get to -- "software caused connection abort" and friends --
are going to be more involved (probably requiring some cross-platform
notion of help contexts), and these ones hardly seem worth the effort.
Still, I've done it now.

Side-effect: Pageant now uses the same `hinst' and `hwnd' globals as
everything else. Tested basic functionality.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5417 cda61777-01e9-0310-a592-d414129be87e

Make it clearer that `-m' is only usable with SSH.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5416 cda61777-01e9-0310-a592-d414129be87e

Move the MessageBox-with-help function out into winutils.c, although it's
still only used for the host key popups. Side-effects:
 - requested_help is a winstuff.h global
 - Pageant now defines winstuff.h globals

(Also, my previous fix to my improved host-key dialogs only got the "changed"
case, not the "unknown" case. Some days I shouldn't be let near a keyboard.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5415 cda61777-01e9-0310-a592-d414129be87e

Make things other than Pageant use request_file(), why not, although all of
its funkiness is switched off so the behaviour is the same as before.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5414 cda61777-01e9-0310-a592-d414129be87e

Fix for `pageant-dirhandle': a new wrapper functions `request_file()' maintains
a separate CWD for the file requester, so that when the Open File box is not
open Pageant should stay where it was started.
(Also some other minor cleanups in this area of Pageant.)

git-svn-id: svn://svn.tartarus.org/sgt/putty@5413 cda61777-01e9-0310-a592-d414129be87e

PSFTP now needs wildcard.o, and was only getting it by luck. Make it explicit.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5411 cda61777-01e9-0310-a592-d414129be87e

Move SaneDialogBox()/SaneEndDialog() from winmisc.c to windlg.c, since they
seem to be PuTTY(tel)-specific (at least at the moment). Might save a bit
of space in the other binaries.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5410 cda61777-01e9-0310-a592-d414129be87e

Improve Pageant's error reporting for private key load failures.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5409 cda61777-01e9-0310-a592-d414129be87e

Improvements to PuTTYgen error reporting:
 - will now display a reason when it fails to load a key
 - uses existing error return from native keys
 - import.c had a lot of error descriptions which weren't going anywhere;
   since the strings are probably taking up space in the binary, we
   may as well use them

git-svn-id: svn://svn.tartarus.org/sgt/putty@5408 cda61777-01e9-0310-a592-d414129be87e

Warn on close.  Also warn before use.

git-svn-id: svn://svn.tartarus.org/sgt/putty@5407 cda61777-01e9-0310-a592-d414129be87e