diff options
author | Mike Macgirvin <mike@macgirvin.com> | 2010-08-05 02:57:03 -0700 |
---|---|---|
committer | Mike Macgirvin <mike@macgirvin.com> | 2010-08-05 02:57:03 -0700 |
commit | 40a06771ff2a22f3801dfe174ca318cf4f67c1b1 (patch) | |
tree | 118bb7d76726b68d2155229b8039170e5c500d9e /include/notifier.php | |
parent | b294ff63024da5629a0d7b6f66aaa9af6d1dd400 (diff) | |
download | volse-hubzilla-40a06771ff2a22f3801dfe174ca318cf4f67c1b1.tar.gz volse-hubzilla-40a06771ff2a22f3801dfe174ca318cf4f67c1b1.tar.bz2 volse-hubzilla-40a06771ff2a22f3801dfe174ca318cf4f67c1b1.zip |
reciprocal verification on notify, poll
Diffstat (limited to 'include/notifier.php')
-rw-r--r-- | include/notifier.php | 14 |
1 files changed, 12 insertions, 2 deletions
diff --git a/include/notifier.php b/include/notifier.php index a2d313b5f..7d9b09da3 100644 --- a/include/notifier.php +++ b/include/notifier.php @@ -267,12 +267,22 @@ echo $xml; $res = simplexml_load_string($xml); - if((intval($res->status) != 0) || (! strlen($res->challenge)) || ($res->dfrn_id != $rr['dfrn-id'])) + if((intval($res->status) != 0) || (! strlen($res->challenge)) || (! strlen($res->dfrn_id))) continue; $postvars = array(); + $sent_dfrn_id = hex2bin($res->dfrn_id); + + $final_dfrn_id = ''; + openssl_public_decrypt($sent_dfrn_id,$final_dfrn_id,$rr['pubkey']); + $final_dfrn_id = substr($final_dfrn_id, 0, strpos($final_dfrn_id, '.')); + if($final_dfrn_id != $rr['dfrn-id']) { + // did not decode properly - cannot trust this site + continue; + } $postvars['dfrn_id'] = $rr['dfrn-id']; + $challenge = hex2bin($res->challenge); openssl_public_decrypt($challenge,$postvars['challenge'],$rr['pubkey']); @@ -295,7 +305,7 @@ echo $xml; // Currently there is no retry attempt for failed mail delivery. // We need to handle this in the UI, report the non-deliverables and try again - if(($cmd == 'mail) && (intval($res->status) == 0)) { + if(($cmd == 'mail') && (intval($res->status) == 0)) { $r = q("UPDATE `mail` SET `delivered` = 1 WHERE `id` = %d LIMIT 1", intval($item_id) |