Project

General

Profile

Actions
Copy link

Bug #38135

closed

Apc Cache backend has side effects

Added by Daniel Poetzinger over 12 years ago. Updated about 6 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2012-06-17
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
6.0
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

If two caches use the APC cache backend, one cache can write and get entries from the other by just using the same key!

Patch:

--- a/t3lib/cache/backend/class.t3lib_cache_backend_apcbackend.php
+++ b/t3lib/cache/backend/class.t3lib_cache_backend_apcbackend.php
@@ -87,7 +87,7 @@ class t3lib_cache_backend_ApcBackend extends t3lib_cache_backend_AbstractBackend
        public function setCache(t3lib_cache_frontend_Frontend $cache) {
                parent::setCache($cache);
                $processUser = extension_loaded('posix') ? posix_getpwuid(posix_geteuid()) : array('name' => 'default');
-               $pathHash = t3lib_div::shortMD5(PATH_site . $processUser['name'] . $this->context, 12);
+               $pathHash = t3lib_div::shortMD5(PATH_site . $processUser['name'] . $this->context . $cache->getIdentifier(), 12);
                $this->identifierPrefix = 'TYPO3_' . $pathHash;
        }

Actions
Copy link
 #1

Updated by Gerrit Code Review over 12 years ago

  • Status changed from New to Under Review

Patch set 1 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #2

Updated by Gerrit Code Review over 12 years ago

Patch set 2 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #3

Updated by Gerrit Code Review over 12 years ago

Patch set 3 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #4

Updated by Gerrit Code Review over 12 years ago

Patch set 4 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #5

Updated by Gerrit Code Review over 12 years ago

Patch set 5 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #6

Updated by Gerrit Code Review over 12 years ago

Patch set 6 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #7

Updated by Gerrit Code Review almost 12 years ago

Patch set 7 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #8

Updated by Gerrit Code Review almost 12 years ago

Patch set 8 for branch master has been pushed to the review server.
It is available at http://review.typo3.org/12145

Actions
Copy link
 #9

Updated by Ernesto Baschny almost 12 years ago

Please note that this class is a direct backport from the TYPO3 Flow Cache class. So consider reporting the issue there also and pointing here, so that a potentially approved fix can be forward ported.

Actions
Copy link
 #10

Updated by Tymoteusz Motylewski almost 12 years ago

I've added an issue on flow bug tracker
http://forge.typo3.org/issues/44471

Actions
Copy link
 #11

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_6-0 has been pushed to the review server.
It is available at https://review.typo3.org/17470

Actions
Copy link
 #12

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-7 has been pushed to the review server.
It is available at https://review.typo3.org/17471

Actions
Copy link
 #13

Updated by Gerrit Code Review almost 12 years ago

Patch set 1 for branch TYPO3_4-6 has been pushed to the review server.
It is available at https://review.typo3.org/17472

Actions
Copy link
 #14

Updated by Daniel Poetzinger almost 12 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 90 to 100
Actions
Copy link
 #15

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Resolved to Under Review

Patch set 1 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/17473

Actions
Copy link
 #16

Updated by Gerrit Code Review almost 12 years ago

Patch set 2 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/17473

Actions
Copy link
 #17

Updated by Gerrit Code Review almost 12 years ago

Patch set 3 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/17473

Actions
Copy link
 #18

Updated by Daniel Poetzinger almost 12 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #19

Updated by Gerrit Code Review almost 12 years ago

  • Status changed from Resolved to Under Review

Patch set 4 for branch TYPO3_4-5 has been pushed to the review server.
It is available at https://review.typo3.org/17473

Actions
Copy link
 #20

Updated by Daniel Poetzinger almost 12 years ago

  • Status changed from Under Review to Resolved
Actions
Copy link
 #21

Updated by Andreas Kienast almost 7 years ago

  • Related to Bug #83470: Translations can be dragged to default column (in page module -> language) added
Actions
Copy link
 #22

Updated by Andreas Kienast almost 7 years ago

  • Related to deleted (Bug #83470: Translations can be dragged to default column (in page module -> language))
Actions
Copy link
 #23

Updated by Benni Mack about 6 years ago

  • Status changed from Resolved to Closed
Actions
Copy link

Also available in: Atom PDF