Frame Buster Buster buster code needed

Question

Let s say you don t want other sites to  frame  your site in an  lt iframe gt     lt iframe src  http   example org  gt  lt  iframe gt    So you insert anti-framing  frame busting JavaScript into all your pages      break us out of any containing iframes    if  top    self    top location replace self location href       Excellent  Now you  bust  or break out of any containing iframe automatically  Except for one small problem   As it turns out  your frame-busting code can be busted  as shown here    lt script type  text javascript  gt      var prevent bust   0       window onbeforeunload   function     prevent bust           setInterval function             if  prevent bust  gt  0              prevent bust -  2           window top location    http   example org page-which-responds-with-204                     1     lt  script gt    This code does the following    increments a counter every time the browser attempts to navigate away from the current page  via the window onbeforeunload event handler sets up a timer that fires every millisecond via setInterval    and if it sees the counter incremented  changes the current location to a server of the attacker s control that server serves up a page with HTTP status code 204  which does not cause the browser to navigate anywhere   My question is -- and this is more of a JavaScript puzzle than an actual problem -- how can you defeat the frame-busting buster    I had a few thoughts  but nothing worked in my testing    attempting to clear the onbeforeunload event via onbeforeunload   null had no effect adding an alert   stopped the process let the user know it was happening  but did not interfere with the code in any way  clicking OK lets the busting continue as normal I can t think of any way to clear the setInterval   timer   I m not much of a JavaScript programmer  so here s my challenge to you  hey buster  can you bust the frame-busting buster

User · Answer

If you add an alert right after the buster code, then the alert will stall the javascript thread, and it will let the page load. This is what StackOverflow does, and it busts out of my iframes, even when I use the frame busting buster. It also worked with my simple test page. This has only been tested in Firefox 3.5 and IE7 on windows.

Code:

<script type="text/javascript">
if (top != self){
  top.location.replace(self.location.href);
  alert("for security reasons bla bla bla");
}
</script>

User · Answer

After pondering this for a little while  I believe this will show them who s boss     if top    self      window open location href    top        Using  top as the target parameter for window open   will launch it in the same window

User · Answer

If you look at the values returned by setInterval   they are usually single digits  so you can usually disable all such interrupts with a single line of code   for  var j   0   j  lt  256     j  clearInterval j

User · Answer

I m going to be brave and throw my hat into the ring on this one  ancient as it is   see how many downvotes I can collect   Here is my attempt  which does seem to work everywhere I have tested it  Chrome20  IE8 and FF14     function         if  top    self            return             setInterval function             top location replace document location           setTimeout function                 var xhr   new XMLHttpRequest                xhr open                   get                    http   mysite tld page-that-takes-a-while-to-load                   false                            xhr send null              0          1           I placed this code in the  lt head gt  and called it from the end of the  lt body gt  to ensure my page is rendered before it starts arguing with the malicious code  don t know if this is the best approach  YMMV   How does it work      I hear you ask - well the honest answer is  I don t really know  It took a lot of fudging about to make it work everywhere I was testing  and the exact effect that it has varies slightly depending on where you run it   Here is the thinking behind it    Set a function to run at the lowest possible interval  The basic concept behind any of the realistic solutions I have seen is to fill up the scheduler with more events than the frame buster-buster has  Every time the function fires  try and change the location of the top frame  Fairly obvious requirement  Also schedule a function to run immediately which will take a long time to complete  thereby blocking the frame buster-buster from interfering with the location change   I chose a synchronous XMLHttpRequest because it s the only mechanism I can think of that doesn t require  or at least ask for  user interaction and doesn t chew up the user s CPU time    For my http   mysite tld page-that-takes-a-while-to-load  the target of the XHR  I used a PHP script that looks like this    lt  php sleep 5     What happens    Chrome and Firefox wait the 5 seconds while the XHR completes  then successfully redirect to the framed page s URL  IE redirects pretty much immediately   Can t you avoid the wait time in Chrome and Firefox   Apparently not  At first I pointed the XHR to a URL that would return a 404 - this didn t work in Firefox  Then I tried the sleep 5   approach that I eventually landed on for this answer  then I started playing around with the sleep length in various ways  I could find no real pattern to the behaviour  but I did find that if it is too short  specifically Firefox will not play ball  Chrome and IE seem to be fairly well behaved   I don t know what the definition of  too short  is in real terms  but 5 seconds seems to work every time     If any passing Javascript ninjas want to explain a little better what s going on  why this is  probably  wrong  unreliable  the worst code they ve ever seen etc I ll happily listen

User · Answer

