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

From: Sam Tobin-Hochstadt (samth at ccs.neu.edu)
Date: Tue Jul 2 18:42:57 EDT 2013

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

Posted on the dev mailing list.