Skip to content

Commit fd4bf52

Browse files
authored
Merge pull request #1 from iliakan/patch-1
2 parents b1b1699 + 2affd9d commit fd4bf52

File tree

1 file changed

+28
-30
lines changed

1 file changed

+28
-30
lines changed

4-frames-and-windows/06-clickjacking/article.md

+28-30
Original file line numberDiff line numberDiff line change
@@ -4,22 +4,20 @@ The "clickjacking" attack allows an evil page to click on a "victim site" *on be
44

55
Many sites were hacked this way, including Twitter, Facebook, Paypal and other sites. They are all fixed, of course.
66

7-
[cut]
8-
97
## The idea
108

119
The idea is very simple.
1210

1311
Here's how clickjacking was done with Facebook:
1412

15-
1. A visitor is lured to the evil page. No matter how.
16-
2. The page has a harmlessly-looking link on it (like "get rich now" or "click here, very funny" and so on).
13+
1. A visitor is lured to the evil page. It doesn't matter how.
14+
2. The page has a harmless-looking link on it (like "get rich now" or "click here, very funny").
1715
3. Over that link the evil page positions a transparent `<iframe>` with `src` from facebook.com, in such a way that the "Like" button is right above that link. Usually that's done with `z-index`.
18-
4. Clicking on that link, the visitor in fact presses that button.
16+
4. In attempting to click the link, the visitor in fact clicks the button.
1917

2018
## The demo
2119

22-
Here's how the evil page looks like. To make things clear, the `<iframe>` is half-transparent (in real evil pages it's fully transparent):
20+
Here's how the evil page looks. To make things clear, the `<iframe>` is half-transparent (in real evil pages it's fully transparent):
2321

2422
```html run height=120 no-beautify
2523
<style>
@@ -53,7 +51,7 @@ The full demo of the attack:
5351

5452
Here we have a half-transparent `<iframe src="facebook.html">`, and in the example we can see it hovering over the button. A click on the button actually clicks on the iframe, but that's not visible to the user, because the iframe is transparent.
5553

56-
As a result if the visitor is authorized on facebook ("remember me" is usually turned on), then it adds a "Like". On Twitter that would be a "Follow" button.
54+
As a result, if the visitor is authorized on Facebook ("remember me" is usually turned on), then it adds a "Like". On Twitter that would be a "Follow" button.
5755

5856
Here's the same example, but closer to reality, with `opacity:0` for `<iframe>`:
5957

@@ -68,30 +66,30 @@ Technically, if we have a text field to hack, then we can position an iframe in
6866
6967
But then there's a problem. Everything that the visitor types will be hidden, because the iframe is not visible.
7068
71-
So that would look really odd to the user, and he will stop.
69+
People will usually stop typing when they can't see their new characters printing on the screen.
7270
```
7371

7472
## Old-school defences (weak)
7573

76-
The oldest defence method is the piece of JavaScript that forbids to open the page in a frame (so-called "framebusting").
74+
The oldest defence is a bit of JavaScript which forbids opening the page in a frame (so-called "framebusting").
7775

78-
Like this:
76+
That looks like this:
7977

8078
```js
8179
if (top != window) {
8280
top.location = window.location;
8381
}
8482
```
8583

86-
That is: if the window finds out that it's not on the top, then it automatically makes itself the top.
84+
That is: if the window finds out that it's not on top, then it automatically makes itself the top.
8785

88-
As of now, that's not reliable, because there are many ways to hack around it. Let's cover a few.
86+
This not a reliable defence, because there are many ways to hack around it. Let's cover a few.
8987

9088
### Blocking top-navigation
9189

