git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: About git and the use of SHA-1

From
SRSverre Rabbelier <alturin@gmail.com>
Date
Apr 29, 2008, 11:05 UTC
Message-ID
<bd6139dc0804290405w4a7a94a7s15a85285b2122f2f@mail.gmail.com>
In-Reply-To
<4816CC80.9080705@op5.se>
On Tue, Apr 29, 2008 at 9:21 AM, Andreas Ericsson <ae@op5.se> wrote:
> Russ Dill wrote:
>  If the server is hacked and objects are replaced, they will either
>  no longer match their cryptographic signature, meaning they'll be
>  new objects or git will determine that they are corrupt, or they

We were assuming here that once SHA-1 is broken really determined hackers will be able to come up with objects that -do- match the SHA-1, so the above is not relevant.

>  *will* match an existing object, but then that object won't be
>  propagated to other repositories since git refuses to overwrite
>  already existing objects. [...]

What about new users cloning the repo? They're just out of luck? I don't think this argument holds, if we want to 'advertise' that git is cryptographically secure we can do so only as long as our hashing algorithm is. (As such, should SHA-1 ever be fully broken we'd need to either switch to another algorithm or stop advertising being cryptographically secure.)

>  [...] Either way, gits refusal to overwrite
>  objects it already has plays a part in making malicious actions
>  futile, since malicious code is only worth something if it's
>  propagated and actually used.

Of course this is true, it makes it a lot harder to do damage, but it doesn't eliminate the problem, it's just a free 'extra protection'. Yes, malicious code is only worth something if it's propagated and actually used, no, it is not impossible to do so in git if/when SHA-1 turns out to have collisions every other file.

-- 
Cheers,

Sverre Rabbelier
Previous: Andreas EricssonNext: Andreas Ericsson
Message 8 of 38 in “About git and the use of SHA-1”
  1. Henrik AustadApr 28, 2008
  2. Daniel BarkalowApr 28, 2008
  3. Henrik AustadApr 28, 2008
  4. Daniel BarkalowApr 28, 2008
  5. Andreas EricssonApr 29, 2008
  6. Russ DillApr 29, 2008
  7. Andreas EricssonApr 29, 2008
  8. Sverre RabbelierApr 29, 2008
  9. Andreas EricssonApr 29, 2008
  10. Paolo BonziniApr 29, 2008
  11. Andreas EricssonApr 29, 2008
  12. Paolo BonziniApr 29, 2008
  13. Russ DillApr 29, 2008
  14. Jurko GospodnetićApr 29, 2008
  15. Russ DillApr 29, 2008
  16. Geoffrey IrvingApr 29, 2008
  17. Daniel BarkalowApr 29, 2008
  18. Dmitry PotapovApr 29, 2008
  19. Andreas EricssonApr 29, 2008
  20. Nicolas PitreApr 29, 2008
  21. Geoffrey IrvingApr 29, 2008
  22. Nicolas PitreApr 29, 2008
  23. Geoffrey IrvingApr 29, 2008
  24. Nicolas PitreApr 29, 2008
  25. Geoffrey IrvingApr 29, 2008
  26. Daniel BarkalowApr 29, 2008
  27. Geoffrey IrvingApr 29, 2008
  28. Fredrik SkolmliApr 29, 2008
  29. Geoffrey IrvingApr 29, 2008
  30. Fredrik SkolmliApr 29, 2008
  31. Martin LanghoffApr 30, 2008
  32. Geoffrey IrvingApr 30, 2008
  33. David BrownApr 30, 2008
  34. Martin LanghoffApr 30, 2008
  35. Matthieu MoyApr 29, 2008
  36. Fredrik SkolmliApr 29, 2008
  37. Tom WidmerApr 29, 2008
  38. Tom WidmerApr 29, 2008

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.