All the proposed solutions directly force a change in the location of the top window  What if a user wants the frame to be there  For example the top frame in the image results of search engines   I wrote a prototype where by default all inputs  links  forms and input elements  are disabled and or do nothing when activated   If a containing frame is detected  the inputs are left disabled and a warning message is shown at the top of the page  The warning message contains a link that will open a safe version of the page in a new window  This prevents the page from being used for clickjacking  while still allowing the user to view the contents in other situations   If no containing frame is detected  the inputs are enabled   Here is the code  You need to set the standard HTML attributes to safe values and add additonal attributes that contain the actual values  It probably is incomplete and for full safety additional attributes  I am thinking about event handlers  will probably have to be treated in the same way    lt  DOCTYPE HTML PUBLIC  -  W3C  DTD HTML 3 2 Final  EN  gt   lt html gt     lt head gt       lt title gt  lt  title gt       lt script gt  lt  --       function replaceAttributeValuesWithActualOnes  array  attributeName  actualValueAttributeName  additionalProcessor             for   var elementIndex   0  elementIndex  lt  array length  elementIndex    1               var element   array  elementIndex              var actualValue   element getAttribute  actualValueAttributeName              if   actualValue    null                 element  attributeName     actualValue                         if   additionalProcessor    null                 additionalProcessor  element                                         function detectFraming             if   top    self               document getElementById   framingWarning    style display    block             else             replaceAttributeValuesWithActualOnes  document links   href    acme href                replaceAttributeValuesWithActualOnes  document forms   action    acme action   function   form                 replaceAttributeValuesWithActualOnes  form elements   disabled    acme disabled                                              -- gt       lt  script gt     lt  head gt     lt body onload  detectFraming    gt       lt div id  framingWarning  style  display  none  border-style  solid  border-width  4px  border-color   F00  padding  6px  background-color   FFF  color   F00   gt         lt div gt           lt b gt SECURITY WARNING lt  b gt   Acme App is displayed inside another page          To make sure your data is safe this page has been disabled  lt br gt           lt a href  framing-detection html  target   blank  style  color   090  gt Continue working safely in a new tab window lt  a gt         lt  div gt       lt  div gt       lt p gt        Content   lt a href     acme href  javascript window alert   Action performed      gt Do something lt  a gt       lt  p gt       lt form name  acmeForm  action     acme action  real-action html  gt         lt p gt Name   lt input type  text  name  name  value    disabled  disabled  acme disabled    gt  lt  p gt         lt p gt  lt input type  submit  name  save  value  Save  disabled  disabled  acme disabled    gt  lt  p gt       lt  form gt     lt  body gt   lt  html gt

User · Answer

setInterval and setTimeout create an automatically incrementing interval   Each time setTimeout or setInterval is called  this number goes up by one  so that if you call setTimeout  you ll get the current  highest value      var currentInterval   10000     currentInterval    setTimeout  gotoHREF  100       for  var i   0  i  lt  currentInterval  i     top clearInterval  i          Include setTimeout to avoid recursive functions     for  i   0  i  lt  currentInterval  i         top clearTimeout  i        function gotoHREF               top location href    http   your url here          Since it is almost unheard of for there to be 10000 simultaneous setIntervals and setTimeouts working  and since setTimeout returns  last interval or timeout created   1   and since top clearInterval is still accessible  this will defeat the black-hat attacks to frame websites which are described above

User · Answer

Use htaccess to avoid high-jacking frameset  iframe and any content like images   RewriteEngine on RewriteCond   HTTP REFERER    http   www  yoursite  com   NC  RewriteCond   HTTP REFERER      RewriteRule         copyrights html  L    This will show a copyright page instead of the expected

User · Answer

As of 2015  you should use CSP2 s frame-ancestors directive for this  This is implemented via an HTTP response header   e g   Content-Security-Policy  frame-ancestors  none    Of course  not many browsers support CSP2 yet so it is wise to include the old X-Frame-Options header   X-Frame-Options  DENY   I would advise to include both anyway  otherwise your site would continue to be vulnerable to Clickjacking attacks in old browsers  and of course you would get undesirable framing even without malicious intent  Most browsers do update automatically these days  however you still tend to get corporate users being stuck on old versions of Internet Explorer for legacy application compatibility reasons

User · Answer

I think you were almost there   Have you tried   window parent onbeforeunload   null  window parent location replace self location href     or  alternatively   window parent prevent bust   0    Note  I didn t actually test this

User · Answer

What about calling the buster repeatedly as well  This ll create a race condition  but one may hope that the buster comes out on top    function         if top     self            top location href   self location href          setTimeout arguments callee  0

User · Answer

