[racket-dev] [plt] Push #27069: master branch updated

From: Matthew Flatt (mflatt at cs.utah.edu)
Date: Tue Jul 2 18:48:23 EDT 2013

I think various `read's should probably be wrapped with
`with-module-reading-parameterization', but I haven't yet looked
closely.

At Tue, 2 Jul 2013 18:42:57 -0400, Sam Tobin-Hochstadt wrote:
> I'm sure there are lots of others.  Furthermore, there are a bunch of
> other calls to `read` just in the core, let alone the rest of the
> distribution, that call `read` without setting parameters. Roughly all
> of them are bugs, and maybe security bugs.  This one bit me because
> it's causing a test failure.
> 
> Sam
> 
> On Tue, Jul 2, 2013 at 6:35 PM, Robby Findler
> <robby at eecs.northwestern.edu> wrote:
> > Are there other parameters that can mess this up? case-sensitivity, numbers,
> > etc.?
> >
> > Robby
> >
> >
> > On Tue, Jul 2, 2013 at 3:26 PM, <samth at racket-lang.org> wrote:
> >>
> >> samth has updated `master' from 4dcfe9b8b9 to 45c276b5db.
> >>   http://git.racket-lang.org/plt/4dcfe9b8b9..45c276b5db
> >>
> >> =====[ One Commit ]=====================================================
> >> Directory summary:
> >>  100.0% racket/lib/collects/setup/
> >>
> >> ~~~~~~~~~~
> >>
> >> 45c276b Sam Tobin-Hochstadt <samth at racket-lang.org> 2013-07-02 16:14
> >> :
> >> | Fix reading of config.rktd file when in strange readtable.
> >> :
> >>   M racket/lib/collects/setup/dirs.rkt | 4 +++-
> >>
> >> =====[ Overall Diff ]===================================================
> >>
> >> racket/lib/collects/setup/dirs.rkt
> >> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> >> --- OLD/racket/lib/collects/setup/dirs.rkt
> >> +++ NEW/racket/lib/collects/setup/dirs.rkt
> >> @@ -26,7 +26,9 @@
> >>                   (if (file-exists? p)
> >>                       (call-with-input-file*
> >>                        p
> >> -                      (lambda (in) (read in)))
> >> +                      (lambda (in)
> >> +                        (parameterize ([current-readtable (make-readtable
> >> #f)])
> >> +                          (read in))))
> >>                       #hash()))
> >>                 #hash()))))
> >>
> >
> _________________________
>   Racket Developers list:
>   http://lists.racket-lang.org/dev

Posted on the dev mailing list.