From 27bfd1211d64298ee3c3b2d82d7b38ca1e1167ad Mon Sep 17 00:00:00 2001
From: Brion Vibber <brion@pobox.com>
Date: Mon, 22 Mar 2010 12:17:45 -0700
Subject: Math_BigInteger doesn't correctly handle
 serialization/deserialization for a value of 0, which can end up spewing
 notices to output and otherwise intefering with Salmon signature setup and
 verification when using memcached. Worked around this with a subclass that
 fixes the wakeup, used for the stored 0 value in the subclassed Crypt_RSA.

---
 plugins/OStatus/lib/safecrypt_rsa.php       | 18 ++++++++++++++++++
 plugins/OStatus/lib/safemath_biginteger.php | 20 ++++++++++++++++++++
 2 files changed, 38 insertions(+)
 create mode 100644 plugins/OStatus/lib/safecrypt_rsa.php
 create mode 100644 plugins/OStatus/lib/safemath_biginteger.php

(limited to 'plugins/OStatus/lib')

diff --git a/plugins/OStatus/lib/safecrypt_rsa.php b/plugins/OStatus/lib/safecrypt_rsa.php
new file mode 100644
index 000000000..f3aa2c928
--- /dev/null
+++ b/plugins/OStatus/lib/safecrypt_rsa.php
@@ -0,0 +1,18 @@
+<?php
+
+require_once 'Crypt/RSA.php';
+
+/**
+ * Crypt_RSA stores a Math_BigInteger with value 0, which triggers a bug
+ * in Math_BigInteger's wakeup function which spews notices to log or output.
+ * This wrapper replaces it with a version that survives serialization.
+ */
+class SafeCrypt_RSA extends Crypt_RSA
+{
+    function __construct()
+    {
+        parent::__construct();
+        $this->zero = new SafeMath_BigInteger();
+    }
+}
+
diff --git a/plugins/OStatus/lib/safemath_biginteger.php b/plugins/OStatus/lib/safemath_biginteger.php
new file mode 100644
index 000000000..c05e24d1e
--- /dev/null
+++ b/plugins/OStatus/lib/safemath_biginteger.php
@@ -0,0 +1,20 @@
+<?php
+
+require_once 'Math/BigInteger.php';
+
+/**
+ * Crypt_RSA stores a Math_BigInteger with value 0, which triggers a bug
+ * in Math_BigInteger's wakeup function which spews notices to log or output.
+ * This wrapper replaces it with a version that survives serialization.
+ */
+class SafeMath_BigInteger extends Math_BigInteger
+{
+    function __wakeup()
+    {
+        if ($this->hex == '') {
+            $this->hex = '0';
+        }
+        parent::__wakeup();
+    }
+}
+
-- 
cgit v1.2.3-54-g00ecf