The new forums will be named Coin Return (based on the most recent vote)! You can check on the status and timeline of the transition to the new forums here.
The Guiding Principles and New Rules document is now in effect.

[TRENCHES] Tuesday, May 15, 2012 - Infiltrator

GethGeth LegionPerseus VeilRegistered User, Moderator, Penny Arcade Staff, Vanilla Staff vanilla
edited May 2012 in The Penny Arcade Hub
Infiltrator


Infiltrator
http://trenchescomic.com/comic/post/infiltrator

SQL Injection

Anonymous

I was a contract business/data analyst brought into a “WoW-killer” MMO project with a massive amount of funding from a first tier company. I had two months to burn and usually looked for small, interesting projects to work on until I got the next project, and being a remote gig, I ran with it.  I have been on some hell requirement verifications before and thought this would be just as bad.

It was worse.

Two days into the contract, I was testing the chat system, trying combinations of strings would cause errors, entry speeds, etc. It was open testing. Type in commands and see what breaks. I had some previous test material from another project, so I copy and pasted it into text prompt, trying to overload the message queue.

It did a great job of taking all of the text, including some SQL commands. I ran through a SQL injection test, got some positives, notated the variation, and moved on.  A normal test for my other projects.

My first hint that this WASN’T normal was a conference call from the engagement manager, test lead, and a lawyer. It seemed that they didn’t think it was possible to do what I did and rudely expressed that I must have “hacked the client.” 

I spun up a remote session, brought them into it, and showed how I did it, following the documentation I had sent, with screenshots. I also mentioned that I was saving the session and was recording the conversation, since they were the ones who brought in the lawyers. It got really quiet on the other side, which I interpret as the lawyer putting them on mute, and when they came back, I was told thanks, my contract would be paid out and ended, and all access removed. 

I thanked them for the experience and had an enjoyable vacation from the contract.


Geth on