9290
We can block the transition caused by changing `top.location` in the [beforeunload](info:onload-ondomcontentloaded#window.onbeforeunload) event.
9391

94-
The top page (that belongs to the hacker) sets a handler to it, and when the `iframe` tries to change `top.location` the visitor gets a message asking him whether he wants to leave.
92+
The top page (belonging to the hacker) sets a handler to it, and when the `iframe` tries to change `top.location` the visitor gets a message asking him whether he wants to leave.
9593

9694
Like this:
9795
```js
@@ -101,7 +99,7 @@ window.onbeforeunload = function() {
10199
};
102100
```
103101

104-
In most cases the visitor would answer negatively, because he doesn't know about the iframe, all he can see is the top page, there's no reason to leave. And so the `top.location` won't change!
102+
In most cases the visitor would answer negatively, because he doesn't know about the iframe, all he can see is the top page, leading him to think there is no reason to leave. So `top.location` won't change!
105103

106104
In action:
107105

@@ -111,7 +109,7 @@ In action:
111109

112110
One of the things restricted by the `sandbox` attribute is navigation. A sandboxed iframe may not change `top.location`.
113111

114-
So we can add the iframe with `sandbox="allow-scripts allow-forms"`. That would relax the restrictions allowing scripts and forms. But we don't put `allow-top-navigation` in the value so that the navigation is still forbidden. And the change of `top.location` won't work.
112+
So we can add the iframe with `sandbox="allow-scripts allow-forms"`. That would relax the restrictions, permitting scripts and forms. But we omit `allow-top-navigation` so that changing `top.location` is forbidden.
115113

116114
Here's the code:
117115

@@ -123,21 +121,21 @@ There are other ways to work around that simple protection too.
123121

124122
## X-Frame-Options
125123

126-
Server-side header `X-Frame-Options` can allow or forbid showing the page inside a frame.
124+
The server-side header `X-Frame-Options` can permit or forbid displaying the page inside a frame.
127125

128-
It must be sent by the server: browser ignore it if found in `<meta>` tags. So `<meta http-equiv="X-Frame-Options"...>` won't do anything.
126+
It must be sent *by the server*: the browser will ignore it if found in a `<meta>` tag. So, `<meta http-equiv="X-Frame-Options"...>` won't do anything.
129127

130128
The header may have 3 values:
131129

132130

133131
`DENY`
134-
: Never ever show the page inside an iframe.
132+
: Never ever show the page inside a frame.
135133

136134
`SAMEORIGIN`
137-
: Allow to show inside an iframe if the parent document comes from the same origin.
135+
: Allow inside a frame if the parent document comes from the same origin.
138136

139137
`ALLOW-FROM domain`
140-
: Allows to show inside an iframe if the parent document is from the given domain.
138+
: Allow inside a frame if the parent document is from the given domain.
141139

142140
For instance, Twitter uses `X-Frame-Options: SAMEORIGIN`. Here's the result:
143141

@@ -147,15 +145,15 @@ For instance, Twitter uses `X-Frame-Options: SAMEORIGIN`. Here's the result:
147145

148146
<iframe src="https://twitter.com"></iframe>
149147

150-
Depending on the browser, `iframe` above is either empty or it has a message telling that "the browser can't show it".
148+
Depending on your browser, the `iframe` above is either empty or alerting you that the browser won't permit that page to be navigating in this way.
151149

152150
## Showing with disabled functionality
153151

154-
The protecting `X-Frame-Options` header has a side-effect. Other sites can't show our page in an `iframe`, even if they have "legal" reasons to do so.
152+
The `X-Frame-Options` header has a side-effect. Other sites won't be able to show our page in a frame, even if they have good reasons to do so.
155153

156-
So there are other solutions. For instance, we can "cover" the page with a `<div>` with `height:100%;width:100%`, so that it handles all clicks. That `<div>` should disappear if `window == top` or we figure out that we don't need protection.
154+
So there are other solutions... For instance, we can "cover" the page with a `<div>` with `height: 100%; width: 100%;`, so that it intercepts all clicks. That `<div>` should disappear if `window == top` or if we figure out that we don't need the protection.
157155

158-
Like this:
156+
Something like this:
159157

160158
```html
161159
<style>
@@ -188,13 +186,13 @@ The demo:
188186

189187
## Summary
190188

191-
Clickjacking is a way to "trick" users into a clicking on a victim site without even knowing what happens. That's dangerous if there are important click-activated actions.
189+
Clickjacking is a way to "trick" users into clicking on a malicious site without even knowing what's happening. That's dangerous if there are important click-activated actions.
192190

193-
A hacker can post a link to his evil page in a message or lure visitors to his page by other means. There are many variants.
191+
A hacker can post a link to his evil page in a message, or lure visitors to his page by some other means. There are many variations.
194192

195-
From one side -- the attack is "not deep": all a hacker can do is one click. But from another side, if the hacker knows that after the click another control appears, then it may use cunning messages to make the user to click on it as well.
193+
From one perspective -- the attack is "not deep": all a hacker is doing is intercepting a single click. But from another perspective, if the hacker knows that after the click another control will appear, then he may use cunning messages to coerce the user into clicking on them as well.
196194

197-
The attack is quite dangerous, because when we engineer the UI we usually don't think that a hacker can click on behalf of the visitor. So vulnerabilities can be found in totally unexpected places.
195+
The attack is quite dangerous, because when we engineer the UI we usually don't anticipate that a hacker may click on behalf of the visitor. So vulnerabilities can be found in totally unexpected places.
198196

199-
- It's recommended to use `X-Frame-Options: SAMEORIGIN` on pages that are totally not meant to be shown inside iframes (or just for the whole site).
200-
- Use a covering `<div>` if we want to allow our pages to be shown in iframes, and still stay safe.
197+
- It is recommended to use `X-Frame-Options: SAMEORIGIN` on pages (or whole websites) which are not intended to be viewed inside frames.
198+
- Use a covering `<div>` if we want to allow our pages to be shown in iframes, but still stay safe.

0 commit comments

Comments
 (0)