aboutsummaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorFriendika <info@friendika.com>2010-12-02 21:09:55 -0800
committerFriendika <info@friendika.com>2010-12-02 21:09:55 -0800
commit4d84a87ce4b93061d46958f70ac9c833caa800cd (patch)
tree4aa132bb437faf9bb83b724cb383cc63c7790f36 /include
parent041933db8794244aa894c0412aab421b6412837a (diff)
downloadvolse-hubzilla-4d84a87ce4b93061d46958f70ac9c833caa800cd.tar.gz
volse-hubzilla-4d84a87ce4b93061d46958f70ac9c833caa800cd.tar.bz2
volse-hubzilla-4d84a87ce4b93061d46958f70ac9c833caa800cd.zip
cache result of (expensive) security check for visitor rights
begin tightening x-profile security
Diffstat (limited to 'include')
-rw-r--r--include/security.php40
1 files changed, 28 insertions, 12 deletions
diff --git a/include/security.php b/include/security.php
index f37603916..5e79e1edd 100644
--- a/include/security.php
+++ b/include/security.php
@@ -2,15 +2,27 @@
function can_write_wall(&$a,$owner) {
- if((! (local_user())) && (! (remote_user())))
- return false;
- $uid = local_user();
+ static $verified = 0;
- if(($uid) && ($uid == $owner)) {
- return true;
- }
+ if((! (local_user())) && (! (remote_user())))
+ return false;
+
+ $uid = local_user();
+
+ if(($uid) && ($uid == $owner)) {
+ return true;
+ }
+
+ if(remote_user()) {
- if(remote_user()) {
+ // user 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
+
+ if($verified === 2)
+ return true;
+ elseif($verified === 1)
+ return false;
+ else {
$r = q("SELECT `contact`.*, `user`.`page-flags` FROM `contact` LEFT JOIN `user` on `user`.`uid` = `contact`.`uid`
WHERE `contact`.`uid` = %d AND `contact`.`id` = %d AND `contact`.`blocked` = 0 AND `contact`.`pending` = 0
AND `readonly` = 0 AND ( `contact`.`rel` IN ( %d , %d ) OR `user`.`page-flags` = %d ) LIMIT 1",
@@ -20,11 +32,15 @@ function can_write_wall(&$a,$owner) {
intval(REL_BUD),
intval(PAGE_COMMUNITY)
);
+ if(count($r)) {
+ $verified = 2;
+ return true;
+ }
+ else {
+ $verified = 1;
+ }
}
- if(count($r))
- return true;
-
-
- return false;
+ }
+ return false;
}