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 > <ro...@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, <sa...@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 <sa...@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 _________________________ Racket Developers list: http://lists.racket-lang.org/dev