News for hackers. Stuff that matters.
Re:Git client vulnerability announced by tptacek on Thursday December 18, 2014 @04:34PM (#8769751)
It's a very high priority, because there are things that transparently use Git and don't host all their repositories on Github. Update ASAP.
Re:Git client vulnerability announced by jzwinck on Thursday December 18, 2014 @07:27PM (#8770502)
Things which transparently use Git on Windows seem likely to bundle their own copy of Git in their installer. I don't know of specific examples, but if this happens it may be trickier for average people to stay safe. Worse, I don't know how to generate a list of such programs.
Re:Git client vulnerability announced by jackmaney on Thursday December 18, 2014 @06:52PM (#8770385)
Yes. This. I work in a fairly Enterprisey company that uses git as their VCS. I'm just hoping that IT is relatively calm about this and just pushes out updates/nags people about upgrading their git client.
Re:Git client vulnerability announced by califield on Thursday December 18, 2014 @04:40PM (#8769787)
Yeah, but typically you have a certain level of trust in your project dependencies. Adding a library to your project often means granting access to your system anyway (if the dependency contains executable code).
Re:Git client vulnerability announced by peff on Thursday December 18, 2014 @04:45PM (#8769823)
You were and are vulnerable to malicious projects by running:

    git clone git://...
    make
or anything similar, since you are running arbitrary code out of the repository. This release fixes the problem of:

    git clone git://...
    git show
etc. Git cannot fix the "clone and run" problem, which is a social one. But it should be safe to run git commands to inspect the repository contents.
Re:Git client vulnerability announced by comex on Thursday December 18, 2014 @05:43PM (#8770093)
I don't think the GP should be downvoted. What you say is exactly correct - however - I can't even think of a time I've git cloned some piece of code and not proceeded to run some code from it at some point, typically on the same machine. I download code for the purpose of using it, and while I could hypothetically inspect the entire repository for malicious code, I don't think I'm unusual in not doing that on a regular basis.

I guess maybe Docker/Vagrant/etc. users don't normally run code directly on their development machine, so it can be high priority for them. But as someone who doesn't use these tools (not a web developer), for me the vulnerability is extremely low priority.

Re:Git client vulnerability announced by peff on Friday December 19, 2014 @09:37AM (#8772354)
I think we are actually agreeing.

I think it is important to have this Git fix, because it lets people be careful if they choose. But in practice most people are _not_ careful, and will happily clone and run code without inspecting it (or pipe curl to bash!). It's almost impossible to do otherwise, as there are only so many hours in the day.

Ultimately I think we are mostly protected by the fact that this kind of attack is simply not all that common. And if it were done in a very widespread way, somebody would probably notice and the repo would come under scrutiny. It would probably be very effective as a directed attack against a small number of people, especially if the code you executed was sneaky (i.e., install a rootkit, not `rm -rf /`).

Re:Git client vulnerability announced by Bahamut on Thursday December 18, 2014 @04:46PM (#8769829)
All it takes is one dependency getting infected to ruin your day (or more).
Re:Git client vulnerability announced by mikeash on Thursday December 18, 2014 @05:14PM (#8769971)
You can at least in theory inspect the library code after cloning it but before compiling it. Insta-owning your machine just because you cloned it is considerably worse. This is especially true if you always run the library in a sandbox of some sort (e.g. you're building an Android app and your code always runs on an Android device or emulator).
Re:Git client vulnerability announced by dlubarov on Thursday December 18, 2014 @07:40PM (#8770555)
With Android apps you still build locally, typically with Gradle, allowing arbitrary code execution on the host.
Re:Git client vulnerability announced by mikeash on Thursday December 18, 2014 @09:05PM (#8770810)
Oh yes, I suppose a bad build script could undo you. In the land of iOS there are sometimes "libraries" that are just distributed as source files that you add to your own project, so I suppose that at least would qualify.
Re:Git client vulnerability announced by mike_hearn on Friday December 19, 2014 @05:47AM (#8771739)
Additionally, are you willing to bet that there are no buffer overflows in gcc, sed, awk, and whatever other tools you use to build software from git on a regular basis?

Heck, some of the Google security guys have been discovering lately that you can pwn someone just by getting them to run less on a file. How many people start by doing "less README"?

Git should ideally not have this vulnerability, but panicking over this seems overkill. If you want to suck down and work with large amounts of code from a possibly malicious source, you get into virtual machines territory.

Re:Git client vulnerability announced by mikeash on Friday December 19, 2014 @09:43AM (#8772379)
I'm sure there are such bugs, but a bug's severity doesn't decrease just because there are other bugs of similar severity elsewhere.