Fix so database_attachments::cleanup() does not remove attachments from oth...

Aleksander Machniak

2015-09-28 ba48318e2c87f4d07982978e539151c1abcf2114

Fix so database_attachments::cleanup() does not remove attachments from other sessions (#1490542)

Conflicts:
	CHANGELOG

 CHANGELOG

@@ -3,6 +3,7 @@

- Fix so Installer requires PHP5
- Make brute force attacks harder by re-generating security token on every failed login (#1490549)
- Fix so database_attachments::cleanup() does not remove attachments from other sessions (#1490542)

RELEASE 1.1.3
-------------

 plugins/database_attachments/database_attachments.php

@@ -22,7 +22,7 @@
    protected $cache;

    // A prefix for the cache key used in the session and in the key field of the cache table
    protected $prefix = "db_attach";
    const PREFIX = "ATTACH";

    /**
     * Save a newly uploaded attachment
@@ -153,9 +153,16 @@
            $ttl    = 12 * 60 * 60; // default: 12 hours
            $ttl    = $rcmail->config->get('database_attachments_cache_ttl', $ttl);
            $type   = $rcmail->config->get('database_attachments_cache', 'db');
            $prefix = self::PREFIX;

            // Add session identifier to the prefix to prevent from removing attachments
            // in other sessions of the same user (#1490542)
            if ($id = session_id()) {
                $prefix .= $id;
            }

            // Init SQL cache (disable cache data serialization)
            $this->cache = $rcmail->get_cache($this->prefix, $type, $ttl, false);
            $this->cache = $rcmail->get_cache($prefix, $type, $ttl, false);
        }

        return $this->cache;

 plugins/redundant_attachments/redundant_attachments.php

@@ -36,7 +36,7 @@
class redundant_attachments extends filesystem_attachments
{
    // A prefix for the cache key used in the session and in the key field of the cache table
    private $prefix = "ATTACH";
    const PREFIX = "ATTACH";

    // rcube_cache instance for SQL DB
    private $cache;
@@ -46,13 +46,6 @@

    private $loaded;

    /**
     * Default constructor
     */
    function init()
    {
        parent::init();
    }

    /**
     * Loads plugin configuration and initializes cache object(s)
@@ -68,15 +61,20 @@
        // load configuration
        $this->load_config();

        $ttl = 12 * 60 * 60; // 12 hours
        $ttl = $rcmail->config->get('redundant_attachments_cache_ttl', $ttl);
        $ttl    = 12 * 60 * 60; // 12 hours
        $ttl    = $rcmail->config->get('redundant_attachments_cache_ttl', $ttl);
        $prefix = self::PREFIX;

        if ($id = session_id()) {
            $prefix .= $id;
        }

        // Init SQL cache (disable cache data serialization)
        $this->cache = $rcmail->get_cache($this->prefix, 'db', $ttl, false);
        $this->cache = $rcmail->get_cache($prefix, 'db', $ttl, false);

        // Init memcache (fallback) cache
        if ($rcmail->config->get('redundant_attachments_memcache')) {
            $this->mem_cache = $rcmail->get_cache($this->prefix, 'memcache', $ttl, false);
            $this->mem_cache = $rcmail->get_cache($prefix, 'memcache', $ttl, false);
        }

        $this->loaded = true;