Slashdot Mirror


Open Source Firm Releases Patch for IE Bug [UPDATED]

An anonymous reader writes "An open source and freeware software development web site has released a patch to fix the URL spoofing vulnerability in Internet Explorer, which can be exploited by scammers who try to trick people into revealing details of online banking accounts or other private information." Naturally, the source for the patch is available as well. Update: 12/19 15:06 GMT by M : Sadly, the patch appears to contain a buffer overflow and some possibly-malicious code - see an analysis and news story, and this comment which suggests the patch author is trying to figure out who is taking advantage of the original vulnerability. Caveat patcher.

81 of 544 comments (clear)

  1. DMCA violator by DigiShaman · · Score: 5, Insightful

    In other news....M$ slams a DMCA lawsuit for "hacking".

    --
    Life is not for the lazy.
  2. well done by b4rB3li7h · · Score: 4, Insightful

    trust OS people to fix what M$ can't find profit for!

  3. No Trusted Computing logo on patch? by Anonymous Coward · · Score: 5, Funny

    I'm not downloading anything that isn't part of a MS plan. Sounds like a trojan attempt to me.

    1. Re:No Trusted Computing logo on patch? by Anonymous Coward · · Score: 5, Funny

      Did you know that MS are now sending out these patches direct via email? Be sure to install it when it arrives.

    2. Re:No Trusted Computing logo on patch? by zin · · Score: 3, Funny

      Yeah next XP service pack won't install because you have a corrupt OS file (due to an unauthorized patch).

      --
      -ZiN-
    3. Re:No Trusted Computing logo on patch? by nacturation · · Score: 4, Funny

      Of course it isn't a trojan. It's a legitimate security update which gets run on your system and makes IE invulnerable to that particular spoof attack. Why, openwares.org even has a definition on their site of what a trojan is:
      • Trojan and/or Worm loaders

        Trick unsuspecting users into downloading harmful viruses
        by disguising them as legitimate security updates.

      So you see, this is nothing more than a legitimate security upd... wait a second!!
      --
      Want to improve your Karma? Instead of "Post Anonymously", try the "Post Humously" option.
  4. How were they able to make such a patch... by znode · · Score: 5, Interesting

    Without the original source to IE?

    1. Re:How were they able to make such a patch... by epiphani · · Score: 4, Interesting

      Exactly what I was going to ask. How do you "patch" software without the original code? You'd basically have to reverse engineer the software, back to some other form of programming language - probably ASM.

      Now, just as a quick check, isnt reverse engineering any M$ product against the EULA? I seriously expect a lawsuit about this.

      Also, patching a binary - that requires *very* detailed knowledge of the binary itself, not? You cant just diff two binaries, and apply patches like that, can you? Run into adressing problems, not? I've never really studied the end result of my code beyond a little gdb'ing.

      --
      .
    2. Re:How were they able to make such a patch... by WolfWithoutAClause · · Score: 4, Informative
      You'd basically have to reverse engineer the software, back to some other form of programming language - probably ASM.

      Off-hand- I'd probably stick a debugger on it, viewing the code at assembler level, and trace the carriage return in from the OS; or something like that. I mean the OS has to call or return to IE when the carriage return is hit; there can't be that many places in the code where it is waiting for input- stick a breakpoint on all of them, and whichever one gets hit after you click on the carriage return is starting to process the code. Run it multiple times with different input and pretty soon you should start to see the patterns.

      It's not especially easy, but it's doable, I've done stuff like that before. It's easier if you have the source code, but it's just slower if you don't.

      --

      -WolfWithoutAClause

      "Gravity is only a theory, not a fact!"
    3. Re:How were they able to make such a patch... by umofomia · · Score: 4, Interesting
      Exactly what I was going to ask. How do you "patch" software without the original code? You'd basically have to reverse engineer the software, back to some other form of programming language - probably ASM.

      Now, just as a quick check, isnt reverse engineering any M$ product against the EULA? I seriously expect a lawsuit about this.

      Actually they didn't have to do that... they just redirect the URLs you click on into their own cgi script off on their server to determine if it's a valid URL.

      I don't know about you, but I prefer that the URLs I go to not be sent to some random server out there. Isn't this basically the definition of spyware!? Also, what happens if their server goes down? Does that mean I'm unable to browse the web at all?

      Wait for Microsoft to come out with a better fix that properly addresses this issue.

    4. Re:How were they able to make such a patch... by KFK+-+Wildcat · · Score: 5, Informative
      It only redirects if the address seems like it contains illegal caracters (and thus tries to spoof the address), not for all webpages accessed.

      See http://www.openwares.org/cgi-bin/exploit.cgi?slash dot.org&www.goatse.cx for instance.

      It might log the addresses attempting to spoof webpages, but I'm all for that. And at least this explains clearly that a spoof was attempted through this exploit. I think it's better than just correcting the string, which would access a spoofed webpage anyways, even if showing the right address at the top... which of course would not work as well but many would still fall for it no matter, especially since it probably would look like http://www.paypal.com@paypal.something.net/ which would seem legitimate to the casual looker.

    5. Re:How were they able to make such a patch... by netsharc · · Score: 5, Informative

      Wrong. :) The URL I found in the source code is http://www.openwares.org/cgi-bin/exploit.cgi? .. try it with http://www.openwares.org/cgi-bin/exploit.cgi?slash dot.org. It's the error page that the program displays when it hits a probable exploit. The program does the checking in your computer and when the link doesn't have %00 or %01, it just shows it normally. Only when it does see a %00 or %01, it sends the link to the above mentioned page.

      If you ask me, maybe they want to have a record of which evil Paypal clone-sites are taking advantage of the exploit so they can tell the cops. Maybe they want to make it easy to tell the users that "MS has issued an update for this problem, please download it!", but of course maybe they want to display ads on that error page (Heh I would do the same).

      But no, URLs that are okay are not being sent to that site.

      --
      What time is it/will be over there? Check with my iPhone app!
    6. Re:How were they able to make such a patch... by meanfriend · · Score: 3, Insightful
      Now, just as a quick check, isnt reverse engineering any M$ product against the EULA? I seriously expect a lawsuit about this.

      While I dont think any reverse engineering took place here, I dont think it would be illegal.

      EULAS are not contracts, you did not sign anything and EULAS cannot override the laws of that country. If reverse engineering is legal, then no amount of draconian wording or clicking on "I Agree" can change that. So if the EULA prohibits me from backing up my copy of Windows (as an example), yet the copyright laws of the country (Canada, in my case) specifically permit me one backup copy, then I am allowed: 1 backup copy

      Some types reverse engineering are prohibited. Like hacking copy protection (as it's covered by the lovely DMCA). But there are efforts to reverse engineer other MS products, like the MSWord format or NTFS and I dont think those are coming under fire. (MS might try to obfuscate or change the formats rapidly, but the very process of RE is not illegal)

      IANALBISLTPOOT (I am not a lawyer but I'd sure like to play one on TV!)

    7. Re:How were they able to make such a patch... by crapulent · · Score: 5, Informative

      From looking at the source it's not actually a patch so much as a 'wedge'. It creates a typelib (or COM object of some sort) that registers itself with the system. By doing this it hooks into the IE API, such that it is called every time a URL is visited. If it detects that the URL contains the spoof, it redirects you to their site, where a CGI script gives you an IE-error-like page: For example if the faked part of the URL was 'fake.com' and the real site was 'real.com' it would redirect you to http://www.openwares.org/cgi-bin/exploit.cgi?true. com&http://fake.com

      So this is not so much a patch as a 'workaround'. It doesn't fix anything, it just intercepts those URLs and warns you about it.

    8. Re:How were they able to make such a patch... by Geek+of+Tech · · Score: 4, Funny
      >> Hmm, don't like that, it would be better to redirect it to someplace harmless like http://127.0.0.1

      Don't bother. I'm so 31337 that I just hacked that 127.0.0.1 loser... In a minute someone should be noticing their root file system missing.... Heheheh

      Hmmmm.... That's funny.... Where'd my MP3's go......

      --
      Stop the Slashdot effect! Don't read the articles!
  5. Re:Acceptance? by TellarHK · · Score: 4, Funny

    Why is the group releasing this on their own?
    To quote the wise sages of the Quake 3 voiceover...

    HUMILIATION!

  6. New MS Security Fix by Ironclad2 · · Score: 5, Funny

    This patch fixes a security bug in Internet Explorer that could allow someone who actually knows what they're doing to repair buggy programs on your computer.

  7. Good to know... by TSR+Wedge · · Score: 4, Interesting

    Good to know that while Microsoft is leaving its users hanging out to dry patch-wise, the community still cares enough to fix the problems. Who knows -- maybe we'll see more effective (i.e., fixing more problems than they cause) patches from here forward.

    --
    What if the hokey-pokey really is what it's all about?
  8. And this matters why? by Anonymous Coward · · Score: 5, Insightful

    So, there is an open source patch for a browser that the people that would have heard of the patch wouldn't use, the /. readers ought to be using mozilla and they know it, if they aren't using mozilla they probably will not install the patch either.

    the people that would likely be fooled by this haven't heard of mozilla and haven't heard of open source and will not hear of this patch.

    so this patch is pointless
    (cool that it can be done though)

    1. Re:And this matters why? by s20451 · · Score: 4, Insightful

      so this patch is pointless
      (cool that it can be done though)


      Ah, but my good Mr. Coward, far from being pointless, the patch puts Microsoft in a delicious conundrum! Either accept and distribute an open source patch (thereby publicly validating the open source model), or ignore the patch and get sued by customers, because a patch existed that they did not publicize.

      ps. Are you related to Noel Coward? Send my regards.

      --
      Toronto-area transit rider? Rate your ride.
    2. Re:And this matters why? by aled · · Score: 4, Funny

      Sued by... by customers bwahaha haha... not... 'nough...ha haha... air...got...to...breath...hahah sued.... customers....

      --

      "I think this line is mostly filler"
    3. Re:And this matters why? by jrumney · · Score: 3, Insightful
      You'd think that Slashdot readers would read the source before installing something claiming to be a security fix from a previously unknown outfit:
      // Terms of Agreement:
      //
      // By using this source code, you agree to the
      // following terms:
      //
      // 1) You may use the source code, resource
      // files for educational purposes only.
      // 2) You MAY NOT redistribute this source code
      // without written permission. Failure to do
      // so is a violation of copyright laws.
      // 3) The author of this code may have retained
      // certain "additional copyright rights".
      // If so, this is indicated in the author's
      // description.
      //
      Yet another example of someone paying lip service to "open source". Do you trust them with the information they are collecting on who is gullible enough to click on links to scams by other parties? Who is to say they aren't running their own scams and allowing them through exploit.cgi while blocking the competition?
  9. What the "patch" really does.... by mikewren420 · · Score: 5, Funny

    What the article doesn't say is that the "patch" just removes IE and installs Mozilla. :)

  10. Direct Link to patch by bogie · · Score: 4, Informative

    For the adventurous among you.

    http://www.openwares.org/downloads/IEpatch.EXE

    --
    If you wanna get rich, you know that payback is a bitch
    1. Re:Direct Link to patch by GaelenBurns · · Score: 4, Informative

      Thanks. I've patched my test system and it didn't even require a reboot! Windows has come so far... when you use as little MS software on it as possible.

      Anyway, I've tested IE by running through some windows updates and going to a few exploit test sites. Everything has behaved as it should.

      By the way, one of the joys of this patch is that when you browse to a site attempting the exploit, you get one of those nice IE error pages, formatted in the traditional way. Except, instead of seeing Microsoft branding all over it, the Openware patch is referenced. I don't know... having this little bit of OSS within IE warms my heart. And just in time for the holidays!

  11. This doesn't actually fix the problem by realdpk · · Score: 4, Interesting

    If you check the code, all it appears to do is redirect the browser to http://www.openwares.org/cgi-bin/exploit.cgi?URL if someone clicks on a bogus URL.

    The overpresence of "strcpy" is a bit unsettling, too.

    While it's a nice step, it's no replacement for an official Microsoft patch.

    1. Re:This doesn't actually fix the problem by Ironica · · Score: 3, Funny

      While it's a nice step, it's no replacement for an official Microsoft patch.

      It's no replacement for... nothing, in other words?

      Microsoft hasn't even said they're *going* to patch this yet, you may be waiting an awful long time.

      --
      Don't you wish your girlfriend was a geek like me?
    2. Re:This doesn't actually fix the problem by crapulent · · Score: 4, Informative
      Yeah no shit, you'd expect better code from "Security researchers." This thing is ripe with bad code (it's sprinked with gotos for error handling) as well as at least one probably exploitable buffer overflow. Observe: here is a bit of the code for the main URL checking routine: ('dest' holds the URL in question and can be up to 256 chars long)


      char surl[256];
      strcpy(surl,"http://www.openwares.org/cgi-bin/ex pl oit.cgi?");

      char sFake[256];
      char sTrue[256];

      if (NULL != strstr(dest,"\2"))
      {
      strcpy(sFake,strstr(dest,"\2") +1);
      _mbsnbcpy((unsigned char*)sTrue,(unsigned char*)dest,strlen(dest)-strlen(sFake)-1);
      sTrue[strlen(dest)-strlen(sFake)-1]='\0';
      RemoveAtAnd(sTrue);
      RemoveAtAnd(sFake);

      } else if (NULL != strstr(dest,"\1"))
      {
      strcpy(sFake,strstr(dest,"\1") +1);
      _mbsnbcpy((unsigned char*)sTrue,(unsigned char*)dest,strlen(dest)-strlen(sFake)-1);
      sTrue[strlen(dest)-strlen(sFake)-1]='\0';
      RemoveAtAnd(sTrue);
      RemoveAtAnd(sFake);
      }
      else
      {
      strcpy(sFake,"unknown");
      strcpy(sTrue,"unknown");
      RemoveAtAnd(sTrue);
      RemoveAtAnd(sFake);
      }

      strcat(surl,sFake);
      strcat(surl,"&");
      strcat(sur l,sTrue);

      Notice the parts in bold. Is it not apparent that 'surl' can easily be overflowed if strlen(sFake) + strlen(sTrue) + strlen("http://www.openwares.org/cgi-bin/exploit.c gi?") exceeds 256. This is really sloppy code.
  12. How? by blair1q · · Score: 4, Insightful

    How do you patch closed source code?

    By violating the EULA by disassembling IE?

    Lovely. I want Bill Gates poking around my sock drawer because I installed an unauthorized patch...

  13. Can we really trust this patch? by GoofyBoy · · Score: 3, Insightful


    A third party releasing a patch to a browser. How safe is this?

    Yes the source code is there, but how do we know the executable doesn't have crap in there?

    Even if everything is clean now, how about the next patch from another source?

    (Not even saying anything about testing and how it can break something. They don't even have the source code of the original product.)

    --
    The surprise isn't how often we make bad choices; the surprise is how seldom they defeat us.
    1. Re:Can we really trust this patch? by Atlantix · · Score: 3, Insightful

      Good questions. It's hard (maybe impossible) to know that an open source patch to a closed source product doesn't break something else. On the bright side, you can know the executable doesn't have extra crap. The point of releasing the source code is so anyone can compile it and verify it actually produces the executable.

      --Atlantix

    2. Re:Can we really trust this patch? by Mikey-San · · Score: 4, Funny

      How do we know the executable doesn't have crap in there?

      You know, the same could be asked of Internet Explorer.

      --
      Mikey-San
      Karma: +Eleventy billion (mostly affected by watching Celebrity Jeopardy)
  14. Re:Seriously. by 56uSquareWave · · Score: 5, Insightful

    Ahem you cant see the source code of IE but you trust that? okay then

    --
    - meta language used, please apply your own spelling and gramma
  15. Will this violate the EULA? by jaxdahl · · Score: 3, Insightful

    Does applying a third party patch violate the EULA for IE?

  16. No thanks by Anonymous Coward · · Score: 5, Funny

    Sorry, but its going to be a cold day in hell when I run something from a website named "openwarez.org".

  17. OMG!!! by Infernon · · Score: 4, Funny

    It didn't ask me to reboot afterwards!!!
    Someone start knitting a sweater for Satan...

  18. Mmf. by BJH · · Score: 5, Informative

    It's only "open source" in the very loosest sense. From the patch:

    Internet Explorer URL Spoofing Security Patch

    Developed by Opensoft Corporation, Vanuatu

    Contact: opensoft@openwares.org

    Opensoft Corporation, Vanuatu
    Copyright 2003 All rights reserved.

    Terms of Agreement:

    By using this source code, you agree to the
    following terms:

    1) You may use the source code, resource
    files for educational purposes only.
    2) You MAY NOT redistribute this source code
    without written permission. Failure to do
    so is a violation of copyright laws.
    3) The author of this code may have retained
    certain "additional copyright rights".
    If so, this is indicated in the author's
    description.

  19. Re:Acceptance? by DavesWorld334 · · Score: 5, Insightful

    Pretty sure this makes Microsoft look really inept. I mean, if the largest and richest software company in the world can't patch their own products before a group of volunteer coders can figure out a fix ... seems to me that makes M$ look like fools.

    My US$0.02, unadjusted for inflation of course.

  20. Re:Seriously. by Atlantix · · Score: 4, Insightful

    Sounds like you're in a no-win situation. You won't install a patch without the MS seal of approval but the patch (allegedly) repairs a known flaw in a product that HAD the MS seal of approval. So that begs the question: What is the value of the MS seal of approval if they're wrong? You'll never be able to install anything!!!

    --Atlantix

  21. This will go far by Ridgelift · · Score: 3, Interesting

    While Microsoft has released an article providing details about the vulnerability, the company is yet to provide a patch.

    I hope this become a trend and attitude among the Open Source community. I must admit that I've been a Microsoft-hater for years, but over time I found that people are really put off by anti-corporation sentiments. I suppose it makes sense in a way; If I invested thousands in a technology for my business, I wouldn't want people telling me "Aw man! You got totally taken! Windows is total crap!"

    If the Open Source community begins patching Windows before Microsoft, not only does it help consumers deal with problems they can't solve, but it bring honor and respect to the Open Source community. Then when people consider Open Source, they're more likely to conclude that Open Source programmers are more competant than corporate programmers.

    It's a win-win-lose. Open Source wins, Consumers win, and Microsoft loses. Which is what I wanted in the first place.

    ESR's right in his article "How to Become a Hacker"

    Q: Do I need to hate and bash Microsoft?

    A: No, you don't. Not that Microsoft isn't loathsome, but there was a hacker culture long before Microsoft and there will still be one long after Microsoft is history. Any energy you spend hating Microsoft would be better spent on loving your craft. Write good code -- that will bash Microsoft quite sufficiently without polluting your karma.

  22. did anyone else feel it... by Stevyn · · Score: 4, Funny

    when hell just froze over? Will microsoft actually have to acknowledge them? Thank them?

    1. Re:did anyone else feel it... by WolfWithoutAClause · · Score: 5, Funny
      Will microsoft actually have to acknowledge them?

      Yes, of course! The subpoena will mention them by name.

      --

      -WolfWithoutAClause

      "Gravity is only a theory, not a fact!"
  23. Re:... huh? by arvindn · · Score: 4, Funny
    Try some of these (funny yet scary at the same time):
    • Next time there's a hole in MSIE so big you can drive a cart through it, MS will release a patch in a week and say: "See! We told you we're more secure than open source. We have a patch out already and openwares.org hasn't yet!"
    • People will believe them when they say that
    • Openwares is going to get sued by MS claiming there's no way they could have released a patch unless they illegally obtained the source
    • I'm sure there's a joke or three out there about the name (wares->warez) but I can't find it :)
  24. FWIW... by NickFitz · · Score: 3, Insightful

    this is the whois record for that domain from whois.networksolutions.com:

    Domain ID:D98313967-LROR
    Domain Name:OPENWARES.ORG
    Created On:03-Jul-2003 22:49:55 UTC
    Last Updated On:02-Sep-2003 03:58:23 UTC
    Expiration Date:03-Jul-2004 22:49:55 UTC
    Sponsoring Registrar:R14-LROR
    Status:OK
    Registrant ID:WBMRD
    Registrant Name:ori rejwan
    Registrant Street1:52 Herbert Samuel St.
    Registrant City:Tel Aviv
    Registrant State/Province:NA
    Registrant Postal Code:63304
    Registrant Country:IL
    Registrant Phone:+1.97250314892
    Registrant Email:orejwan@yahoo.com
    Admin ID:WBMRD
    Admin Name:ori rejwan
    Admin Street1:52 Herbert Samuel St.
    Admin City:Tel Aviv
    Admin State/Province:NA
    Admin Postal Code:63304
    Admin Country:IL
    Admin Phone:+1.97250314892
    Admin Email:orejwan@yahoo.com
    Tech ID:AD384-ORG
    Tech Name:Mohammed Zarqa
    Tech Organization:Tri State Contracting
    Tech Street1:POBox 455
    Tech City:East Brunswick
    Tech State/Province:NJ
    Tech Postal Code:08816
    Tech Country:US
    Tech Phone:+1.7322383766
    Tech Email:mzarqa@aol.com
    Name Server:NS2.ABAC.COM
    Name Server:NS1.ABAC.COM

    It's up to you to decide whether you trust them or not.

    --
    Using HTML in email is like putting sound effects on your phone calls. Just say <strong>no</strong>.
  25. Re:No updates for December? by Neo-Rio-101 · · Score: 4, Insightful

    That's not the point. The point is that MS has ignored patching this vulnerability for far too long. It put its promise of "no patches for December" above the real and critical need to update the most common browser running on the worlds computers from hack attacks. Whether you install it or not is your business, and further more, if the patch was truly buggy everyone would be screaming about it by now.

    --
    READY.
    PRINT ""+-0
  26. In other news... by BladeMelbourne · · Score: 5, Funny

    Open Source Firm Releases Patch for IE Bug

    In other news...

    Today Micro$oft contributed code to the Linux kernel, and announced plans to help iron out differences between Mozilla and MSIE :-)

  27. Inept and free! by fm6 · · Score: 4, Interesting
    Pretty sure this makes Microsoft look really inept.
    Since when have they needed any help with that?

    If people are doing open source IE patches, would somebody please fix this sucker? Thousands of people are complaining about this bug online, yet MS hasn't even officially admitted its existence. Now that's inept!

    1. Re:Inept and free! by lurker412 · · Score: 5, Funny

      Yeah, patch Q824145. In my case, it turned out to be a blessing. I got so pissed off that MSFT broke standard UI scrolling behavior that I switched to Firebird. I don't understand how a large, successful software company can do such sloppy QA and think that nobody will notice. But then, there are many things that I don't understand.

  28. Free IE patch and fix. by ratfynk · · Score: 4, Funny

    Found a wonderful fix it is called cfdisk! and slackware 9.1 setup, works great and no IE security issues!

    --
    OH THE SHAME I fell off the wagon and use sigs again!
  29. using the API by ramzak2k · · Score: 4, Interesting

    If i am correct all microsoft applications do have allow access to APIs (Application Programming interfaces). I have written a simple application in Visual Basic once that used the API of MSN instant messenger to listen to the messages sent to me and do a custom auto reply saying things like "i will be back in a few mins".

    Once someone has a grip of IE's API, this shouldnt have been too difficult - after all they just check if the URL requested for(which should be triggering an event in the API) has a particular type of input. If so they redirect it to a different URL (their own website).

    If the patch has been done this way it is more reason not to apply it - it is not exactly the cleanest way to fix it.

    --

    Siggy Say, Siggy Do
  30. How about this one .... by taniwha · · Score: 4, Funny

    M$ picks up an open source bug fix off the net, rolls it into IE and releases it real fast ..... 2 weeks later the FSF comes a knocking wanting to know where the source for IE is and "didn't you say in court your browser is so highly integrated into your OS it can't be removed ... we'll have the source to that too please" ....

    1. Re:How about this one .... by CaptBubba · · Score: 5, Interesting
      I actually don't see any mention of the GPL on any of the pages, or in the downloadable source.

      Doesn't this mean that nobody else is allowed to distribute it? I mean, MS could still get in a whole lot of trouble for inclusing this code in its patch, but they wouldn't risk losing source code.

    2. Re:How about this one .... by jujitsustab · · Score: 5, Insightful

      Why would Microsoft use this code in their patch ? This patch code is based upon readily available IE com interfaces which allow addon IE programs to interact with browser operations. In fact, this patch simply checks the url for the vulnerability every time you navigate to the page. If the vulnerability is found it instead naviagtes to: http://www.openwares.org/cgi-bin/exploit.cgi?A&amp ;B where A is the spoofed url and B is the actual url. Microsoft would fix this vulnerability in the actual IE code, not in a bolted on module like this.

    3. Re:How about this one .... by aulendil · · Score: 3, Informative
      Doesn't this mean that nobody else is allowed to distribute it?

      No, the GPL isn't the only way for distributing source code. The code could be in the public domain (or no license), allowing anyone to use/distribute it, it could well be under a BSD style license which would allow anyone to do whatever with the code. And no, whichever way the code is licensed Microsoft couldn't lose source code. As this is a standalone fix using the IE COM interface the code doesn't link to any Microsoft object code. Therefore the GPL doesn't apply.

  31. I wouldn't call this a patch... by goranb · · Score: 5, Insightful

    Judging from the source it's a quite simple COM object, which hooks into IE and checks URLs before IE actually starts "processing" them (opening connections, parsing...)
    If it finds anything out of the ordinary (like an exploit) it just redirects IE to their own site. Specifically to http://www.openwares.org/cgi-bin/exploit.cgi. It adds a few paramters (the fake url among other), so I guess they will be building a database of exploiters...

    It's no patch, IE stays as it is. It's more a workaround. I'm not sure whether these hooks are documented (allthough being a windows system programmer I never liked IE and stayed as far away from it as possible), but if yes, Microsoft might actually have nothing on openwaves...

  32. Re:... huh? by Niten · · Score: 5, Funny

    If this patch gets the press coverage that it deserves, maybe people will learn to take Microsoft's claims of better security response rates than those open-source folk, with a grain of salt.

    Or maybe Microsoft will actually start working harder to keep their software secure in a timely manner?

    </fingers_crossed>
  33. Memory leak by Anonymous Coward · · Score: 4, Informative

    From a cursory look at the source code, it looks to me as though there are at least two memory leaks. To be more specific, in function BeforeNavigateEvent(), there are two calls to malloc(), but no calls to free(), and the pointers that malloc() returns are stored in local variables, so there is no possibility that a parent function free()s them. Having said this, I haven't written any code under Windows, so maybe there is some kind of garbage collection in the Windows memory model that I am ignorant of?

  34. The time problem has nothing to do with the patch by SonicBurst · · Score: 5, Insightful

    The time it takes to patch the problem is miniscule compared to the regression testing done to make sure the patch fucks up as little as possible. They test EXTENSIVELY and even so you still get the occasional patch that interacts with other software and ways you can't predict and breaks something. It happens. Any code monkey could hack out a patch, but I know damn well they haven't tested this as much as a corporation supporting 90% of the world's browser users would. That's where the time is, so quit bitching about how long it takes to release a patch. Now, the time it takes to ACKNOWLEDGE a bug is a different story....

    --

    Geek used to be a four letter word. Now it's a six-figure one.
  35. This "patch" leaks memory - and other bad stuff by bighoov · · Score: 5, Informative

    A list of the bad things about this "patch", just at first glance:

    1. Leaks 256 bytes on every URL navigation
    2. Leaks 512 additional bytes if it finds an exploit URL
    3. Creates a string with the \1 char in it on every call, but does nothing with it
    4. Will overwrite stuff on the stack if the URL has the exploit and is very close to 256 chars in length.

    It's a good thing these guys aren't on the real IE dev team.

  36. RTFC by b17bmbr · · Score: 5, Informative

    if you'd have taken a few minutes (or seconds w/broadband) to get the source and look at the code, you'd see this:

    By using this source code, you agree to the following terms: 1) You may use the source code, resource files for educational purposes only. 2) You MAY NOT redistribute this source code without written permission. Failure to do so is a violation of copyright laws. 3) The author of this code may have retained certain "additional copyright rights". If so, this is indicated in the author's description.

    since i doubt there'd be anything educational about IE source code...and by the way, i don't think this qualifies as an open source license.

    --
    My problem? I was perfectly gruntled, until some numbnuts came by and dissed me.
    1. Re:RTFC by Holi · · Score: 4, Interesting

      I disagree. Just be3cause something is open source (as in they offer the source with the binary) does not mean it must be GPL (freely ditributable (ok nevermind the really poor spelling)). GPL is not the only open source license. In fact I wish more commercial software were like this. It would make security auditing much easier. Yes I cannot use their code in my projects but at least I could have some trust in their programs.

      --
      Sorry, teleporters just kill you and then make a copy. A perfect, soul-less copy.
  37. Do Not Use It-It's Got a Huge Vulnerability Itself by DmitriA · · Score: 5, Informative
    For one thing, it's an IE add-on (similar to a GoogleBar and others), not a patch. So it's a messy solution to begin with.

    On top of that, it's buggy. It has a memory leak in its BeforeNavigatorEvent() IE callback function which gets triggered before a loading of each new page. There they allocate a string of 256 bytes, but never even bother to clean it up!
    I'm not even sure if that memory is going to be cleaned up when you close all the IE windows, since it's really a Windows system component and this DLL may not be unloaded even with the closing of IE. But I may wrong that point...

    But even that's not the worst thing. Their code actually contains a buffer overflow, allowing the attacker to execute code on your machine with the privileges of the IE process just by crafting an invalid URL link and getting you to click on it!

    Basically, they use WideCharToMultiByte() to convert the unicode URL string to that allocated 256-byte ASCII character array. They tell the function the size of their array, but if the URL string exceed 256 characters in length, it will not overwrite that buffer and cause an immediate buffer overflow. Instead it will fail and tell you to increase your buffer. Well, guess what? They don't check for that failure condition (and, incidentally, it may fail for many other reasons during the Unicode->ASCII conversion) and happily proceed to use it in a strcpy() later on, overwriting another 256-byte character array which is now located on the stack. A nasty buffer overflow just waiting to be exploited...

    So to summarize, they took a relatively minor problem (URL spoofing) and made it a hundred times worse with their 'solution'. Great job, guys!

    Offending code:
    /* memory leak */
    char *dest = (char *)malloc(256*sizeof(char));

    /* Unicode->ASCII conversion that doesn't do error checking */
    WideCharToMultiByte( CP_ACP, 0, BSTR)url->bstrVal, -1, dest, 256, NULL, NULL );

    ...

    /* vulnerable arrays on the stack */
    char sFake[256];
    char sTrue[256];

    ...

    /* please overwrite the return address on the stack and execute my shellcode */
    strcpy(sFake,strstr(dest,"\2") +1);
  38. Re:Do Not Use It-It's Got a Huge Vulnerability Its by DmitriA · · Score: 5, Informative

    Eh. Just realized that since WideCharToMultiByte() will fail, it will not actually copy the URL to the dest[] array and thus, you probably can't overwrite the return address with a legitimate value and get it to point at your shellcode. It's still easy to overwrite it with a random value (with whatever is sitting at the time in the uninitialized dest[] array) and cause a crash, but executing malicious code may be a little harder to pull off...

  39. Opera by 10scjed · · Score: 4, Interesting

    Opera7.23- not only is it not vulnerable to this exploit, it pops up a dialog box to advise you're being redirected to a user@ address (and shows the real address in the bar).

    --
    --10scjed IANAL,AFAIK
  40. Over hyped. by jag164 · · Score: 4, Interesting
    First of all. This 'patch' isn't too extrodinary. This is a plugin similar to the google bar. There is no reverse engineering, thus no threat of DMCA. It's really not a patch, it's more of a work around. It's also a publicity stunt. Upon detecting a spoof, there should me no reason to go to the 'patch makers' website for info when it could be done within the plugin.

    Second, it's a horrible precedent for closed source software. Let close source fixed close source. This may seem like a good thing(tm) for the OSS communtity, but you know damn well that not-so-good-intentioned 'patches' will soon follow. Post some source on a site, provide an EXE(that of course didn't come from the source) and you've fished in countless joe users before the real word is out that a copy cat has duped you. Too late for some.

    I can only see bad things(tm) coming from this idea. Geeks know who and what to trust, but Joe User doesn't. And when joe user screws up it screws us all.

    The sum: This may have a greater negative impact in the long run then the good one it was intended to have.

  41. Re:Do Not Use It-It's Got a Huge Vulnerability Its by DmitriA · · Score: 4, Insightful

    Well, this is hilarious. I guess I should never assume anything until I try it out myself. Apparently when WideCharToMultiByte() fails, it DOES overwrite your string until but presumably does not go over the specified bounds. So their code is still vulnerable to remote code execution since you can fill the dest[] array with the shellcode and a new return address that would point to it. You only have 256 bytes to work with (in reality even less, since they have some other stuff on the stack that you need to get over before you get to the return address), but if you are good with assembly, that should be enough to do some fun stuff... In comparison, Slammer was 306 bytes in size, but of course did quite a bit too...

  42. Re:Acceptance? by daniel23 · · Score: 4, Interesting

    I think mozilla misrepresents the url in the status line while the address line shows the url correctly.
    MSIE, on the other hand, fails completly.

    In fact, on some versions of mozilla you even can spot a control char in the status line, too. But real spoofing depends on the address line.

    heise (German)

    As a test:
    http://www.mozilla.org%00@www.heisec.de

    is shown as http://www.heisec.de in mozilla, while msie puts http://www.mozilla.org into the address line.

    --
    605413? Yes, it's a prime.
  43. Re:... huh? by BiggerIsBetter · · Score: 4, Interesting

    Or maybe Microsoft will figure it doesn't need to provide patches in a timely manner, because the user community will do it for them.

    --
    Forget thrust, drag, lift and weight. Airplanes fly because of money.
  44. Re:Do Not Use It-It's Got a Huge Vulnerability Its by qtp · · Score: 4, Funny

    It seems you've got a good handle on this, so when can Openwares expect your patch for the vulnerability in thier patch?

    --
    Read, L
  45. Re:... huh? by hdparm · · Score: 3, Interesting
    Exactly. This is only good for Microsoft. They've got their ways of convincing masses why anything that comes out in press is actually good and why is Microsoft the one to praise.

    They have thousands of programmers, let them move their butts and do their fucking job. More holes in IE, easier to convince people to switch to Mozilla.

  46. In Other Other News... by Anonymous Coward · · Score: 4, Funny

    SCO Group of Lindon Utah announces that it has filed suit against Microsoft for including Unix/Linux code in Microsoft's Internet Explorer. Darl McBride says "There's no way these burger flipping losers could fix IE without our help. Microsoft couldn't even fix it without our lawyers."

    Shrewd investors continue to laugh at the SCO Group's activities and have the following comments:

    "The funniest thing I've seen since the Paris Hilton tapes!" - MSN

    "A gut buster worthy of John Belushi - but SCO does more drugs" - Timothy Leary

    SCO also announced that Caldera Linux licences still outpace all other SCO products - excluding lawsuits - by a 2:1 margin. Darl announced that they expect to make that 3 to 1 by next summer before they are purchased outright by IBM for $1.50 and a can of Red Bull.

  47. And if it were MS code by phorm · · Score: 3, Insightful

    Then nobody would have noticed the stack vulnerability, unless you had either a machine vulnerable to the original exploit, or a machine vulnerable to a new exploit as per being patched

    Since it is open-source, however, somebody can fix that bug nice and quick before it becomes another problem (gee, imagine that).

    Lack of foresite on the behalf of the patch developer is a bit disturbing, but not a bad reflection on OS code at all :-)

  48. FoxPro was patched sans source ... by kupci · · Score: 3, Interesting
    Granted, not your average programmer can do this, and yes you're right, it does take detailed knowledge, and a little familiarity with assembly.

    Microsoft, in it's efforts to steer people away from FoxPro to Access, many years ago, decided to not bother patching some serious issues with FoxPro. What happened was there was a very poor piece of code that tried to figure out how fast your processor was when FoxPro started up, I forget exactly what it was for, but the programmer(s) made a small bug where if the processor was extremely fast, the value would be set to -1, and FoxPro would promptly crash. Worked fine for years until some of the new processors came out.

    Anyway, Microsoft stalled on fixing this timing issue bug, so some smart fellow tweaked the exe file to fix it. Yeah, not even assembler, we're talking hex. Pretty damn cool.

  49. That's why OSS is more secure... by kupci · · Score: 3, Interesting
    For one thing, it's an IE add-on (similar to a GoogleBar and others), not a patch. So it's a messy solution to begin with.

    There's a saying for this: crap built upon crap.

    There they allocate a string of 256 bytes, but never even bother to clean it up! I'm not even sure if that memory is going to be cleaned up when you close all the IE windows, since it's really a Windows system component ...[more scary windows stuff]

    Seems like a combination of the lousy design of the Windows components coupled with using C. Long, long time since I've worried about destroy and the like, what with the availability of better languages like Java, etc. Granted once buffer overflows are a thing of the past, there will be new holes, but at least we will be moving forward.

    But even that's not the worst thing. Their code actually contains a buffer overflow, allowing the attacker to execute code on your machine with the privileges of the IE process just by crafting an invalid URL link and getting you to click on it!

    Good catch. So one security flaw fixed, opening up another flaw - a little embarrasing, except MSFT did the same thing a few weeks ago in their flurry of untested patches. But it does show the inherent advantage of open source in that *anyone* can review the code, and fix it, without resorting to messy hacks such as this.

  50. Re:The time problem has nothing to do with the pat by 1lus10n · · Score: 3, Informative

    To quote: "MS TRIES to make sure that their patches don't break 3rd party apps."

    Bullshit ! MS only tests for apps that have parent companies they get along with (also known has, they haven't tried to start a monopoly in that market yet.). As a matter of fact they were convicted in court of releasing patches that BROKE third party functionality on PURPOSE.

    Who ever modded you as insightful was an ass.

    --
    "Two things are infinite: the universe and human stupidity; and I'm not sure about the the universe." --Albert Einstein
  51. Re:Holy FuckBalls by Anonymous Coward · · Score: 3, Informative
    Don't program much do you? Tell you what sunshine why don't you tell me what this if statement does:
    if (NULL != strstr(dest,"\2") || NULL != strstr(dest,"\1") || NULL != strstr(dest,"\218"))
    Since you missed it the first time let me spell it out. They search the destination string for a "\2", "\1" or a "\218". If they find it then they redirect to their own server sending if the offending URL.
  52. Re:Hey, morons by Anonymous Coward · · Score: 5, Interesting

    hmm...
    In ::BeforeNavigateEvent (IETray.cpp)
    It copies the string to a MBCS buffer, and scans for %01, %02, and %DA. If none of these exist, the rest of the function is skipped. Don't see how this phones home.

    Of course, the strings is malloc()ed but never free()ed... But that's another matter. That and for some reason they don't just use all-unicode (use wcsstr() etc.)... What if I wanted to surf to a site with a character that is not in the current code page? (e.g., search for Japanese text on Google using an English O/S) (Note that IE has the option of always sending the URL in UTF-8, so it has to be able to deal with characters not in the ACP)

  53. Re:Hey, morons by KarmaPolice · · Score: 4, Insightful

    You do realize this patch phones home, don't you? Slashdot just advertised a piece of spyware. It phones home to validate every URL. Read the website.

    The patch is open source. I don't even know if you are right in your statement but if you are, then download the source and change the way it works! Or live in fear...

  54. No, not so much. by Jeff+Mahoney · · Score: 4, Informative

    Check the code again.

    The only URLs that get sent to their servers are the ones that it's filtering out, ones that would normally exploit the bug. At the other end (granted, at least for now) is an IE-lookalike error message saying that the exploit was caught.

    The first line before all that stuff involving redirection through their servers:
    if (NULL != strstr(dest,"\2") || NULL != strstr(dest,"\1") || NULL != strstr(dest,"\218"))

    It only matches URLs containing %01, %02, or %8F, which doesn't really "fix" the problem, but it's at least a workaround.

  55. Re:Avoiding buffer overflows in C by Ninja+Programmer · · Score: 4, Informative
    Here's the rule:
    Instead of using any of
    strcat(), strcpy(), sprintf(), gets()

    you use
    strncat(), strncpy(), snprintf(), fgets()
    This is hardly a sufficient recommendation for significantly reducing buffer overflow problems in C code. It changes the problem into a length management problem, where the unskilled C coder (after all, didn't they have a buffer overflow in their code in the first place?) is not necessarily going to fare any better.

    If you want to really reduce buffer overflow problems I suggest you visit the following two web pages:

    The Better String Library

    and

    Getting user Input

    I personally guarantee that buffer overflows in your code will dramatically decrease if you use the ideas spoken of and the source code on those pages.
  56. Use MyIE2 0.9.11 by SuckItTrebek · · Score: 3, Insightful

    You should use MyIE2 instead, http://www.myie2.com Fixed "IE URL Spoofing Vulnerability" problem. You also get the following: Tabbed Browsing Interface Mouse Gestures Super Drag&Drop Privacy Protection AD Hunter Google Bar Support External Utility Bar Skinning What else could you ask for?