From 34ebb3cfa25d0cca42de50fe1b5126e12ff7f95b Mon Sep 17 00:00:00 2001
From: peterbulmer <peterbulmer>
Date: Sun, 28 Sep 2008 04:18:28 +0000
Subject: [PATCH] MDL-16286 MDL-16285 better variable names and improved
 structure Another one of those diffs that is a little more understandable
 when ignoring whitespace changes.

Author: Peter Bulmer <peter.bulmer@catalyst.net.nz>
---
 mnet/xmlrpc/client.php | 44 +++++++++++++++++++++---------------------
 1 file changed, 22 insertions(+), 22 deletions(-)

diff --git a/mnet/xmlrpc/client.php b/mnet/xmlrpc/client.php
index 43eaa46ac86..17ea7e9d449 100644
--- a/mnet/xmlrpc/client.php
+++ b/mnet/xmlrpc/client.php
@@ -252,32 +252,32 @@ class mnet_xmlrpc_client {
             // The faultString is the new key - let's save it and try again
             // The re_key attribute stops us from getting into a loop
             if($this->response['faultCode'] == 7025 && empty($mnet_peer->re_key)) {
-                $record                     = new stdClass();
-                $record->id                 = $mnet_peer->id;
-                if($this->response['faultString'] == clean_param($this->response['faultString'], PARAM_PEM)) {
-                    $record->public_key         = $this->response['faultString'];
-                    $details                    = openssl_x509_parse($record->public_key);
-                    if(is_array($details) && isset($details['validTo_time_t'])) {
-                        $record->public_key_expires = $details['validTo_time_t'];
-                        $DB->update_record('mnet_host', $record);
-                        $mnet_peer2 = new mnet_peer();
-                        $mnet_peer2->set_id($record->id);
-                        $mnet_peer2->re_key = true;
-                        $this->send($mnet_peer2);
-                    } else {
-                        $this->error[] = $this->response['faultCode'] . " : " . $this->response['faultString'];
-                    }
-                } else {
+                // If the new certificate doesn't come thru clean_param() unmolested, error out
+                if($this->response['faultString'] != clean_param($this->response['faultString'], PARAM_PEM)) {
                     $this->error[] = $this->response['faultCode'] . " : " . $this->response['faultString'];
                 }
-            } else {
-                if (!empty($CFG->mnet_rpcdebug)) {
-                    $guidance = get_string('error'.$this->response['faultCode'], 'mnet');
-                } else {
-                    $guidance = '';
+                $record                     = new stdClass();
+                $record->id                 = $mnet_peer->id;
+                $record->public_key         = $this->response['faultString'];
+                $details                    = openssl_x509_parse($record->public_key);
+                if(!isset($details['validTo_time_t'])) {
+                    $this->error[] = $this->response['faultCode'] . " : " . $this->response['faultString'];
                 }
-                $this->error[] = $this->response['faultCode'] . " : " . $this->response['faultString'] ."\n".$guidance;
+                $record->public_key_expires = $details['validTo_time_t'];
+                $DB->update_record('mnet_host', $record);
+
+                // Create a new peer object populated with the new info & try re-sending the request
+                $rekeyed_mnet_peer = new mnet_peer();
+                $rekeyed_mnet_peer->set_id($record->id);
+                $rekeyed_mnet_peer->re_key = true;
+                return $this->send($rekeyed_mnet_peer);
             }
+            if (!empty($CFG->mnet_rpcdebug)) {
+                $guidance = get_string('error'.$this->response['faultCode'], 'mnet');
+            } else {
+                $guidance = '';
+            }
+            $this->error[] = $this->response['faultCode'] . " : " . $this->response['faultString'] ."\n".$guidance;
         }
         return empty($this->error);
     }