Bug 1299018
Summary: | tar suffers race condition failure when simultaneous directory creates are attempted | ||
---|---|---|---|
Product: | Red Hat Enterprise Linux 6 | Reporter: | Martin Poole <mpoole> |
Component: | tar | Assignee: | Pavel Raiskup <praiskup> |
Status: | CLOSED WONTFIX | QA Contact: | qe-baseos-daemons |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | 6.7 | CC: | mpoole |
Target Milestone: | rc | ||
Target Release: | --- | ||
Hardware: | Unspecified | ||
OS: | Unspecified | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2016-07-25 12:48:48 UTC | Type: | Bug |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: | |||
Bug Depends On: | |||
Bug Blocks: | 1269194 |
Description
Martin Poole
2016-01-15 17:27:31 UTC
Martin, there has been at least dozen of patches related to this TOCTOU, the major rewrite has IMO started by this patch: commit 4bde4f39d08f000f7e63a832b08a2525c1262f84 Author: Paul Eggert <eggert.edu> AuthorDate: Sat Sep 18 23:37:45 2010 -0700 tar: prefer openat-style functions http://git.savannah.gnu.org/cgit/tar.git/commit/?id=4bde4f39d08f000f But that patch broke _a lot_ of assumptions, and people are still not sure that the actual git HEAD of GNU tar is fixed after that "major" rewrite. Backporting all of this to 1.23 would mean unacceptable risk of issues. So I'm not sure what exactly is the issue here: - is it the TOCTOU in general? Then I would rather prefer WONTFIX in this case (as it is low severity) and suggest using RHEL7 -- should be in better shape. Note that GNU tar documentation says that 'tar' should be run while there are no concurrent processes. - or are there some issues we need to fix by referenced patch? If yes, please post reproducer/patch and we should try to discuss whether it is good time to work on the fix (but note we are in product phase 2). Thanks, Pavel Thanks for the update. |