From: Mike Gran <spk121@yahoo.com>
To: "Ludovic Courtès" <ludo@gnu.org>, guile-devel@gnu.org
Subject: Re: Optimization & factorization of ‘write’
Date: Wed, 15 Sep 2010 07:28:50 -0700 (PDT) [thread overview]
Message-ID: <257144.86099.qm@web37904.mail.mud.yahoo.com> (raw)
In-Reply-To: <87sk1bgrpe.fsf@gnu.org>
> From: Ludovic Courtès <ludo@gnu.org>
> Hi,
>
> Mike Gran <spk121@yahoo.com> writes:
>
> > I have the attached. It is not really a test because it doesn't
> > have criteria for right and wrong, but, you can diff the changes
> > of the output files after committing to see if output formats
> > have stayed the same.
>
> OK, interesting. Any idea how well it covers the display/write code
> compared to the test suite?
No. That's a good question. I had done this type of script testing
when I was hacking the code, but, now, if there are any corner cases
that this catches, they should be rolled into the test suite.
>
> > But I tried it just now on HEAD, and it triggered a 'glibc detected
> > corrupteddouble-linked list' error. Curious.
>
> Hmm indeed. Can you come up with a reduced test case and perhaps a
> backtrace or something?
>
in scm_i_unistring_escapes_to_r6rs_escapes(), you can have a write
off the end of a string when the buffer passed into the function contains
only 4-digit hex unistring hex escapes, such as "\u1100". The
R6RS-escaped string will be longer "\x1100;" causing the memcpy at the
end of the function to write off then end of the string.
Thanks,
Mike
next prev parent reply other threads:[~2010-09-15 14:28 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-09-14 14:21 Optimization & factorization of ‘write’ Ludovic Courtès
2010-09-14 18:31 ` Mike Gran
2010-09-14 19:12 ` Ludovic Courtès
2010-09-14 23:04 ` Ludovic Courtès
2010-09-15 4:19 ` Mike Gran
2010-09-15 11:40 ` Ludovic Courtès
2010-09-15 14:28 ` Mike Gran [this message]
2010-09-15 21:35 ` Ludovic Courtès
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
List information: https://www.gnu.org/software/guile/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=257144.86099.qm@web37904.mail.mud.yahoo.com \
--to=spk121@yahoo.com \
--cc=guile-devel@gnu.org \
--cc=ludo@gnu.org \
/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 read-only IMAP folder(s) and NNTP newsgroup(s).