#491 closed defect (fixed)
URIs do not refer to unique files in Allmydata Tahoe
Reported by: | zooko | Owned by: | zooko |
---|---|---|---|
Priority: | major | Milestone: | 1.2.0 |
Component: | code-encoding | Version: | 1.1.0 |
Keywords: | integrity | Cc: | |
Launchpad Bug: |
Description
As Christian Grothoff observed, it is possible for an uploader to make some shares produce one file, and other shares produce another file. The integrity check that is currently required -- the Merkle Tree over the shares -- ensures that only one set of shares can be used for a given read-cap or verify-cap, but it doesn't ensure that only one file can be produced.
The intended semantics of Tahoe immutable files are that there is only one file that can be denoted by a given read-cap or write-cap, so this is a bug. It isn't a major security issue for the typical current use case, since only the original uploader can construct a file to have this ambiguity -- this cannot be used to attack the integrity of a file if you are not the original uploader of that file. However, it isn't the property that we want and it could be used for mischief, so we're going to fix it.
Christian's advisory:
http://crisp.cs.du.edu/?q=node/88
His post to tahoe-dev:
http://allmydata.org/pipermail/tahoe-dev/2008-July/000689.html
Change History (6)
comment:1 Changed at 2008-07-21T17:24:01Z by zooko
comment:2 Changed at 2008-07-30T21:05:59Z by zooko
- Resolution set to fixed
- Status changed from new to closed
This was fixed by 9461887e0a98274e and released in Tahoe 1.2.0. The known_issues.txt describes it, r2788, line 37 ("issue 9"):
http://allmydata.org/trac/tahoe/browser/docs/known_issues.txt?rev=5b84721c7ec215e8#L37
comment:3 Changed at 2008-09-03T01:18:13Z by warner
- Milestone changed from 1.3.1 to 1.3.0
comment:4 Changed at 2008-09-03T16:39:14Z by zooko
- Milestone changed from 1.3.0 to 1.2.0
comment:5 Changed at 2009-12-15T17:59:30Z by zooko
Christian Grothoff won a place on the I Hacked Tahoe! Hall of Fame for this:
comment:6 Changed at 2009-12-16T00:35:55Z by davidsarah
- Keywords integrity added
For historical reference, the URL of Christian's original advisory should have been http://crisp.cs.du.edu/?q=node/90
I updated docs/known_issues.txt to describe this issue.