Page MenuHomePhabricator

Move away from SHA1 in Files
Closed, ResolvedPublic

Assigned To
None
Authored By
epriestley
Mar 29 2017, 7:06 AM
Referenced Files
None
Subscribers
Tokens
"Y So Serious" token, awarded by chad.

Description

Files use plain SHA1 hashes in some cases, and an attack in this form may be possible in some situations now that a SHA1 collision is well-known:

  1. Construct evil.exe and good.exe, which have the same SHA1 checksum.
  2. Upload evil.exe first.
  3. Give another user good.exe and convince them to upload it. Then, convince them to download it and execute it.
  4. If the stars align, they may be served the file data for evil.exe instead of the data for good.exe.

I believe there are a lot of other intermediate issues and (3) is probably pretty suspicious in most situations, but we should move to a stronger hash in the next iteration on Files.

Revisions and Commits

Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision
Restricted Differential Revision

Event Timeline

epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 9:32 PM
epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 9:40 PM
epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 10:24 PM
epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 10:38 PM
epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 10:48 PM
epriestley added a revision: Restricted Differential Revision.Apr 4 2017, 10:54 PM
epriestley added a commit: Restricted Diffusion Commit.Apr 4 2017, 11:18 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Apr 4 2017, 11:22 PM
epriestley added a commit: Restricted Diffusion Commit.
epriestley added a commit: Restricted Diffusion Commit.Apr 4 2017, 11:24 PM
epriestley closed this task as Resolved by committing Restricted Diffusion Commit.Apr 4 2017, 11:56 PM
epriestley added a commit: Restricted Diffusion Commit.