[issue12856] tempfile PRNG reuse between parent and child process

2011-11-25 Thread Antoine Pitrou
Antoine Pitrou added the comment: Patch committed, thank you! -- resolution: -> fixed stage: patch review -> committed/rejected status: open -> closed ___ Python tracker ___ __

[issue12856] tempfile PRNG reuse between parent and child process

2011-11-25 Thread Roundup Robot
Roundup Robot added the comment: New changeset 3c9ddd93c983 by Antoine Pitrou in branch '3.2': Issue #12856: Ensure child processes do not inherit the parent's random seed for filename generation in the tempfile module. http://hg.python.org/cpython/rev/3c9ddd93c983 New changeset 588087429809 b

[issue12856] tempfile PRNG reuse between parent and child process

2011-11-05 Thread Charles-François Natali
Charles-François Natali added the comment: The patch looks good to me. Note that the whole kill(pid, SIGKILL) looks overkill to me... -- nosy: +neologix ___ Python tracker ___ _

[issue12856] tempfile PRNG reuse between parent and child process

2011-08-29 Thread Ferringb
Ferringb added the comment: > the test must be skipped where os.fork() isn't available (namely, under > Windows) Done, although I still humbly suggest telling windows to bugger off ;) > I would do os.read(fd, 100) (or some other large value) rather than > os.read(fd, 6), so that the test doe

[issue12856] tempfile PRNG reuse between parent and child process

2011-08-29 Thread Antoine Pitrou
Antoine Pitrou added the comment: Interesting, thank you. Two nits: - the test must be skipped where os.fork() isn't available (namely, under Windows) - I would do os.read(fd, 100) (or some other large value) rather than os.read(fd, 6), so that the test doesn't depend on the exact length of th

[issue12856] tempfile PRNG reuse between parent and child process

2011-08-29 Thread Ferringb
Changes by Ferringb : Removed file: http://bugs.python.org/file23066/unique-seed-per-process-tempfile.patch ___ Python tracker ___ ___ Python

[issue12856] tempfile PRNG reuse between parent and child process

2011-08-29 Thread Ferringb
New submission from Ferringb : Roughly; tempfile's uniqueness is derived from a global random instance; while there are protections for thread access, a forked child process /will/ inherit that PRNG source, resulting in children/parent trying the same set of names. Mostly it's proving annoying

[issue12856] tempfile PRNG reuse between parent and child process

2011-08-29 Thread Ferringb
Ferringb added the comment: Bleh; pardon, reuploading the patch. hg export aparently appends to the output file rather than overwriting it (last patch had duplicated content in it). -- Added file: http://bugs.python.org/file23067/unique-seed-per-process-tempfile.patch __