unofficial mirror of guile-devel@gnu.org 
 help / color / mirror / Atom feed
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



  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).