News for hackers. Stuff that matters.
Re:Elon Musk emails employees about 'extensive and damaging ... by tlb on Monday June 18, 2018 @07:22PM (#17342440)
With a vanilla git configuration, it's as easy as:

  $ vi centrifuge_rpm_controller.cc
  $ git commit --author "Bob Goodguy <[email protected]>" -a -m "Totally harmless changes"
  $ git push
In a high-stakes environment, it's probably worth signing commits. https://git-scm.com/book/en/v2/Git-Tools-Signing-Your-Work
Re:Elon Musk emails employees about 'extensive and damaging ... by flyingswift on Monday June 18, 2018 @09:40PM (#17343137)
Are we really saying that Tesla, a company building highly sensitive software, is not employing basic branch policies? How is it that these changes could have made it through a code review process and deployed to 'production'?

What I take away from this is that one malicious actor was able to single handedly deploy malicious code and that there were no processes in place to stop this. That is a far more worrying issue

Re:Elon Musk emails employees about 'extensive and damaging ... by giarc on Monday June 18, 2018 @10:30PM (#17343371)
Not sure if it changes your point at all, but the changes were to the manufacturing operating system, not the car OS. I'm not familiar with manufacturing but I imagine there are a ton of different systems and perhaps not all are as secure as the other. I recall reading they needed some paint improvements, so perhaps they changed something there. I don't imagine they would have software controlling paint totally locked down.
Re:Elon Musk emails employees about 'extensive and damaging ... by gregpilling on Wednesday June 20, 2018 @10:57AM (#17355996)
Most manufacturing machines run G-code. Simply changing one of the G-code values (text editing) would cause the machine to make parts incorrectly, and this would screw up things down the line. https://en.wikipedia.org/wiki/G-code

on my Haas mill, the code is directly editable by the operator. On my robots, the code is 'taught' by moving the arm and telling the machine that the new location is the updated. It is routine to have to adjust minor things in the robots, the operator would have to have access to do their job.

I had a welder put a washer under a sensor on my robot one time, making every weld 1 mm in the wrong spot. We couldn't figure it out and ended up reprogramming the entire setup, 8 hours, to fix it. A week later I noticed the washer, put there by a guy who I fired for bad attitude a week before. Once I had found the washer, I had the choice of going back to the old code, or leaving it with the new code.

I left it with the new code. I didn't have another 8 hours to waste to fix the fix.

Even tiny companies like mine can have sabotage. Why not Tesla?

Re:Elon Musk emails employees about 'extensive and damaging ... by avs733 on Tuesday June 19, 2018 @12:01AM (#17343734)
From the friends I know who worked (past tense) at Tesla, that is worrying but far from surprising.
Re:Elon Musk emails employees about 'extensive and damaging ... by topspin on Tuesday June 19, 2018 @10:35PM (#17352118)
"Are we really saying that Tesla, a company building highly sensitive software, is not employing basic branch policies?"

I suspect they aren't. They're trying to hit a number. Elon is on site personally stepping on anything that might impede that goal. They're in desperation mode, running a car company like a startup, erecting tents to house parts of the plant, firing contractors left and right. Are some of their coders banging in changes with little to no review and pushing to "production" -- such as it is -- right off master? Would not surprise me at all.

I don't know what to believe. On one hand Elon claims they've got some sort of admission from someone. On the other I remember the theories he entertained when a Falcon 9 blew up on the pad; “We literally thought someone had shot the rocket.” I tried to explain how infeasible that would be but people don't want to hear it; lots irrational thinking around all things Musk/Telsa.

Anyhow, sabotage isn't uncommon in big plants. Grievances emerge and people sometimes pull some Bolt Weevil stunts to hit back. Usually manufacturers deal with the problem as quietly as possible. Elon, knowing perfectly well his email would leak, decided to make a splash instead. Maybe looking to line up some excuses if they miss the number. Who knows...

"That is a far more worrying issue"

Not for me. We have filled the world with safety; everyone playing by Marquess of Queensberry rules. Nice to see an exception. Yes, even if it costs a few lives.

Re:Elon Musk emails employees about 'extensive and damaging ... by bitL on Tuesday June 19, 2018 @04:30AM (#17344713)
Do they even use git? Maybe they run some prehistoric CVS to save costs...
Re:Elon Musk emails employees about 'extensive and damaging ... by mlindner on Tuesday June 19, 2018 @04:55AM (#17344840)
No they use their own home-brewed management system. This isn't a code repository...
Re:Elon Musk emails employees about 'extensive and damaging ... by prepend on Monday June 18, 2018 @07:50PM (#17342610)
My company only allows authenticated users to commit changes. Even pull requests on our public repos get scanned and reviewed before human review. And pull requests are infrequent enough that unknown users usually involve some interaction.

It seems like if an organization is allowing randos to push changes, they would also allow randos with valid pgp to push changes as well. So the investigation would change to “Who the fudge is B8956DBFEE7C105C?” instead of “Who the fudge is Bob Goodguy?”

Re:Elon Musk emails employees about 'extensive and damaging ... by zxcmx on Monday June 18, 2018 @08:23PM (#17342767)
What you might not realise is that the author of a commit is not related to the (authenticated) user who pushes.

You couldn’t merge changes from multiple people otherwise.

Signed commits do fix this although you have to be careful about your threat model. Lots of software (e.g. gitlab) will prevent alice from pretending to be bob but will not protect you against server compromise (because public key directory and signing enforcement are done on the server).

Working at an org which does signed commits, it doesn’t buy us quite as much as we would like.

Also, if you enforce signing you can’t mash a server-side “merge” button anymore because that commit would need to be signed...

This complicates the pr/mr workflow for teams and increases resistance from those who don’t see the value in signing.

Re:Elon Musk emails employees about 'extensive and damaging ... by prepend on Wednesday June 20, 2018 @08:22AM (#17354666)
What I mean is that making fake author accounts and then merging with an authenticated account shouldn’t help a malicious actor. Any competent project will be interested in the accounts contributing.

Maybe my org is just simple but if “John Doe” started submitting pull requests and prepend merged them, it’s me on the hook and there are people interested in who John Doe is. And in my org, even John Doe needs a network credential so even their local git instance (which I’ve never seen anyone use) would need a credential.

Re:Elon Musk emails employees about 'extensive and damaging ... by solatic on Tuesday June 19, 2018 @12:46AM (#17343918)
> but will not protect you against server compromise (because public key directory and signing enforcement are done on the server)

Which is why it's important for the security team to directly manage and be responsible for the code repository server. Usually the server also stores all the git repositories in some filesystem directory, meaning that server compromise would allow an attacker to remove all the company's code, regardless of permissions set for those repositories, and potentially leak it.

Re:Elon Musk emails employees about 'extensive and damaging ... by jaggederest on Monday June 18, 2018 @11:23PM (#17343570)
Github signs merge commits FWIW. They're pretty good about it.