diff options
| author | Guido van Rossum <guido@python.org> | 2007-08-07 23:29:20 (GMT) |
|---|---|---|
| committer | Guido van Rossum <guido@python.org> | 2007-08-07 23:29:20 (GMT) |
| commit | 75c26bc6a78169bf621885fa91b0d8f93241e682 (patch) | |
| tree | c7d2ca9081ef06f7cb0edbc687fc780e3a385300 /Modules/bz2module.c | |
| parent | a05577059d6ed67f69e9384252647e2232315f2f (diff) | |
| download | cpython-75c26bc6a78169bf621885fa91b0d8f93241e682.zip cpython-75c26bc6a78169bf621885fa91b0d8f93241e682.tar.gz cpython-75c26bc6a78169bf621885fa91b0d8f93241e682.tar.bz2 | |
BZ2File.read(0) should return b"" rather than raising ValueError.
This fixes test_tarfile.py.
I've added a unit test for the correct bz2 behavior.
Diffstat (limited to 'Modules/bz2module.c')
| -rw-r--r-- | Modules/bz2module.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/Modules/bz2module.c b/Modules/bz2module.c index ee2186e..954c914 100644 --- a/Modules/bz2module.c +++ b/Modules/bz2module.c @@ -431,7 +431,7 @@ BZ2File_read(BZ2FileObject *self, PyObject *args) goto cleanup; } ret = PyBytes_FromStringAndSize((char *)NULL, buffersize); - if (ret == NULL) + if (ret == NULL || buffersize == 0) goto cleanup; bytesread = 0; |