Fix setting max packet size for DB caches and check packet size also in sha...

Aleksander Machniak

2015-02-19 81d4ff214e7cf2253c270a8f8de533bebbf88c15

Fix setting max packet size for DB caches and check packet size also in shared cache

 CHANGELOG

@@ -4,6 +4,7 @@
- Plugin API: Add special onload() method to execute plugin actions before startup (session and GUI initialization)
- Add possibility to print contact information (of a single contact)
- Fix refreshing of drafts list when sending a message which was saved in meantime (#1490238)
- Fix setting max packet size for DB caches and check packet size also in shared cache

RELEASE 1.1.0
-------------

 program/lib/Roundcube/rcube_cache.php

@@ -605,8 +605,8 @@
            $this->max_packet = 2097152; // default/max is 2 MB

            if ($this->type == 'db') {
                $value = $this->db->get_variable('max_allowed_packet', 1048500);
                $this->max_packet = min($value, $this->max_packet) - 2000;
                $value = $this->db->get_variable('max_allowed_packet', $this->max_packet);
                $this->max_packet = max($value, $this->max_packet) - 2000;
            }
            else if ($this->type == 'memcache') {
                $stats = $this->db->getStats();

 program/lib/Roundcube/rcube_cache_shared.php

@@ -44,6 +44,7 @@
    private $cache         = array();
    private $cache_changes = array();
    private $cache_sums    = array();
    private $max_packet    = -1;


    /**
@@ -312,13 +313,19 @@
     * Writes single cache record into DB.
     *
     * @param string $key  Cache key name
     * @param mxied  $data Serialized cache data 
     * @param mixed  $data Serialized cache data
     *
     * @param boolean True on success, False on failure
     */
    private function write_record($key, $data)
    {
        if (!$this->db) {
            return false;
        }

        // don't attempt to write too big data sets
        if (strlen($data) > $this->max_packet_size()) {
            trigger_error("rcube_cache: max_packet_size ($this->max_packet) exceeded for key $key. Tried to write " . strlen($data) . " bytes", E_USER_WARNING);
            return false;
        }

@@ -578,4 +585,30 @@

        return $this->packed ? @unserialize($data) : $data;
    }

    /**
     * Determine the maximum size for cache data to be written
     */
    private function max_packet_size()
    {
        if ($this->max_packet < 0) {
            $this->max_packet = 2097152; // default/max is 2 MB

            if ($this->type == 'db') {
                $value = $this->db->get_variable('max_allowed_packet', 1048500);
                $this->max_packet = min($value, $this->max_packet) - 2000;
            }
            else if ($this->type == 'memcache') {
                $stats = $this->db->getStats();
                $remaining = $stats['limit_maxbytes'] - $stats['bytes'];
                $this->max_packet = min($remaining / 5, $this->max_packet);
            }
            else if ($this->type == 'apc' && function_exists('apc_sma_info')) {
                $stats = apc_sma_info();
                $this->max_packet = min($stats['avail_mem'] / 5, $this->max_packet);
            }
        }

        return $this->max_packet;
    }
}