Friendica/friendica-github
1
0
Fork 0
mirror of https://github.com/friendica/friendica synced 2025-01-25 18:59:46 +00:00
Code Issues Projects Releases Packages Wiki Activity

Fix formatting include/security

Browse source
This commit is contained in:
Hypolite Petovan 2017-12-16 19:19:51 -05:00
parent 6d6d83012c
commit 09467be02a
1 changed files with 61 additions and 81 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

104
include/security.php
View file

@ -13,7 +13,8 @@ use Friendica\Database\DBM;
* *
* @return string Hashed data * @return string Hashed data
*/ */
function cookie_hash($user) { function cookie_hash($user)
{
return(hash("sha256", Config::get("system", "site_prvkey") . return(hash("sha256", Config::get("system", "site_prvkey") .
$user["prvkey"] . $user["prvkey"] .
$user["password"])); $user["password"]));
@ -25,8 +26,8 @@ function cookie_hash($user) {
* @param int $time * @param int $time
* @param array $user Record from "user" table * @param array $user Record from "user" table
*/ */
function new_cookie($time, $user = array()) { function new_cookie($time, $user = array())
{
if ($time != 0) { if ($time != 0) {
$time = $time + time(); $time = $time + time();
} }
@ -35,18 +36,25 @@ function new_cookie($time, $user = array()) {
$value = json_encode(array("uid" => $user["uid"], $value = json_encode(array("uid" => $user["uid"],
"hash" => cookie_hash($user), "hash" => cookie_hash($user),
"ip" => $_SERVER['REMOTE_ADDR'])); "ip" => $_SERVER['REMOTE_ADDR']));
} } else {
else {
$value = ""; $value = "";
} }
setcookie("Friendica", $value, $time, "/", "", setcookie("Friendica", $value, $time, "/", "", (Config::get('system', 'ssl_policy') == SSL_POLICY_FULL), true);
(Config::get('system', 'ssl_policy') == SSL_POLICY_FULL), true);
} }
function authenticate_success($user_record, $login_initial = false, $interactive = false, $login_refresh = false) { /**
* @brief Sets the provided user's authenticated session
*
* @todo Should be moved to Friendica\Core\Session once it's created
*
* @param type $user_record
* @param type $login_initial
* @param type $interactive
* @param type $login_refresh
*/
function authenticate_success($user_record, $login_initial = false, $interactive = false, $login_refresh = false)
{
$a = get_app(); $a = get_app();
$_SESSION['uid'] = $user_record['uid']; $_SESSION['uid'] = $user_record['uid'];
@ -156,13 +164,11 @@ function authenticate_success($user_record, $login_initial = false, $interactive
} }
} }
function can_write_wall(App $a, $owner)
{
function can_write_wall(App $a, $owner) {
static $verified = 0; static $verified = 0;
if ((! (local_user())) && (! (remote_user()))) { if (!local_user() && !remote_user()) {
return false; return false;
} }
@ -173,10 +179,8 @@ function can_write_wall(App $a, $owner) {
} }
if (remote_user()) { if (remote_user()) {
// use remembered decision and avoid a DB lookup for each and every display item // use remembered decision and avoid a DB lookup for each and every display item
// DO NOT use this function if there are going to be multiple owners // DO NOT use this function if there are going to be multiple owners
// We have a contact-id for an authenticated remote user, this block determines if the contact // We have a contact-id for an authenticated remote user, this block determines if the contact
// belongs to this page owner, and has the necessary permissions to post content // belongs to this page owner, and has the necessary permissions to post content
@ -213,8 +217,7 @@ function can_write_wall(App $a, $owner) {
if (DBM::is_result($r)) { if (DBM::is_result($r)) {
$verified = 2; $verified = 2;
return true; return true;
} } else {
else {
$verified = 1; $verified = 1;
} }
} }
@ -223,9 +226,8 @@ function can_write_wall(App $a, $owner) {
return false; return false;
} }
function permissions_sql($owner_id, $remote_verified = false, $groups = null)
function permissions_sql($owner_id, $remote_verified = false, $groups = null) { {
$local_user = local_user(); $local_user = local_user();
$remote_user = remote_user(); $remote_user = remote_user();
@ -243,8 +245,7 @@ function permissions_sql($owner_id, $remote_verified = false, $groups = null) {
/** /**
* Profile owner - everything is visible * Profile owner - everything is visible
*/ */
if ($local_user && $local_user == $owner_id) {
if (($local_user) && ($local_user == $owner_id)) {
$sql = ''; $sql = '';
} elseif ($remote_user) { } elseif ($remote_user) {
/* /*
@ -265,8 +266,8 @@ function permissions_sql($owner_id, $remote_verified = false, $groups = null) {
$groups = init_groups_visitor($remote_user); $groups = init_groups_visitor($remote_user);
} }
} }
if ($remote_verified) {
if ($remote_verified) {
$gs = '<<>>'; // should be impossible to match $gs = '<<>>'; // should be impossible to match
if (is_array($groups) && count($groups)) { if (is_array($groups) && count($groups)) {
@ -274,20 +275,6 @@ function permissions_sql($owner_id, $remote_verified = false, $groups = null) {
$gs .= '|<' . intval($g) . '>'; $gs .= '|<' . intval($g) . '>';
} }
/*
* @TODO old-lost code found?
$sql = sprintf(
" AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' )
AND ( deny_cid = '' OR NOT deny_cid REGEXP '<%d>' )
AND ( allow_gid = '' OR allow_gid REGEXP '%s' )
AND ( deny_gid = '' OR NOT deny_gid REGEXP '%s')
",
intval($remote_user),
intval($remote_user),
dbesc($gs),
dbesc($gs)
);
*/
$sql = sprintf( $sql = sprintf(
" AND ( NOT (deny_cid REGEXP '<%d>' OR deny_gid REGEXP '%s') " AND ( NOT (deny_cid REGEXP '<%d>' OR deny_gid REGEXP '%s')
AND ( allow_cid REGEXP '<%d>' OR allow_gid REGEXP '%s' OR ( allow_cid = '' AND allow_gid = '') ) AND ( allow_cid REGEXP '<%d>' OR allow_gid REGEXP '%s' OR ( allow_cid = '' AND allow_gid = '') )
@ -303,13 +290,12 @@ function permissions_sql($owner_id, $remote_verified = false, $groups = null) {
return $sql; return $sql;
} }
function item_permissions_sql($owner_id, $remote_verified = false, $groups = null)
function item_permissions_sql($owner_id, $remote_verified = false, $groups = null) { {
$local_user = local_user(); $local_user = local_user();
$remote_user = remote_user(); $remote_user = remote_user();
/** /*
* Construct permissions * Construct permissions
* *
* default permissions - anonymous user * default permissions - anonymous user
@ -321,9 +307,7 @@ function item_permissions_sql($owner_id, $remote_verified = false, $groups = nul
AND `item`.private = 0 AND `item`.private = 0
"; ";
/** // Profile owner - everything is visible
* Profile owner - everything is visible
*/
if ($local_user && ($local_user == $owner_id)) { if ($local_user && ($local_user == $owner_id)) {
$sql = ''; $sql = '';
} elseif ($remote_user) { } elseif ($remote_user) {
@ -355,16 +339,6 @@ function item_permissions_sql($owner_id, $remote_verified = false, $groups = nul
} }
$sql = sprintf( $sql = sprintf(
/*" AND ( private = 0 OR ( private in (1,2) AND wall = 1 AND ( allow_cid = '' OR allow_cid REGEXP '<%d>' )
AND ( deny_cid = '' OR NOT deny_cid REGEXP '<%d>' )
AND ( allow_gid = '' OR allow_gid REGEXP '%s' )
AND ( deny_gid = '' OR NOT deny_gid REGEXP '%s')))
",
intval($remote_user),
intval($remote_user),
dbesc($gs),
dbesc($gs)
*/
" AND ( `item`.private = 0 OR ( `item`.private in (1,2) AND `item`.`wall` = 1 " AND ( `item`.private = 0 OR ( `item`.private in (1,2) AND `item`.`wall` = 1
AND ( NOT (`item`.deny_cid REGEXP '<%d>' OR `item`.deny_gid REGEXP '%s') AND ( NOT (`item`.deny_cid REGEXP '<%d>' OR `item`.deny_gid REGEXP '%s')
AND ( `item`.allow_cid REGEXP '<%d>' OR `item`.allow_gid REGEXP '%s' OR ( `item`.allow_cid = '' AND `item`.allow_gid = ''))))) AND ( `item`.allow_cid REGEXP '<%d>' OR `item`.allow_gid REGEXP '%s' OR ( `item`.allow_cid = '' AND `item`.allow_gid = '')))))
@ -380,7 +354,6 @@ function item_permissions_sql($owner_id, $remote_verified = false, $groups = nul
return $sql; return $sql;
} }
/* /*
* Functions used to protect against Cross-Site Request Forgery * Functions used to protect against Cross-Site Request Forgery
* The security token has to base on at least one value that an attacker can't know - here it's the session ID and the private key. * The security token has to base on at least one value that an attacker can't know - here it's the session ID and the private key.
@ -392,7 +365,8 @@ function item_permissions_sql($owner_id, $remote_verified = false, $groups = nul
* Actually, important actions should not be triggered by Links / GET-Requests at all, but somethimes they still are, * Actually, important actions should not be triggered by Links / GET-Requests at all, but somethimes they still are,
* so this mechanism brings in some damage control (the attacker would be able to forge a request to a form of this type, but not to forms of other types). * so this mechanism brings in some damage control (the attacker would be able to forge a request to a form of this type, but not to forms of other types).
*/ */
function get_form_security_token($typename = '') { function get_form_security_token($typename = '')
{
$a = get_app(); $a = get_app();
$timestamp = time(); $timestamp = time();
@ -401,7 +375,8 @@ function get_form_security_token($typename = '') {
return $timestamp . '.' . $sec_hash; return $timestamp . '.' . $sec_hash;
} }
function check_form_security_token($typename = '', $formname = 'form_security_token') { function check_form_security_token($typename = '', $formname = 'form_security_token')
{
if (!x($_REQUEST, $formname)) { if (!x($_REQUEST, $formname)) {
return false; return false;
} }
@ -423,10 +398,13 @@ function check_form_security_token($typename = '', $formname = 'form_security_to
return ($sec_hash == $x[1]); return ($sec_hash == $x[1]);
} }
function check_form_security_std_err_msg() { function check_form_security_std_err_msg()
{
return t('The form security token was not correct. This probably happened because the form has been opened for too long (>3 hours) before submitting it.') . EOL; return t('The form security token was not correct. This probably happened because the form has been opened for too long (>3 hours) before submitting it.') . EOL;
} }
function check_form_security_token_redirectOnErr($err_redirect, $typename = '', $formname = 'form_security_token') {
function check_form_security_token_redirectOnErr($err_redirect, $typename = '', $formname = 'form_security_token')
{
if (!check_form_security_token($typename, $formname)) { if (!check_form_security_token($typename, $formname)) {
$a = get_app(); $a = get_app();
logger('check_form_security_token failed: user ' . $a->user['guid'] . ' - form element ' . $typename); logger('check_form_security_token failed: user ' . $a->user['guid'] . ' - form element ' . $typename);
@ -435,7 +413,9 @@ function check_form_security_token_redirectOnErr($err_redirect, $typename = '',
goaway(System::baseUrl() . $err_redirect); goaway(System::baseUrl() . $err_redirect);
} }
} }
function check_form_security_token_ForbiddenOnErr($typename = '', $formname = 'form_security_token') {
function check_form_security_token_ForbiddenOnErr($typename = '', $formname = 'form_security_token')
{
if (!check_form_security_token($typename, $formname)) { if (!check_form_security_token($typename, $formname)) {
$a = get_app(); $a = get_app();
logger('check_form_security_token failed: user ' . $a->user['guid'] . ' - form element ' . $typename); logger('check_form_security_token failed: user ' . $a->user['guid'] . ' - form element ' . $typename);