id	summary	reporter	owner	description	type	status	priority	milestone	component	resolution	keywords	cc	parents
647	Uncontrolled reading of files into memory	Jakob Kramer		"Hi!

When I was looking at the attachments source code I have seen code like this:

    with file:
        file.write(another_file.read())

This is dangerous because you read `another_file` into memory completely and this can lead to MemoryErrors.  There is a function `shutil.copyfileobj` that already does what we need: reading chunk for chunk into memory and writing it to another file object."	defect	closed	major	0.8.0	programming	fixed		Stephen Compall Jakob Kramer Matt Molyneaux