Thanks to visit codestin.com
Credit goes to github.com

Skip to content
This repository was archived by the owner on Jan 13, 2022. It is now read-only.

Conversation

@kilotaras
Copy link
Contributor

See the next scenario.

  1. Alice is logged into Facebook
  2. Alice visits example.com/with_js_sdk.php (our PHP SDK example endpoint). The SDK will write her signed_request, user_id, and access_token to a session cookie.
  3. Alice is then tricked into visiting example.com/with_js_sdk.php?signed_request={Bob's signed request}. The SDK will then set the user_id to Bob, while keeping Alice's access_token in persistent storage
  4. If example.com has another endpoint that authenticates based on getUser(), Bob now has access to data returned by Alice's access_token.

This fixes it.

oyvindkinsey added a commit that referenced this pull request Jan 15, 2013
Fixed a vulnerability with signed requests
@oyvindkinsey oyvindkinsey merged commit bf99924 into facebookarchive:master Jan 15, 2013
@kilotaras kilotaras deleted the vulnerability_bug branch January 15, 2013 22:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants