[racket-dev] [plt] Push #27069: master branch updated
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