[issue16858] tarfile silently hides errors

[R. David Murray]

R. David Murray added the comment:

Closing this in favor of issue 27590, which at least got a little discussion.

--
nosy: +r.david.murray
resolution:  -> duplicate
stage:  -> resolved
status: open -> closed
superseder:  -> tarfile module next() method hides exceptions

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Martin Panter]

Changes by Martin Panter :


--
Removed message: http://bugs.python.org/msg265966

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Berker Peksag]

Changes by Berker Peksag :


--
Removed message: http://bugs.python.org/msg265967

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Berker Peksag]

Changes by Berker Peksag :


--
Removed message: http://bugs.python.org/msg270206

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Berker Peksag]

Changes by Berker Peksag :


--
Removed message: http://bugs.python.org/msg270204

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

Марк Коренберг added the comment:

What the hell! sorry again :(

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

Марк Коренберг added the comment:

http://stackoverflow.com/questions/1964806/short-read-from-filesystem-when-can-it-happen

Disk-based filesystems generally use uninterruptible reads, which means that 
the read operation generally cannot be interrupted by a signal. Network-based 
filesystems sometimes use interruptible reads, which can return partial data or 
no data. (In the case of NFS this is configurable using the intr mount option.) 
They sometimes also implement timeouts.

> can return partial data

Seems reading tar-file from NFS-filesystem may trigger that bug.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

Марк Коренберг added the comment:

OOPS, closed wrong task, reopening.

--
resolution: wont fix -> 
status: closed -> open

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

Марк Коренберг added the comment:

Can not reproduce in Linux. Will reopen if reproduced on MacOS X or BSD.

#!/usr/bin/python3.5

import os
import signal


p = os.getpid()

if os.fork() == 0:
while True:
try:
os.kill(p, signal.SIGCHLD)
except (ProcessLookupError, KeyboardInterrupt):
break
os._exit(0)

qwe = open('qwe.dat', 'w+b')
qwe.seek(2*1024*1024*1024*1024)
qwe.write(b'0')
qwe.flush()
qwe.seek(0)
while True:
d = qwe.read(65536 * 32)
if len(d) != 65536 * 32:
raise Exception('!')

--
resolution:  -> wont fix
status: open -> closed

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Ned Deily]

Changes by Ned Deily :


--
nosy: +lars.gustaebel
versions:  -Python 3.5

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

Марк Коренберг added the comment:

Ups. hiding EOFHeaderError is not an error.
But handilng of other errors is not perfect. Please review TarFile.next() for 
cases where .tar file is corrupted. For example,

TruncatedHeaderError is re-raised only if problem at the start of the file. 
Really, it can appear in the (original) middle of the file.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue16858] tarfile silently hides errors

[Марк Коренберг]

New submission from Марк Коренберг:

TarFile.next() silently drop error of one of 

- EOFHeaderError
- InvalidHeaderError

occur. So, extracting corrupted .tar file will not raise error.

--
components: Library (Lib)
messages: 179005
nosy: mmarkk
priority: normal
severity: normal
status: open
title: tarfile silently hides errors
type: behavior
versions: Python 2.7, Python 3.2, Python 3.3, Python 3.4, Python 3.5

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com