Slashdot Mirror


Security Lessons Learned From the Diaspora Launch

patio11 writes "Diaspora, the privacy-respecting OSS social network, did a code release last week. Attention immediately focused on security. In fact the code base included several severe security bugs. This post walks through the code, showing what went wrong, and what it would let an attacker do to someone who was using Diaspora." The developer who wrote the post ends with: "You might believe in the powers of OSS to gather experts (or at least folks who have shipped a Rails app, like myself) to Diaspora’s banner and ferret out all the issues. You might also believe in magic code-fixing fairies. Personally, I’d be praying for the fairies because if Diaspora is dependent on the OSS community their users are screwed."

39 of 338 comments (clear)

  1. Security by Anonymous Coward · · Score: 4, Informative

    Because of course, obscurity is proper security.

    1. Re:Security by the_womble · · Score: 3, Interesting

      The OSS model has already proven better in this instance.

      If Diaspora had been closed source, we would not have known about the vulnerabilities until AFTER they had been exploited - very exploited on a large scale. Because the code is open, it has been reviewed and the flaws spotted while it is still in alpha.

      That said, I will still not use this. I am not a real developer and I would be unlikely to make some of the mistakes that these people are making.

  2. ...huh? by Pojut · · Score: 3, Insightful

    Because if Diaspora is dependent on the OSS community their users are screwed.

    Isn't that a bit like saying "if getting this building completed is dependent on construction workers, we're screwed"? Why would you make such a disparaging remark about the very people that will be keeping this thing going?

    1. Re:...huh? by hedwards · · Score: 3, Informative

      Isn't that a bit like saying "if getting this building completed is dependent on volunteer construction workers, we're screwed"?

      FTFY

    2. Re:...huh? by MozeeToby · · Score: 3, Interesting

      Yeah, volunteers have never put up a building before.

    3. Re:...huh? by bigrockpeltr · · Score: 4, Informative

      The summary took the quote slightly out of context. what i understood from TFA is that they are screwed in terms of meeting their (one month?) deadline.

      The team is manifestly out of their depth with regards to web application security, and it is almost certainly impossible for them to gather the required expertise and still hit their timetable for public release in a month.

      --
      $ unzip, strip, touch, finger, grep, mount, fsck, more, yes,fsck,fsck,fsck,umount, sleep
    4. Re:...huh? by jridley · · Score: 4, Informative

      I work HfH construction once in a while. They hire professionals to do the important bits and the large stuff; excavating, pouring the foundation, wiring, plumbing, and often the finish carpentry. If you happen to have someone relatively skilled there, they may assist the pros; I've helped with all; wiring, plumbing, finish carpentry. But you don't let someone who is enthusiastic but doesn't know what they're doing do finish carpentry, they'll probably just wind up wrecking a lot of material. And if you let them do plumbing in an area where code requires copper pipe, you'll probably wind up with a mess that will take a pro 3 times longer to fix than if he'd just done it himself to start with.

      I think the latter may be the case when it comes to this project. I really, really hope this project comes together, but as a programmer I fear that if they've built this thing from the ground up without a good basic understanding of web security, the thing may have to be gutted and rewritten to get to where it needs to be.

      Lots of people can write web apps. Heck, I pretty much write web apps all day long, but I write them for intranet use, they're not accessible to the internet at large. If my stuff had to be hardened against the kind of general attack Diaspora is going to have to endure, I'd have to learn a lot more than I know now.

    5. Re:...huh? by Anonymous Coward · · Score: 3, Informative

      Those services from professionals are almost always paid for not volunteered.

      [Citation needed]. My uncle worked on a HFH home as an electrician and he was not paid for his time.

  3. WTF? by berryjw · · Score: 4, Insightful

    Um, and if closed-source project were to receive the same level of public scrutiny, the users would be any less screwed?

    1. Re:WTF? by gazbo · · Score: 5, Informative
      You've been taken in by Slashdot's trademark selective quoting. What was actually written was:

      The team is manifestly out of their depth with regards to web application security, and it is almost certainly impossible for them to gather the required expertise and still hit their timetable for public release in a month. You might believe in the powers of OSS to gather experts (or at least folks who have shipped a Rails app, like myself) to Diaspora's banner and ferret out all the issues. You might also believe in magic code-fixing fairies. Personally, I'd be praying for the fairies because if Diaspora is dependent on the OSS community their users are screwed.

      (my bold) So he's not actually saying anything bad at all about OSS; he's just saying that being OSS doesn't mean that they can magically gain experience (or experienced developers) and fix their entire codebase in a month. The notion that OSS development is to blame was purely down to Slashdot (or the submitter).

    2. Re:WTF? by locallyunscene · · Score: 5, Informative

      Goddammit kdawson. That's it, your articles are blocked. You're the f***ing New York Post of Slashdot. Whatever merit any article you post may have you manage to completely overpower it with sensationalist editorial bias.

  4. Alternatives to Diaspora by Anonymous Coward · · Score: 5, Informative

    Here is a list of alternative open source Peer-to-peer social networking softwares

    Note that The Appleseed Project has existed since 2004 and is the first.

  5. Invalid Argument by aBaldrich · · Score: 4, Insightful

    if Diaspora is dependent on the OSS community their users are screwed.

    If it wasn't for the OSS community, everybody would believe they've released a safe program. Thanks to OSS, we now know that installing it is not the best decision yet.
    I'd say the users would be screwed if diaspora was not open source. Linus Law once again.

    I was not surprised to find out that the author sells proprietary software. I think that maybe, just maybe he's biased against FLOSS?

    --
    In soviet russia the government regulates the companies.
    1. Re:Invalid Argument by aBaldrich · · Score: 4, Interesting

      I don't think "that it was open source that made people think they ought to test and review code". I think that open source makes it possible (not necessary) to increase the total number of people able to review the code, by orders of magnitude. The diaspora team has 4 people. The total number of forks in github is 403, with over 2500 watchers.

      --
      In soviet russia the government regulates the companies.
    2. Re:Invalid Argument by TheSunborn · · Score: 4, Interesting

      I don't think unproven oss assertion is that "many eyes make bugs shallow". I can accept that. The unproven oss assertion is that many(More then for a similar closed source program) eyes will ever look at the code just because it is open source. I am for example coding c,c++ and Java and running a Fedora Core 13 as my desktop os, but I have newer looked at any any source for any operation system or applications I have been running.

  6. Re:Well, the "developer" doesn't get it by Wolvenhaven · · Score: 4, Insightful

    The editor forgot to mention that the post didn't actually end with what he claims it did, making out the writer to hate diaspora, the post actually ended with:

    Include here the disclaimer that I like OSS, think the Diaspora team is really cool, and don’t mean to crush their spirits when I say that their code is unprofessional and not ready to be exposed to dedicated attackers any time soon.

    He was doing exactly what OSS is for, reading the code, finding the bugs, and informing the developers so they can be fixed, he's only being vilified because the summary is written that way.

    --
    Orwell was an optimist.
  7. Re:Let's give it more than a few hours ... by shadowrat · · Score: 4, Insightful

    it didn't "launch". as i understand it, they released some kind of alpha. I know i've worked for many managers who have this weird idea that software should be perfect before it's even done, but i didn't expect so many people in this community to hold that ideal.

  8. Re:Axe job by Capt'n+Hector · · Score: 5, Insightful

    It doesn't really matter that it's pre alpha, or whatever designation you want to give it. A platform touted as being a secure replacement for facebook ought to consist of secure code from day 1. Security needs to be built into the bones of the program, and If you read the article, you'd see that the errors made were pretty egregious. Fact is, any programmer worth his or her salt would not have started out with code like this with a plan to fix it later in future releases, they'd get it right from the very beginning. That they made these mistakes so early on speaks wonders about how inexperienced these programmers were (and hopefully are no longer now that they've learned their lesson). I'm still rooting for Diaspora but am a little disappointed by this.

    --
    Quid festinatio swallonis est aetherfuga inonusti?
    Africus aut Europaeus?
  9. Re:Axe job by BlueKitties · · Score: 3, Informative

    It's supposed to make your data less completely-controlled by a single mega corporation. Security will be an issue no matter who controls the data, what matters here though is the gatekeeper.

    --
    "Sorrow is better than laughter, for by sadness of face the heart is made glad." [Ecclesiastes 7:3]
  10. Re:Axe job by TheSunborn · · Score: 4, Insightful

    but the conceptual idea of Diaspora is sound

    That may be and nobody is arguing about the concept itself. But a concept with not of much use without any usefull implementation.

    And hell, the majority of the security issues found appear to be rather simple to fix.

    This is exactly why this is so bad. The mistakes are so big and so obvious for any developer with experience in web applications that the developers which worked on Diaspora can not be trusted to write secure code. They have clearly demonstrated that they have absolutely no knowledge of security. They did not just make a security hole due to some obscure implementation detail, they designed and implemented a framework with no security at all.

    And security is not something you can add after you write the code. Just ask Microsoft about that.

    The only solution I see is to get a new team which know how to write code, and then ask them to take over(Or rewrite) the application.

  11. Volunteers by Thyamine · · Score: 4, Insightful

    I think the point they are trying to make (and perhaps badly) is that anytime you have to rely on volunteers you have the potential to get bit in the ass. Any volunteer organization or group has this problem, it's not just open source. Churches, after school groups, the Elks, etc. When volunteers are the main way you expect to get support, you are at their whim. This week people are busy, so no one shows up, or the kids have a soccer game, or some new more exciting group has their interest so you lose a few people.

    I don't think the idea is that the open source community is going to screw people, but that the idea of expecting volunteers to always be plentiful and useful is a good way to cause yourself problems.

    --
    I will shred my adversaries. Pull their eyes out just enough to turn them towards their mewing, mutilated faces. Illyria
    1. Re:Volunteers by JaredOfEuropa · · Score: 4, Insightful

      ...the idea of expecting volunteers to always be plentiful and useful is a good way to cause yourself problems.

      Software projects in business suffer from the same problem, actually. Oh, programmers are plentiful as long as you have budget to spare, but not all professional programmers are created equal, peer reviews / code inspections are slipshod or even omitted, and testing is haphazard. In fact sometimes there's a conscious decision to take shortcuts in those areas because of pressure on the timeline.

      The potential to be bitten in the ass by substandard work that goes undetected is always there, in business as well as OSS projects.

      --
      If construction was anything like programming, an incorrectly fitted lock would bring down the entire building...
  12. Re:Axe job by eln · · Score: 5, Funny

    Exactly. It does exactly what it's supposed to do: Instead of having a single mega-corp have complete control of your data, it does completely the opposite and lets everyone in the world have complete control of your data! It truly is the anti-Facebook.

  13. Re:Axe job by idontgno · · Score: 4, Interesting

    You're overlooking a few points.

    TFA's author acknowledges that it's a pre-alpha preview release. In a sane world, that means no one should ever go on-line with this code. But this is not a sane world, and he very specifically addresses how this release should have been done:

    If you put a gun to my head and said "Our donations came from 6,000 people who want to see progress, give me something to show them", I would have released the code that they had with the registration pages elided, forcing people to only add new users via Rake tasks or the console. That preserves 100% of the ability of developers to work on the project, and for news outlets to take screenshots, without allowing technically unsophisticated people to successfully sign up to the Diaspora seed sites.

    In other words, defang the thing before you turn it loose on an unsuspecting community. If I can successfully develop an open-source backyard nuclear fission generator, and release the pre-alpha blueprints, I would be rightly criticized for the occasional containment failure and subsequent deaths or injuries.

    Also, the attitude of "meh, the security issues are trivially easy to fix" completely misses the point. If the known issues are trivially easy to fix, why weren't they trivially easy to avoid in the first place? Because, apparently, the core developers aren't sufficiently competent or committed to actual application and architectural security. So there's no reason for confidence that there won't be another batch of crippling security flaws with each new release.

    Yeah, a lot of the backlash is probably in response to the hype around Diaspora. But much of the danger is also because of the hype. If Diaspora were just another quiet little Sourceforge project, it might have the luxury of a slow and casual crawl towards reliable application security. But guess what, Diaspora is the current Open Source equivalent of Paris Hilton. Being this screwed up is not an option, when the project is under such scrutiny and subject to such high expectations.

    --
    Welcome to the Panopticon. Used to be a prison, now it's your home.
  14. A Snippet from the Criticism by eldavojohn · · Score: 5, Insightful

    I mean, nothing seems to point to me that this is shill garbage coming from facebook, but the conceptual idea of Diaspora is sound and the code was released for the precise reason of improving it, as it has done ...

    Okay well, sometimes I look at code and I think "good start" and then sometimes I feel like Simon Cowell ... and ask them to start over. So to determine where I stand with the Diaspora code, allow me to quote the article:

    1.#In photos_controller.rb
    2.def destroy
    3. @album = Album.find_by_id params[:id] # BUG
    4. @album.destroy
    5. flash[:notice] = "Album #{@album.name} deleted."
    6. respond_with :location => albums_url
    7.end

    This basic pattern was repeated several times in Diaspora’s code base: security-sensitive actions on the server used the params hash to identify pieces of data they were to operate on, without checking that the logged in user was actually authorized to view or operate on that data. For example, if you were logged in to a Diaspora seed and knew the ID of any photo on the server, changing the URL of any destroy action from the ID of a photo you own to an ID of any other photo would let you delete that second photo. Rails makes exploits like this child’s play, since URLs to actions are trivially easy to guess and object IDs “leak” all over the place. Do not assume than an object ID is private.

    Okay, I taught myself how to use the rails framework and code Ruby. And one of the things I was amazed at was the Rails magic. Because of how powerful it can be (both good and bad). Yes, it helps you prototype but it's errors like these that make me pause and reconsider if the person coding Ruby on Rails really understands how the framework is attempting to assist them. Obviously if you allow any user to enter any ID of a record in their URL for any CRUD action ... you aren't really understanding what those routes are trying to do for you. And you're a danger to your users.

    While I could quickly remedy the above problem for the Diaspora team by improving the authentication and authorization code checks, it might be better to just start over. Now, I've devoted none of my time to the concept of liberating social network users and for that I thank the Diaspora team. This blog posting -- if true -- sure is a vote of no confidence for their capabilities of developing a realistic system. Can they improve? Certainly. But if you're making errors like that, you might be better off letting someone else take a stab at this. It's a harsh thing to say but you don't understand the tool you're using to prototype if you're even starting at this point.

    I wish them the best of luck and I hope the community reaches out to them. But I'm not interested in recoding everything. I'd sooner simply start my own project.

    --
    My work here is dung.
  15. BAD slashdot! by airfoobar · · Score: 4, Insightful

    Someone wrote a blog post to point out some security issues that need fixing in the pre-Alpha version of Diaspora, and here you are using his words for pointless sensationalism that undermines the work of the Diaspora team and propagates the "Diaspora is shite" gossip that will most certainly haunt the project even after the code has hit Beta. Shameful.

    If you want to do something useful, then instead of repeating how doomed the project is, ask for people to join them (I think we have some capable individuals around here) and help out.

    And no, I'm not affiliated with Diaspora, I'm just annoyed by what this sort of news reporting.

    1. Re:BAD slashdot! by Carnildo · · Score: 3, Insightful

      Someone wrote a blog post to point out some security issues that need fixing in the pre-Alpha version of Diaspora, and here you are using his words for pointless sensationalism that undermines the work of the Diaspora team and propagates the "Diaspora is shite" gossip that will most certainly haunt the project even after the code has hit Beta. Shameful.

      These aren't "security issues that need fixing". These are "My First Web Application"-level mistakes -- things like failure to sanitize input, allowing code injection and SQL injection, and assuming that "user is logged in" or "user provided the super-secret URL" is equivalent to "user is allowed to do this". If these errors are as pervasive as the article says, the best thing that could happen to the code is a ground-up rewrite.

      --
      "They redundantly repeated themselves over and over again incessantly without end ad infinitum" -- ibid.
  16. Re:Axe job by jlechem · · Score: 5, Insightful

    I would agree, but that code was some junior level bullshit. Granted I haven't been doing this for 20 years, but damn that was some horrible coding going on there. Especially when they tout it as some super great OSS alternative to facebook. It almost made me question how good of a choice Ruby on Rails was for the entire project.

    --
    Hold up, wait a minute, let me put some pimpin in it
  17. Arrogant "security researcher" bullshit by Meditato · · Score: 4, Insightful

    I don't really understand what's wrong with this blog author, this "Patrick" fellow. Diaspora is git-release of a pre-alpha. It's essentially proof-of-concept which was released so we can have a look at it and contribute. The author's "if this is OSS, we're screwed" assertion apparently ignores the fact that Chromium, Mozilla, Linux, and dozens of other open source projects work perfectly fine. Additionally, the "their code is unprofessional" accusation is simply wrong-headed. It was never intended to be "professional", so there's no way for it to be "unprofessional". It's a foundation released to the public that other people can build on.

    As for all this worry about zero-day holes...every piece of software has them. If you think that these kids aren't professional because they can't make a perfect, idealized, secure pre-alpha, then you're riding the slopes of a Nirvana fallacy. The entire reason it was open-sourced was to allow researchers the opportunity to improve the code INSTEAD of going public in order to gain visits to their arrogant blog posts and acting like there's some huge problem not covered by the disclaimer. OOPS SORRY IS THAT TOO CLOSE TO HOME, PATRICK? I have never seen more arrogant douchebaggery in a security blog post. This "these are errors that shouldn't be present in any code!" bullshit is a result of Patrick and his circlejerk buds building the project up in their own heads, then being disappointed when the pre-alpha wasn't a facebook-killer.

    Yes it has errors. But the very fact that it's 1) open source, and 2) being debugged even by douches such as Patrick, means that the whole "OSS Diaspora" concept ACTUALLY WORKS IN PRACTICE.

    1. Re:Arrogant "security researcher" bullshit by gaspyy · · Score: 3, Informative

      You are right to a point.

      The way I see it, the real problem is not that Diaspora has bugs; the problem is that it has fundamental bugs, bugs so fundamental that they question authors' understanding of the framework they're working with. It's bugs that shouldn't have been there at all.

      Not verifying whether or not a user has the rights to edit an object is something pretty fundamental in my book.

    2. Re:Arrogant "security researcher" bullshit by Abcd1234 · · Score: 5, Insightful

      I don't really understand what's wrong with this blog author, this "Patrick" fellow. Diaspora is git-release of a pre-alpha. It's essentially proof-of-concept which was released so we can have a look at it and contribute.

      And it contains flaws in its security architecture that are so basic, so deeply fundamental, that it's impossible to have any confidence in the development team. Security and scalability *must be architected up front*. Tacking it on after the fact is a recipe for disaster. And it's blatantly obviously that the people working on the project don't understand this.

  18. Re:Axe job by Evanisincontrol · · Score: 3, Insightful

    Yeah, but his point is that [security] is *the* major feature of diaspora. How could it be missing from any release? It should be in there from the beginning, in the core architecture.

    You make it sound like security is just some on/off switch that they forgot to turn on before making the code publicly viewable. That's not how it works. There will always be security improvements to be made to anything, and even... *gasp*... bugs. Especially in a pre-alpha. (If you don't believe me, then show me a major piece of software that's never had a security patch released).

    I mean, christ, the code isn't done! They were just making it viewable it to the public so they could get suggestions for improvement. You know, open source and stuff?

  19. If this article pisses you off by codepunk · · Score: 4, Interesting

    Read the authors blog just a bit, I am not really sure the guy even wrote this article he may have had it commissioned. The author is a crapware distributor and this article is nothing more
    than a attempt at driving traffic to his site which worked. Now his claim to fame is some "bingo card printing software for teachers".

    A few minutes with a compiler and a few dictionary files will show him exactly what "Open Source" is good for. I could really care less about what he wrote but if I was pissed about it there would be a new open source bingo card printing software package released within the next two hours.

    --


    Got Code?
  20. Re:Axe job by Darkness404 · · Score: 4, Insightful

    Because, apparently, the core developers aren't sufficiently competent or committed to actual application and architectural security.

    That is the entire point of having an open source project is that the developers don't have to be experts. Diaspora was developed not because some guys who were great at security decided one day to lunch an uber secure network, it was developed because people were tired of all the crap that FB had so they developed it. Now that the source code is out, security experts can audit the code and make improvements.

    The original developers of an OSS project are like the managers, yeah, they know a little bit about the technical aspects but the main thing they have is vision then people who use it work on it to improve it. Or do you think Linus is some great wizard of security back when he wrote the very first version of Linux?

    Just because the current main developers aren't that great of security doesn't mean security is compromised, actually its the opposite, they can get security advice from professionals and other people who are good at security.

    --
    Taxation is legalized theft, no more, no less.
  21. Re:Axe job by Abcd1234 · · Score: 5, Insightful

    There is no Silver Bullet in coding. You can't get it right from the beginning always...

    Oh come *on*. The errors in this code were deeply fundamental, and patently obvious to anyone paying any attention. Not authorizing actions performed by authenticated users? Really?? Jesus christ, that's *basic*.

    Sorry, no, what we're talking about, here, are fundamental flaws in their security architecture (or, more to the point, a complete lack of security architecture). And security architecture is something you *have* to get right up front (which is why good software architects cost a lot of money... it's necessary work, and hard to do well).

  22. Re:Let's give it more than a few hours ... by iceaxe · · Score: 5, Insightful

    A former employer of mine had a team build a proof of concept for a large and critical piece of software, on which much of the business would rely. The team worked for several months and produced a functioning proof of concept, which they demonstrated to the management. Management took a good look and said, "Great, install it, and support it."

    Within a few more months almost all of the team had resigned in frustration.

    As far as I know, that proof of concept is still in place, with teams of people dedicated to keeping it duct taped enough to keep staggering on.

    The real, serious, carefully constructed and tested software never got built.

    Somehow, I doubt this is a unique tale.

    Moral of the story: start it the way you intend to keep on.

    --
    WALSTIB!
  23. These are design problems, not coding problems by MobyDisk · · Score: 5, Insightful

    Security is part of the design, not the implementation.

    Most developers still haven't learned that security isn't something you check for at various access points in the code: it is something you build directly into the business layer. For example, your code should not have a method like this anywhere:

    public DeletePicture(int pictureID)

    The method should be:

    public DeletePicture(SecurityCredentials user, int pictureID)

    This way it is impossible for your web to accidentally call DeletePicture() without checking for security. The security check is built-in to the lower-level and there is nothing you can do about it. Having worked on secure web services before, I realize I did not do this in my design, which was great for making simple tools, but it meant that all user-facing code had to have checks for security loopholes. The web is especially weird because users can hack the pages and the HTTP requests to call your methods in ways you never

  24. Re:Axe job by Kijori · · Score: 4, Insightful

    This is where the author's claim is rather worrying. His claim in the article (unfortunately rather lost in this attempt at a summary) is that the team completely lacks the necessary expertise and experience to build secure web apps, and has no chance of finding that experience through the open source model before their timetabled release.

    I suppose it largely just boils down to what has often been apparent: an awful lot of volunteer coders have had no education in computer programming or security. While this is not always an issue, attempting to substitute enthusiasm for ability fails rather catastrophically when, as here, the task becomes difficult.

  25. These are cultural problems, not design problems by Paul+Fernhout · · Score: 3, Insightful

    As I said here: http://groups.google.com/group/diaspora-dev/msg/17cf35b6ca8aeb00
    "The central issue many people are concerned about (reading comments elsewhere) is that security is not an "add on". ... Ideally (though few manage this), security needs to be woven intrinsically and mutually throughout an entire endeavor at all levels of the social process, and from beginning to end, from recruitment to developer training to coding standards to code reviews (or whatever works) to archiving procedures to product announcements to bug fix procedures to communications with the public, as well as at all levels of the code itself, the tests, and so on. For many situations, security is often like a chain -- any weak link makes it fail. The less a project embodies this end-to-end security ethic, the more constant vigilance or constant exercise of power is required by everyone involved in it (extrinsic security and/or unilateral security). ...
        So, in that sense, security is cultural. If you try to bolt on security after the fact (like trying to use a big military to defend long oil supply lines instead of having local power sources like solar panels, or trying to be the one who has all the power and everyone is afraid of rather than being the one who has a lot of friends who all share power and look out for each other) you end up spending a lot of time, money, and lives on "security" and you possibly still end up insecure. :-( ...
        Unfortunately, intentional or not, the first Diaspora release has been taken by some people to be a statement about the culture of Diaspora development as regards end-to-end security, even if it was not an intentional statement or even it it perhaps may not be accurate assessment relative to intent or plans. So, it is going to take a bit of work to recover from that, but no doubt it can be done by showing steady progress to creating a developer culture that has a security mindset woven throughout it.
        So how does one get security in practice, assuming you want to do it end-to-end? What engineering attitude may be best to cultivate within that mindset?
        Often, the best security is just simplicity. ..."

    --
    A 21st century issue: the irony of technologies of abundance in the hands of those still thinking in terms of scarcity.