Posts

  • AriviaArivia I Like A Challenge Earth-1Registered User regular
    So, Bioware didn't think to sanitize SQL in SWTOR chat inputs. Why am I not surprised?

    huntresssig.jpg
  • RMS OceanicRMS Oceanic Registered User regular
    I rather like Isaac's expression in panel three.

    That's quite an interesting bug in the story.

  • CambiataCambiata Commander Shepard The likes of which even GAWD has never seenRegistered User regular
    Could this be part of the reason you can't copy and paste at all in SWTOR's chat system?

    "excuse my French
    But fuck you — no, fuck y'all, that's as blunt as it gets"
    - Kendrick Lamar, "The Blacker the Berry"
  • IncenjucarIncenjucar VChatter Seattle, WARegistered User regular
    It baffles me every time someone mentions lawyers and repercussions in QA testing for doing your job.

    So glad I've never had to put up with or witness that kind of garbage.

  • IvarIvar Oslo, NorwayRegistered User regular
    Cambiata wrote: »
    Could this be part of the reason you can't copy and paste at all in SWTOR's chat system?

    That doesn't sound quite right. There's no difference between manually typing an SQL command and pasting it in. You still have to sanitize data input.

  • HenroidHenroid Mexican kicked from Immigration Thread Centrism is Racism :3Registered User regular
    This is a fantastic strip.

  • AnteCantelopeAnteCantelope Registered User regular
    I don't really get why you'd hire someone to test your chat system, and then get mad at them for doing it. Assuming that he wasn't leaving anything out of the story, he found a SQL injection vulnerability (which should probably have been found much earlier anyway) and this is a problem you need to call the lawyers for?

  • TofystedethTofystedeth Registered User regular
    I don't really get why you'd hire someone to test your chat system, and then get mad at them for doing it. Assuming that he wasn't leaving anything out of the story, he found a SQL injection vulnerability (which should probably have been found much earlier anyway) and this is a problem you need to call the lawyers for?

    and/or already accounted for in any system with user generated text input.

    steam_sig.png
  • Jam WarriorJam Warrior Registered User regular
    They got mad because they didn't believe the things he were doing were possible via the chat system and assumed he must be doing other dodgy unautorised access stuff.

    They let him go on full pay afterwards because they were embarrassed as fuck when they realised how stupid they had been. The false accusation meant he now had a potential grievance against them and thus it was safer to just pay him off and be rid of him.

    MhCw7nZ.gif
  • DedwrekkaDedwrekka Metal Hell adjacentRegistered User regular
    edited May 2012
    Arivia wrote: »
    So, Bioware didn't think to sanitize SQL in SWTOR chat inputs. Why am I not surprised?

    Could easily have been describing Warhammer Online, Age of Conan, or pretty much any MMO in the last 3 years since calling everything new a "WoW-killer" has become boilerplate for MMOs.
    They got mad because they didn't believe the things he were doing were possible via the chat system and assumed he must be doing other dodgy unautorised access stuff.

    They let him go on full pay afterwards because they were embarrassed as fuck when they realised how stupid they had been. The false accusation meant he now had a potential grievance against them and thus it was safer to just pay him off and be rid of him.

    Didn't sound like the person was doing "the usual" either. From the sounds of it they were pulling commands out of a list of commands they had found in other games/programs and only managed to find a few. When you've been doing something for a while, as the person in the story has, you always have some tricks that people tend not to find unless their job is to find those tricks. From the sounds of it, the company already tried to sanitize the input, felt they'd gotten it all worked out, and the tester found some extra lines that caused problems. If the company thought, or was probably told by an overconfident worker, that they'd already worked out the bug, then they probably had reason to think something might be up. Reaction was still more than a little knee-jerk though.

    Dedwrekka on
  • SticksSticks I'd rather be in bed.Registered User regular
    I don't know if I would label my MMO project as a "WoW-killer" at this point. In internet years, WoW is pushing like 80 or 90. It's not really that impressive to kill off an octogenarian.

  • TubeTube Registered User admin
    It's pretty impressive if that octogenarian is a multi-billionaire with a private army and island fortress.

  • fearsomepiratefearsomepirate I ate a pickle once. Registered User regular
    They got mad because they didn't believe the things he were doing were possible via the chat system and assumed he must be doing other dodgy unautorised access stuff.

    They let him go on full pay afterwards because they were embarrassed as fuck when they realised how stupid they had been. The false accusation meant he now had a potential grievance against them and thus it was safer to just pay him off and be rid of him.
    This. It's hardly unique to the gaming industry. I've seen it happen in a wheelchair lift company of all things (a friend of mine noticed internal employee payroll data was in an unprotected spreadsheet on a public folder on the network, notified IT, and got accused of and fired for masterminding a corporate espionage/hacking ring, along with a few of his friends). The process is pretty simple. You need the following toxic combination:

    1. Management that believes it's way, way, way smarter and more knowledgeable about everything than those dumb ol' employees. After all, the employee knew anything, HE'D be the boss, right?

    2. Clever, forward-thinking, creative employee identifies a problem that elderly/stupid/promoted out of their depth/etc management didn't even know could be a problem.

    Because management has already decided that employees can't possibly be clever people, identification of a problem management hasn't already thought of immediately results in accusations of spying or sabotage. If the employee is smart enough to document and prove that he's telling the truth, now management looks stupid...and now they fear that their superiors will notice their stupidity and/or incompetence. Better to get rid of the offending employee than risk your own job.

    Nobody makes me bleed my own blood...nobody.
    PSN ID: fearsomepirate
  • AriviaArivia I Like A Challenge Earth-1Registered User regular
    Dedwrekka wrote: »
    Arivia wrote: »
    So, Bioware didn't think to sanitize SQL in SWTOR chat inputs. Why am I not surprised?

    Could easily have been describing Warhammer Online, Age of Conan, or pretty much any MMO in the last 3 years since calling everything new a "WoW-killer" has become boilerplate for MMOs.

    Remember, it has a top-tier publisher (EA, Activision, maybe Sony or S-E at this point but they don't nearly have that name status) and a simply massive budget.

    Activision doesn't release WoW killers since they have WoW. Square-Enix plays an entirely separate ballpark with FFXI and derivatives. Sony's games by and large have smaller (less prominent) budgets, because they know how to do more with less due to SOE's genre experience. (Additionally, SOE has never released a game tilted as a WoW killer, due to the timing of EQ2 and DCUO being something else entirely.)

    Age of Conan, Rift, et al, don't fit because they don't have a top-tier publisher entwined with and directing development.

    Which leaves EA. You also noted Warhammer Online rightly, but Warhammer wasn't promoted to the press as having a singularly different experience due to its' massive budget spent on polish and art. TOR was.

    The clues are there: the game is SWTOR.

    huntresssig.jpg
  • Lindsay LohanLindsay Lohan Registered User regular
    Couldn't it also be SW Galaxies? I think they threw a good amount of $$ at that one.

    As for the Trenches comic - is it a given that the leak is Q and that's what's going to bring him back into their office somehow.

  • FoefallerFoefaller Registered User regular
    edited May 2012
    Couldn't it also be SW Galaxies? I think they threw a good amount of $$ at that one.

    As for the Trenches comic - is it a given that the leak is Q and that's what's going to bring him back into their office somehow.

    SWG predated WoW by a couple of years, so unless the devs were predicting the future, it wouldn't be that :p

    Couldn't be EQ2 either, the last real big-budget MMO SoE made, as it was released at the same time as WoW. Which really only leaves Warhammer Online and SWTOR as the only two real possiblities.

    Foefaller on
    steam_sig.png
  • PeccaviPeccavi Registered User regular
    It's pretty impressive if that octogenarian is a multi-billionaire with a private army and island fortress.

    Tube knows all about assassination.

  • SticksSticks I'd rather be in bed.Registered User regular
    edited May 2012
    It's pretty impressive if that octogenarian is a multi-billionaire with a private army and island fortress.

    True. I guess I was more thinking, "he's not long for this world anyway, so why bother." I suppose if you were going to do it, the obvious solution would be to do a deal with his son to "help the old man along" in exchange for a cut of the inheritance.

    In this case, that would be.... Diablo 3 I guess? Or would that be more of a nephew? This metaphor is getting confusing.

    Sticks on
  • Ori KleinOri Klein Registered User regular
    edited May 2012
    Could've also been either Mythica or Vanguard:SOH, both were heavily backed by Microsoft at the time (then Mythica got dropped in favor of Vanguard and then in turn Vanguard was dropped due to all company finances being funneled onto Xbox).

    They got mad because they didn't believe the things he were doing were possible via the chat system and assumed he must be doing other dodgy unautorised access stuff.

    They let him go on full pay afterwards because they were embarrassed as fuck when they realised how stupid they had been. The false accusation meant he now had a potential grievance against them and thus it was safer to just pay him off and be rid of him.

    Generic managerial idiocy.
    And to think these people go through human relations courses and what not but no one to date had the sense to teach them that if a human resource produces an incredible result that would classify them as a valuable asset it is likely a very good idea to keep them around and use them to educate other resources in order to elevate their productivity.

    And seriously, you think QA are going to hack the game client just to be able to post false bug reports, complete with reproducible steps, only to get caught on it because 'obviously' it can't be done? Huh!?!?
    GG (un)common-sense.

    For future reference to you manager-youngins, if you ran onto such a situation, here are the proper steps to handle it:
    1) Punch the lawyers (for no reason, really, just do it to be on the safe side of right).
    2) Punch the lead GUI programmer or whomever responsibility it was to build and proof the parser(sorry, you had this coming mate).
    3) Promote the QA guy to sub-lead-tester or something. IDK. Give him a bonus.
    4) Congratulate him on a job well done infront of all of his peers (or, if you were stupid enough to go onto that conference call and accuse him, conference call his peers and lead and apologize like a grown man with a pair).

    Ori Klein on
  • admanbadmanb unionize your workplace Seattle, WARegistered User regular
    Dedwrekka wrote: »
    Arivia wrote: »
    So, Bioware didn't think to sanitize SQL in SWTOR chat inputs. Why am I not surprised?

    Could easily have been describing Warhammer Online, Age of Conan, or pretty much any MMO in the last 3 years since calling everything new a "WoW-killer" has become boilerplate for MMOs.

    Just three? Try eight. Every MMO since WoW came out has been a "WoW killer."

  • FramlingFramling FaceHead Geebs has bad ideas.Registered User regular
    Incenjucar wrote: »
    It baffles me every time someone mentions lawyers and repercussions in QA testing for doing your job.

    So glad I've never had to put up with or witness that kind of garbage.

    All the full-time employees where I work have to do a corporate ethics training thing once a year, we know not to bribe people or blog about proprietary information or what have you. It just a few videos you watch, and then you answer some questions on a little quiz. Usually, most of the videos are along the lines of: Employee does something they're not supposed to; Employee gets in trouble; Employee says to co-worker, "And now I have to go talk to leeeeeeegaaaaalllll!" In all the videos where the employee screws up, having to talk to legal is presented as just the absolute worst-case scenario.

    Makes me feel kinda bad for the people who work in legal.

    you're = you are
    your = belonging to you

    their = belonging to them
    there = not here
    they're = they are
  • LordFizzlebeefLordFizzlebeef Registered User regular
    At one of the companies I worked at, they showed videos to the data entry people telling them to do due diligence on all major contracts before signing off, and make sure to not automatically award a multi-million dollar contract to somebody just because you know them from the golf club. Our supervisors were always at a loss as to why we had to do it, but it was a direct command from management that all the information in the videos directly pertained to us, and if we don't watch them we're all suddenly risks to the company and should be fired.

  • halkunhalkun Registered User regular
    edited May 2012
    Framling wrote: »
    Makes me feel kinda bad for the people who work in legal.

    Don't feel bad, if it's anything like my company even legal has it's teirs. The bottom rung is basic general counsel who does all the dumb legal stuff like running to court, doing filings, and being the firewall to the rest of the company,

    halkun on
  • IncenjucarIncenjucar VChatter Seattle, WARegistered User regular
    Framling wrote: »
    Incenjucar wrote: »
    It baffles me every time someone mentions lawyers and repercussions in QA testing for doing your job.

    So glad I've never had to put up with or witness that kind of garbage.

    All the full-time employees where I work have to do a corporate ethics training thing once a year, we know not to bribe people or blog about proprietary information or what have you. It just a few videos you watch, and then you answer some questions on a little quiz. Usually, most of the videos are along the lines of: Employee does something they're not supposed to; Employee gets in trouble; Employee says to co-worker, "And now I have to go talk to leeeeeeegaaaaalllll!" In all the videos where the employee screws up, having to talk to legal is presented as just the absolute worst-case scenario.

    Makes me feel kinda bad for the people who work in legal.

    Wow, I have new-found appreciation for where I've been working now. I may have to give the building a hug if they call me back before a get a better-paying job. :P

  • DedwrekkaDedwrekka Metal Hell adjacentRegistered User regular
    Generally speaking, people who are going to leak proprietary information or something similarly silly are not going to be dissuaded from doing it because they had to be bored in a classroom for several hours. However, from what I've experienced and seen, the majority of "leaks" are unintentional mentioning of secure information, not intentional misuse or release of privileged info.

  • Jam WarriorJam Warrior Registered User regular
    Dedwrekka wrote: »
    Generally speaking, people who are going to leak proprietary information or something similarly silly are not going to be dissuaded from doing it because they had to be bored in a classroom for several hours. However, from what I've experienced and seen, the majority of "leaks" are unintentional mentioning of secure information, not intentional misuse or release of privileged info.

    These things aren't intended to educate. They're there so there can be no pleading ignorance on the part of any wrongdoer and so top management has its ass covered with a record of telling people how to act if someone goes bad mid way up the chain of command.

    MhCw7nZ.gif
  • Munkus BeaverMunkus Beaver You don't have to attend every argument you are invited to. Philosophy: Stoicism. Politics: Democratic SocialistRegistered User, ClubPA regular
    warhammer online was touted as a WoW Killer, had massive funding by EA, and my sources told me it was broke as fuck for a long time.

    Then they told me not to buy it.

    Humor can be dissected as a frog can, but dies in the process.
  • ApolloinApolloin Registered User regular
    Ori Klein wrote: »
    Generic managerial idiocy.
    And to think these people go through human relations courses and what not but no one to date had the sense to teach them that if a human resource produces an incredible result that would classify them as a valuable asset it is likely a very good idea to keep them around and use them to educate other resources in order to elevate their productivity.

    Oh no, you obviously don't understand. ALL of your employees, despite how clean cut and middle of the road they might look, are basically a heartbeat away from devolving into the kind of people one might find living under an overpass in the wrong part of town. Left to their own devices they will lie, steal, cheat, murder, sell priviliged information and defecate in their cubicles to mark their territory.

    The only way that this sort of anarchic chaos can be forestalled is for management to keep a tight rein on 'em - for their own good! Get them on contracts with punitive clauses and pay them so little that they are basically one paycheque from disaster. That way you can make them care about things like 'corporate culture' and 'face time'. Once you set up a poisonous enough environment the little guys will police themselves!

    Employees whose skills aren't in such demand can be controlled by packing them tightly into test farms. You don't need to waste your time trying to civilise these guys, just scoop two or three of the ones who have made a fuss recently out of there every month and fire them. This will divide them into two camps - the ones who aren't nuts about working in the industry will quietly quit to get a different job whilst the ones who feel some sort of NEED to work in the industry will work like slaves to avoid being the ones who get fired. Either way they'll stop complaining, and that's your objective!

    Some of your employees won't feel the need for 'security' and will have skills where the demand completely outstrips the supply. You're going to be forced to pay these people decently AND give them flexible working conditions. Fortunately this 'special treatment' will usually poison the regular employees against them. If you want to control these anarchic freaks then you're going to have to load their contracts with all kinds of vague and near-meaningless garbage that is essentially unenforcable in an actual court of law. Hopefully they won't want to pay a lawyer of their own to read through their contracts - luckily most don't!

    Obviously if you ever have a problem with employees of this type you'll need to make sure you have a lawyer sit in on the conversation because lawyers are the only ones cunning enough to stop them from weaseling their way out of trouble.

  • ApolloinApolloin Registered User regular
    edited May 2012
    Thanks for the doublepost, forum!

    Apolloin on
  • C2BC2B SwitzerlandRegistered User regular
    Lawyer automatically makes me think of Bethesda.

    Wouldn't the Elder Scrolls MMO fit?

  • admanbadmanb unionize your workplace Seattle, WARegistered User regular
    warhammer online was touted as a WoW Killer, had massive funding by EA, and my sources told me it was broke as fuck for a long time.

    Then they told me not to buy it.

    WAR was fine when it came out. It wasn't really good and it didn't have the polish an MMO needs to compete with WoW, but it wasn't broken on release.

  • NeadenNeaden Registered User regular
    admanb wrote: »
    warhammer online was touted as a WoW Killer, had massive funding by EA, and my sources told me it was broke as fuck for a long time.

    Then they told me not to buy it.

    WAR was fine when it came out. It wasn't really good and it didn't have the polish an MMO needs to compete with WoW, but it wasn't broken on release.
    WAR was best during beta because everyone was low level and just would run around pvping and RvRing queues were really short. The longer time went on the worse it got.
    Also what is SQL injection anyway?

  • DesertChickenDesertChicken Registered User regular
    edited May 2012
    SQL is a kind of database. SQL Injection is finding a space in an application where you can enter text and then tricking the app to run SQL commands for you by typing in SQL patterns.

    DesertChicken on
    PSN/XBLA/SteamID- DesertChicken
    LoL - Renon DeSaxous
  • SticksSticks I'd rather be in bed.Registered User regular
    e.g.

    To get a user's password out of the database, you might type:
    select password from users where user_name = 'Sticks'
    

    However, if instead of putting "Sticks" in the field, I put "Sticks' OR '1'='1" then a (poorly written) application might run
    select password from users where user_name = 'Sticks' OR '1'='1'
    

    '1'='1' is always true, so it would end up returning ALL passwords in the users table.

  • The Good Doctor TranThe Good Doctor Tran Registered User regular
    Also, a lot of the time injection is about identity theft or account access, but it's also frequently about sheer vandalism, c.f.

    exploits_of_a_mom.png

    xkcd

    LoL & Spiral Knights & MC & SMNC: Carrington - Origin: CarringtonPlus - Steam: skdrtran
  • This content has been removed.

  • Eat it You Nasty Pig.Eat it You Nasty Pig. tell homeland security 'we are the bomb'Registered User regular
    every mmo is touted as a wowkiller these days (ironically diablo 3 probably has the best shot at the crown to date.)

    also for all the lawyers talk, it sounds from that story like the lawyer was the only member of the management team who could find his own ass with two hands and map

    hold your head high soldier, it ain't over yet
    that's why we call it the struggle, you're supposed to sweat
  • DedwrekkaDedwrekka Metal Hell adjacentRegistered User regular
    every mmo is touted as a wowkiller these days (ironically diablo 3 probably has the best shot at the crown to date.)

    I don't understand why people even mention Diablo 3, it's not an MMO in any sense. It's a single player game with multiplayer support and no persistent worlds or areas. It's pretty ridiculous when people describe it as an MMO or even "Not a traditional MMO" because it's not even close to an MMO.

  • OptyOpty Registered User regular
    Diablo 3 has definitely drained userbase from WoW, but that's mainly due to WoW being in its final patch for the expansion and everyone has basically done what they've wanted to do so far. They feel no conflict ignoring WoW and moving over to D3 and probably by the time the expansion's out they'll be ready to hop back into WoW.

  • DedwrekkaDedwrekka Metal Hell adjacentRegistered User regular
    Opty wrote: »
    Diablo 3 has definitely drained userbase from WoW, but that's mainly due to WoW being in its final patch for the expansion and everyone has basically done what they've wanted to do so far. They feel no conflict ignoring WoW and moving over to D3 and probably by the time the expansion's out they'll be ready to hop back into WoW.

    This is the important phrase. It's not draining the userbase of WoW in any meaningful way if they're still paying the fees and intend to return to it. This is nothing more than what happens whenever a new high profile single player game comes along. People go play that other game for a few weeks, then they all rubber-band back to playing WoW.

Sign In or Register to comment.