[issue16569] Preventing errors of simultaneous access in zipfile

2014-12-02 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Closed in favor of issue14099. -- resolution: - rejected stage: patch review - resolved status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569

[issue16569] Preventing errors of simultaneous access in zipfile

2014-11-10 Thread David Wilson
David Wilson added the comment: Compared to the cost of everything else ZipExtFile must do (e.g. 4kb string concatenation in a loop, zlib), its surprising that lseek() would measurable at all. The attached file 'patch' is the minimal change I tested. It represents, in terms of computation

[issue16569] Preventing errors of simultaneous access in zipfile

2013-05-07 Thread Serhiy Storchaka
Changes by Serhiy Storchaka storch...@gmail.com: -- Removed message: http://bugs.python.org/msg176850 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2013-05-07 Thread Serhiy Storchaka
Changes by Serhiy Storchaka storch...@gmail.com: -- Removed message: http://bugs.python.org/msg176851 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-29 Thread Serhiy Storchaka
Changes by Serhiy Storchaka storch...@gmail.com: -- assignee: - serhiy.storchaka ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___ ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-06 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Seek can be very cheap. Anybody could actually measure it??? I am waiting for an updated patch for issue14099 to make benchmarks. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-05 Thread Jesús Cea Avión
Jesús Cea Avión added the comment: Seek can be very cheap. Anybody could actually measure it??? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-03 Thread Stepan Kasal
Stepan Kasal added the comment: I agree that reading from a file open for write should be forbidden, no matter whether ZipFile was called with fp or a name. Actually, it is not yet forbidden, and two of the tests in the zipfile.py test suite do actually rely on this misfeature. The first

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-03 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Test: http://bugs.python.org/file24624/Proposed-fix-of-issue14099-second.patch file24624 -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-03 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Test: file24624/Proposed-fix-of-issue14099-second.patch -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-03 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Actually, it is not yet forbidden, and two of the tests in the zipfile.py test suite do actually rely on this misfeature. Indeed. I missed that. Actually these tests work by accident, due to the fact that the contents of the zipfile is placed in the file

[issue16569] Preventing errors of simultaneous access in zipfile

2012-12-03 Thread Stepan Kasal
Stepan Kasal added the comment: but I'm afraid it's impossible to do without performance regression due to seek before every read. I agree that this is key question. I would hope that the performance hit wouldn't be so bad, unless there are actually two decompressions running concurrently.

[issue16569] Preventing errors of simultaneous access in zipfile

2012-11-28 Thread Serhiy Storchaka
New submission from Serhiy Storchaka: If the ZipFile was created by passing in a file-like object as the first argument to the constructor, then simultaneous reading or writing of different file results in an non-consistent state. There is a warning about this in the documentation. The

[issue16569] Preventing errors of simultaneous access in zipfile

2012-11-28 Thread Jesús Cea Avión
Jesús Cea Avión added the comment: I am -0 to this. We can't prevent programmers for shotting in the foot. -- nosy: +jcea ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569 ___

[issue16569] Preventing errors of simultaneous access in zipfile

2012-11-28 Thread Serhiy Storchaka
Serhiy Storchaka added the comment: Reading from closed ZipFile or reading from ZipFile opened for write already forbidden. This is a preventing of the same kind. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue16569