9fans - fans of the OS Plan 9 from Bell Labs
 help / color / mirror / Atom feed
From: "Russ Cox" <rsc@swtch.com>
To: 9fans <9fans@cse.psu.edu>
Cc: sam-fans@hawkwind.utcs.utoronto.ca
Subject: [9fans] sam bug fix
Date: Fri, 12 Jan 2007 11:29:13 -0500	[thread overview]
Message-ID: <ee9e417a0701120829x3856a169s142931a1ba093060@mail.gmail.com> (raw)

I just pushed out a bug fix to sam (not samterm),
both on sources and in plan9port.

This sam should fix a 15-year-old protocol bug
in which the terminal and the sam could get
out of sync during long remote editing sequences.

For example, if you did a global search and
replace in a very large file, it could, very rarely,
happen that afterward the terminal would have
incorrect file addresses for the data it was
displaying, so that subsequent mouse-and-keyboard
edits to the file didn't go where you indicated.

Alternate manifestations of the bug include
individual sam file windows wedging and becoming
unusable (but you could close and reopen the file)
or sam panics.

Rob says he has been getting reports about this
bug infrequently for a long time, but never with
enough information to track it down.  I saw the
symptoms twice in my own editing sessions in the
last six months, but not with enough information
to reconstruct what had happened.

Geoff Collyer wins the prize for the being the
lucky owner of a panicked sam process in which
the bug can be seen chewing on the data structures.

Obviously, there is the potential that I have
introduced a new bug with my fix.  The protocol code
code is historically quite subtle, but I have carefully
reasoned through convincing myself that the changes
are correct, Rob says they are at least believable,
and I have tested them in some unforgiving conditions
that should have teased out any mistakes.

Please let me know if you see any odd behavior or other
problems from sam.

Thanks.
Russ


                 reply	other threads:[~2007-01-12 16:29 UTC|newest]

Thread overview: [no followups] expand[flat|nested]  mbox.gz  Atom feed

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ee9e417a0701120829x3856a169s142931a1ba093060@mail.gmail.com \
    --to=rsc@swtch.com \
    --cc=9fans@cse.psu.edu \
    --cc=sam-fans@hawkwind.utcs.utoronto.ca \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).