I m not sure if this is viable or not - but if you can t break the frame  why not just display a warning  For example  If your page isn t the  quot top page quot  create a setInterval method that tries to break the frame  If after 3 or 4 tries your page still isn t the top page - create a div element that covers the whole page  modal box  with a message and a link like     You are viewing this page in a unauthorized frame window -  Blah blah    potential security issue  click this link to fix this problem  Not the best  but I don t see any way they could script their way out of that

User · Answer

You could improve the whole idea by using the postMessage   method to allow some  domains to access and display your content while blocking all the others  First  the container-parent must introduce itself by posting a message to the contentWindow of the iframe that is trying to display your page  And your page must be ready to accept messages  window addEventListener  quot message quot   receiveMessage  false    function receiveMessage event         Use event origin here like   if event origin     quot https   perhapsyoucantrustthisdomain com quot         code here to block unblock access     a method like the one in user1646111 s post can be good        else       code here to block unblock access     a method like the one in user1646111 s post can be good         Finally don t forget to wrap things inside functions that will wait for load events

User · Answer

Considering current HTML5 standard that introduced sandbox for iframe  all frame busting codes that provided in this page can be disabled when attacker uses sandbox because it restricts the iframe from following   allow-forms  Allow form submissions  allow-popups  Allow opening popup windows  allow-pointer-lock  Allow access to pointer movement and pointer lock  allow-same-origin  Allow access to DOM objects when the iframe loaded form same origin allow-scripts  Allow executing scripts inside iframe allow-top-navigation  Allow navigation to top level window   Please see  http   www whatwg org specs web-apps current-work multipage the-iframe-element html attr-iframe-sandbox  Now  consider attacker used the following code to host your site in iframe    lt iframe src  URI  sandbox gt  lt  iframe gt    Then  all JavaScript frame busting code will fail    After checking all frame busing code  only this defense works in all cases    lt style id  antiClickjack  gt body display none  important   lt  style gt   lt script type  text javascript  gt     if  self     top           var antiClickjack   document getElementById  antiClickjack           antiClickjack parentNode removeChild antiClickjack        else          top location   self location        lt  script gt    that originally proposed by Gustav Rydstedt  Elie Bursztein  Dan Boneh  and Collin Jackson  2010

User · Answer

Ok  so we know that were in a frame  So we location href to another special page with the path as a GET variable   We now explain to the user what is going on and provide a link with a target   TOP  option  It s simple and would probably work  haven t tested it   but it requires some user interaction  Maybe you could point out the offending site to the user and make a hall of shame of click jackers to your site somewhere   Just an idea  but it night work

User · Answer

if  top    self      top location replace location     location replace  about blank       want me framed  no way

User · Answer

We have used the following approach in one of our websites from http   seclab stanford edu websec framebusting framebust pdf   lt style gt   body     display   none       lt  style gt   lt script gt  if self    top    document getElementsByTagName  body   0  style display    block     else  top location   self location     lt  script gt

User · Answer

Came up with this  and it seems to work at least in Firefox and the Opera browser   if top    self     top onbeforeunload   function        top location replace self location href

User · Answer

Well  you can modify the value of the counter  but that is obviously a brittle solution  You can load your content via AJAX after you have determined the site is not within a frame - also not a great solution  but it hopefully avoids firing the on beforeunload event  I am assuming    Edit  Another idea  If you detect you are in a frame  ask the user to disable javascript  before clicking on a link that takes you to the desired URL  passing a querystring that lets your page know to tell the user that they can re-enable javascript once they are there    Edit 2  Go nuclear - if you detect you are in a frame  just delete your document body content and print some nasty message   Edit 3  Can you enumerate the top document and set all functions to null  even anonymous ones

User · Answer

FWIW  most current browsers support the X-Frame-Options  deny directive  which works even when script is disabled   IE8  http   blogs msdn com ie archive 2009 01 27 ie8-security-part-vii-clickjacking-defenses aspx  Firefox  3 6 9  https   bugzilla mozilla org show bug cgi id 475530 https   developer mozilla org en The X-FRAME-OPTIONS response header  Chrome Webkit http   blog chromium org 2010 01 security-in-depth-new-security-features html http   trac webkit org changeset 42333

User · Answer

I might just have just gotten a way to bust the frame buster buster javascript  Using the getElementsByName in my javascript function  i ve set a loop between the frame buster and the actual frame buster buster script  check this post out  http   www phcityonweb com frame-buster-buster-buster-2426

[javascript] Frame Buster Buster ... buster code needed

Examples related to javascript

Examples related to html

Examples related to iframe

Examples related to framebusting