Патч безпеки SUPEE-10415 - можливі проблеми?


37

Випущений новий патч Magento 1, SUPEE-10415 .

Цей патч забезпечує захист від декількох проблем, пов’язаних із безпекою

Сторінка інформації: https://magento.com/security/patches/supee-10415
Сторінка завантаження: https://magento.com/tech-resources/download

На які можливі проблеми слід звернути увагу?

Також, будь ласка, поділіться всіма помилками та проблемами, які ви виявили після встановлення патчу.


  • Проблема із застосуванням SUPEE-10415 на ванілі 1.9.1.1 , шоу не може бути застосована через повідомлення про помилку на файлі Image.php . EDIT: Станом на 7 грудня 2017 року виправлення надано у програмі SUPEE-10497

  • Повинно встановити 8788 Версії 2 , інакше з’являться помилки "Непідтримуваний тип даних". Більше інформації.

  • Помилка "404: Сторінка не знайдена" після помилок / каталогу після оновлення до SUPEE-10415. Ця проблема виникає лише в установках Magento, які виконують певні сторонні розширення.
    Обхід: Підтвердьте, що немає попереджень PHP, створених будь-яким розширенням або налаштуваннями.

5
Тут не розгорнуто жодних проблем на Magento 1.9.2.4CE - більшість змін, мабуть, є санітарією змінних в адміністраторі, зокрема збереження файлів журналу. Здається, що в мильній формі є кілька змін, що стосуються також санітарних засобів. Відгуки також піддаються санітарній обробці в області адміністрування, тому може виникнути можливий подвиг, коли зловмисний код може бути розміщений в огляді на мо (спекулятивний)
Ricky Odin Matthews

2
Розгорнуто на Magento 1.9.3.0 CE тут проблем немає. Патч перед ним 10266 потрібен, якщо ви вже не застосували цей патч.
danmentzer

1
1.9.1.0 - у "Правилах цін на кошик для покупок" патч викликає збій - важка помилка у режимі перегляду - вхід до журналу -a: 5: {i: 0; s: 23: "Непідтримувані дані типу N"; i: 1; s: 1464: "# 0 /opt/bitnami/apps/magento/htdocs/lib/Unserialize/Reader/Arr.php(102): Unserialize_Reader_ArrValue-> read ('N', ';') ---- реверсія виправлень виправлена, так що це щось у SUPEE-10415
Laith

1
Я не маю репутації розміщувати відповідь, але ви не можете оновити до Magento 1.9.3.7, а потім використовувати Git для злиття з цим патчем: константа MAXIMUM_PASSWORD_LENGTH додається двічі, якщо ви використовуєте поведінку Git за замовчуванням (не маю уявлення, чи є налаштування, які є для цього обхідними).
toon81

1
Для питань 1.9.1.1: Будь ласка, використовуйте SUPEE-10497 замість цього, він був нещодавно випущений для вирішення цієї проблеми 1.9.1.1. Будь ласка, прочитайте примітки до випуску, оскільки цей виправлення вимагає видалення SUPEE-10266 перед встановленням.
Пьотр Камінський

Відповіді:


26

Наведені нижче файли оновлюються / додаються після застосованого патчу SUPEE - 10415.

app/Mage.php
app/code/core/Mage/Adminhtml/Block/Report/Review/Detail.php
app/code/core/Mage/Adminhtml/Block/Report/Tag/Product/Detail.php
app/code/core/Mage/Adminhtml/Block/Review/Add.php
app/code/core/Mage/Adminhtml/Block/Review/Edit/Form.php
app/code/core/Mage/Adminhtml/Controller/Action.php
app/code/core/Mage/Adminhtml/Model/LayoutUpdate/Validator.php
app/code/core/Mage/Adminhtml/Model/System/Config/Backend/Filename.php
app/code/core/Mage/Api/Helper/Data.php
app/code/core/Mage/Api/Model/Server/Adapter/Soap.php
app/code/core/Mage/Api/Model/Wsdl/Config.php
app/code/core/Mage/Api/Model/Wsdl/Config/Base.php
app/code/core/Mage/Core/Helper/String.php
app/code/core/Mage/Core/Model/File/Validator/Image.php
app/code/core/Mage/Core/etc/config.xml
app/code/core/Mage/Core/etc/system.xml
app/code/core/Mage/Customer/Model/Customer.php
app/code/core/Mage/Eav/Model/Entity/Attribute/Backend/Serialized.php
app/code/core/Mage/Log/Helper/Data.php
app/code/core/Mage/Rule/Model/Abstract.php
app/code/core/Mage/Sales/Block/Adminhtml/Billing/Agreement/Grid.php
app/code/core/Zend/Form/Decorator/Form.php
app/design/adminhtml/default/default/template/backup/dialogs.phtml
app/design/adminhtml/default/default/template/sales/billing/agreement/view/tab/info.phtml
app/design/adminhtml/default/default/template/xmlconnect/edit/tab/content.phtml
app/design/adminhtml/default/default/template/xmlconnect/edit/tab/design/image_edit.phtml
app/locale/en_US/Mage_Adminhtml.csv
app/locale/en_US/Mage_Customer.csv
js/mage/adminhtml/backup.js
lib/Varien/Filter/FormElementName.php

Деякі важливі моменти:

1) Дозволені розширення файлів: log, txt, html, csv. перевірте нижче файли

app/Mage.php
app/code/core/Mage/Adminhtml/Model/System/Config/Backend/Filename.php
app/code/core/Mage/Core/etc/system.xml
app/code/core/Mage/Log/Helper/Data.php

2) Максимальна встановлена ​​довжина пароля - 256 символів та перевірка перевірки у app/code/core/Mage/Customer/Model/Customer.phpфайлі

@@ -74,6 +74,11 @@ class Mage_Customer_Model_Customer extends Mage_Core_Model_Abstract
     const MINIMUM_PASSWORD_LENGTH = 6;

     /**
+     * Maximum Password Length
+     */
+    const MAXIMUM_PASSWORD_LENGTH = 256;
+
+    /**
      * Model event prefix
      *
      * @var string
@@ -876,6 +881,10 @@ class Mage_Customer_Model_Customer extends Mage_Core_Model_Abstract
             $errors[] = Mage::helper('customer')
                 ->__('The minimum password length is %s', self::MINIMUM_PASSWORD_LENGTH);
         }
+        if (strlen($password) && !Zend_Validate::is($password, 'StringLength', array('max' => self::MAXIMUM_PASSWORD_LENGTH))) {
+            $errors[] = Mage::helper('customer')
+                ->__('Please enter a password with at most %s characters.', self::MAXIMUM_PASSWORD_LENGTH);
+        }
         $confirmation = $this->getPasswordConfirmation();
         if ($password != $confirmation) {
             $errors[] = Mage::helper('customer')->__('Please make sure your passwords match.');
@@ -902,7 +911,7 @@ class Mage_Customer_Model_Customer extends Mage_Core_Model_Abstract
     }

     /**
-     * Validate customer attribute values on password reset
+     * Validate customer password on reset
      * @return bool
      */
     public function validateResetPassword()
@@ -916,6 +925,10 @@ class Mage_Customer_Model_Customer extends Mage_Core_Model_Abstract
             $errors[] = Mage::helper('customer')
                 ->__('The minimum password length is %s', self::MINIMUM_PASSWORD_LENGTH);
         }
+        if (!Zend_Validate::is($password, 'StringLength', array('max' => self::MAXIMUM_PASSWORD_LENGTH))) {
+            $errors[] = Mage::helper('customer')
+                ->__('Please enter a password with at most %s characters.', self::MAXIMUM_PASSWORD_LENGTH);
+        }
         $confirmation = $this->getPasswordConfirmation();
         if ($password != $confirmation) {
             $errors[] = Mage::helper('customer')->__('Please make sure your passwords match.');

Для видання EE додано додаткові чотири файли

app/code/community/OnTap/Merchandiser/Block/Adminhtml/Catalog/Product/List.php
app/design/adminhtml/default/default/template/merchandiser/smartmerch/tab.phtml
app/design/frontend/rwd/enterprise/template/giftcardaccount/onepage/payment/scripts.phtml
app/design/frontend/enterprise/default/template/giftcardaccount/onepage/payment/scripts.phtml

Деякі неактуальні моменти в ЕЕ

Додано умову у наведених нижче файлах

app/design/frontend/rwd/enterprise/template/giftcardaccount/onepage/payment/scripts.phtml 
  app/design/frontend/enterprise/default/template/giftcardaccount/onepage/payment/scripts.phtml

Будь ласка, оновіть наведену нижче умову у файлах тем.

if (elements[i].name == 'form_key') 
{
                continue;
 }

Для отримання додаткової інформації:

https://magento.com/security/patches/supee-10415 http://devdocs.magento.com/guides/m1x/ce19-ee114/ee1.14_release-notes.html#ee114-11436 http: // devdocs. magento.com/guides/m1x/ce19-ee114/ce1.9_release-notes.html#ce19-1936


Після застосування SUPEE-10415, як передній, так і зворотній, отримують помилку 404, як я можу вирішити цю проблему? magento.stackexchange.com/q/215620/57334
zus

Чи можете ви створити окремий квиток та пояснити журнал помилок та додати коментар, я допоможу вам. ця проблема через .htaccess або перевизначення модуля в локальному.
Рама Чандран M


20

SUPEE-10415 ...

  1. Вирішує деякі проблеми XSS панелі адміністратора в таких областях:

    • Звіт про відгуки про товар
    • Звіт про теги товарів
    • Інтерфейс додавання та редагування відгуків товарів
    • Білінгові угоди
    • вкладка редактора вмісту xmlconnect
    • Серіалізовані правила
  2. Обмежує розширення файлів, які можна використовувати для системних та журналів файлів винятку. Допустимі розширення файлів: .log, .txt, .html,.csv

  3. Розміщує верхню межу 256 символів для паролів облікового запису клієнта. Ця зміна є смішною; не впевнений, звідки вони взяли цю світлу ідею.

Жодна з цих змін не є масово порушуючими або зворотно несумісними змінами, за винятком можливо обмеження довжини пароля клієнта.


9
Довжина пароля 256 насправді є помилкою у непатронному Magento, оскільки поле пароля бази даних знаходиться у customer_entity_varchar.value, максимальна довжина = 255. Не впевнений, що відбувається без виправлення, якщо ви намагаєтесь встановити довше 255, ймовірно, це спричинить проблему.
Рікі Одін Метьюз

12
Ви маєте рацію; customer_entity_varchar.value має обмежений розмір стовпця. Однак це не помилка, оскільки пароль не зберігається в цьому стовпці. Солона і хешировать пароль зберігається там. Це значення завжди матиме фіксовану довжину, незалежно від розміру вихідного пароля. До цього виправлення не було б абсолютно жодних проблем із введенням паролю довше 255 символів.
t-richards

3
Я припускаю, що зміна довжини пароля має щось спільне з APPSEC-1330. magento.com/security/patches/supee-10415
квазіоб'єкт

7
Можливо , це їхнє мислення, Роб: stackoverflow.com/a/98857/8199523
RickyMage123

1
@ t-richards чудово, що ти там. Чому ви думаєте, що вони це зроблять? Я думаю, що, можливо, хеш-довгий пароль може спричинити проблему DoS з точки зору використання процесора. Звичайно, довший пароль потребує більше хешу CPU / пам'яті в PHP?
Рікі Одін Меттьюз

12

SUPEE 10415 Потрібен патч 8788 v2

Я зіткнувся з тією ж помилкою, що і в попередньому дописі, але, здається, її видалено.

a:5:{i:0;s:23:"Unsupported data type N";i:1;s:2942:"#0 /chroot/home/mywebroot/html/lib/Unserialize/Reader/Arr.php(102): Unserialize_Reader_ArrValue->  read('N', ';')
#1 /chroot/home/mywebroot/html/lib/Unserialize/Parser.php(53): Unserialize_Reader_Arr->read('N', ';')
#2 /chroot/home/mywebroot/html/app/code/core/Mage/Core/Helper/UnserializeArray.php(44): Unserialize_Parser->unserialize('a:6:{s:4:"type"...')
#3 /chroot/home/mywebroot/html/app/code/core/Mage/Rule/Model/Abstract.php(179): Mage_Core_Helper_UnserializeArray->unserialize('a:6:{s:4:"type"...')
#4 /chroot/home/mywebroot/html/app/code/core/Mage/Rule/Model/Abstract.php(353): Mage_Rule_Model_Abstract->getConditions()
#5 /chroot/home/mywebroot/html/app/code/core/Mage/SalesRule/Model/Validator.php(216): Mage_Rule_Model_Abstract->validate(Object(   Mage_Sales_Model_Quote_Address))
#6 /chroot/home/mywebroot/html/app/code/core/Mage/SalesRule/Model/Validator.php(242): Mage_SalesRule_Model_Validator->_canProcessRule(Object(  Mage_SalesRule_Model_Rule), Object(Mage_Sales_Model_Quote_Address))
#7 /chroot/home/mywebroot/html/app/code/core/Mage/SalesRule/Model/Quote/Freeshipping.php(74): Mage_SalesRule_Model_Validator->processFreeShipping(Object(  Mage_Sales_Model_Quote_Item))
#8 /chroot/home/mywebroot/html/app/code/core/Mage/Sales/Model/Quote/Address.php(1013): Mage_SalesRule_Model_Quote_Freeshipping->collect(Object(    Mage_Sales_Model_Quote_Address))
#9 /chroot/home/mywebroot/html/app/code/core/Mage/Sales/Model/Quote.php(1331): Mage_Sales_Model_Quote_Address->collectTotals()
#10 /chroot/home/mywebroot/html/app/code/core/Mage/Checkout/Model/Cart.php(458): Mage_Sales_Model_Quote->collectTotals()
#11 /chroot/home/mywebroot/html/app/code/core/Mage/Checkout/controllers/CartController.php(127): Mage_Checkout_Model_Cart->save()
#12 /chroot/home/mywebroot/html/app/code/core/Mage/Core/Controller/Varien/Action.php(418): Mage_Checkout_CartController->indexAction()
#13 /chroot/home/mywebroot/html/app/code/core/Mage/Core/Controller/Varien/Router/Standard.php(254): Mage_Core_Controller_Varien_Action->dispatch('index')
#14 /chroot/home/mywebroot/html/app/code/core/Mage/Core/Controller/Varien/Front.php(172): Mage_Core_Controller_Varien_Router_Standard->match(Object(   Mage_Core_Controller_Request_Http))
#15 /chroot/home/mywebroot/html/app/code/core/Mage/Core/Model/App.php(354): Mage_Core_Controller_Varien_Front->dispatch()
#16 /chroot/home/mywebroot/html/app/Mage.php(683): Mage_Core_Model_App->run(Array)
#17 /chroot/home/mywebroot/html/index.php(96): Mage::run('', 'store')
#18 {main}";s:3:"url";s:15:"/checkout/cart/";s:11:"script_name";s:10:"/index.php";s:4:"skin";s:7:"default";}

Я зіткнувся з вищевказаною помилкою, коли встановив новий патч на M 1.8.00 CE. Здається, що конкретна причина пов’язана з тим, що патч v2 8788 ми зробили вручну виправлення більшості питань із цього патча, але ми пропустили несеріалізовані частини.

Замість того, щоб повертати патч, я вручну вносив зміни, і тепер сайт працює належним чином.

Нижче ви знайдете код для патча 8788

diff --git lib/Unserialize/Parser.php lib/Unserialize/Parser.php
index 423902a..2c01684 100644
--- lib/Unserialize/Parser.php
+++ lib/Unserialize/Parser.php
@@ -34,6 +34,7 @@ class Unserialize_Parser
     const TYPE_DOUBLE = 'd';
     const TYPE_ARRAY = 'a';
     const TYPE_BOOL = 'b';
+    const TYPE_NULL = 'N';

     const SYMBOL_QUOTE = '"';
     const SYMBOL_SEMICOLON = ';';

diff --git lib/Unserialize/Reader/Arr.php lib/Unserialize/Reader/Arr.php
index caa979e..cd37804 100644
--- lib/Unserialize/Reader/Arr.php
+++ lib/Unserialize/Reader/Arr.php
@@ -101,7 +101,10 @@ class Unserialize_Reader_Arr
         if ($this->_status == self::READING_VALUE) {
             $value = $this->_reader->read($char, $prevChar);
             if (!is_null($value)) {
-                $this->_result[$this->_reader->key] = $value;
+                $this->_result[$this->_reader->key] =
+                    ($value == Unserialize_Reader_Null::NULL_VALUE && $prevChar == Unserialize_Parser::TYPE_NULL)
+                        ? null
+                        : $value;
                 if (count($this->_result) < $this->_length) {
                     $this->_reader = new Unserialize_Reader_ArrKey();
                     $this->_status = self::READING_KEY;

diff --git lib/Unserialize/Reader/ArrValue.php lib/Unserialize/Reader/ArrValue.php
index d2a4937..c6c0221 100644
--- lib/Unserialize/Reader/ArrValue.php
+++ lib/Unserialize/Reader/ArrValue.php
@@ -84,6 +84,10 @@ class Unserialize_Reader_ArrValue
                     $this->_reader = new Unserialize_Reader_Dbl();
                     $this->_status = self::READING_VALUE;
                     break;
+                case Unserialize_Parser::TYPE_NULL:
+                    $this->_reader = new Unserialize_Reader_Null();
+                    $this->_status = self::READING_VALUE;
+                    break;
                 default:
                     throw new Exception('Unsupported data type ' . $char);
             }

diff --git lib/Unserialize/Reader/Null.php lib/Unserialize/Reader/Null.php
new file mode 100644
index 0000000..93c7e0b
--- /dev/null
+++ lib/Unserialize/Reader/Null.php
@@ -0,0 +1,64 @@
+<?php
+/**
+ * Magento
+ *
+ * NOTICE OF LICENSE
+ *
+ * This source file is subject to the Open Software License (OSL 3.0)
+ * that is bundled with this package in the file LICENSE.txt.
+ * It is also available through the world-wide-web at this URL:
+ * http://opensource.org/licenses/osl-3.0.php
+ * If you did not receive a copy of the license and are unable to
+ * obtain it through the world-wide-web, please send an email
+ * to license@magento.com so we can send you a copy immediately.
+ *
+ * DISCLAIMER
+ *
+ * Do not edit or add to this file if you wish to upgrade Magento to newer
+ * versions in the future. If you wish to customize Magento for your
+ * needs please refer to http://www.magento.com for more information.
+ *
+ * @category    Unserialize
+ * @package     Unserialize_Reader_Null
+ * @copyright  Copyright (c) 2006-2016 X.commerce, Inc. and affiliates (http://www.magento.com)
+ * @license    http://opensource.org/licenses/osl-3.0.php  Open Software License (OSL 3.0)
+ */
+
+/**
+ * Class Unserialize_Reader_Null
+ */
+class Unserialize_Reader_Null
+{
+    /**
+     * @var int
+     */
+    protected $_status;
+
+    /**
+     * @var string
+     */
+    protected $_value;
+
+    const NULL_VALUE = 'null';
+
+    const READING_VALUE = 1;
+
+    /**
+     * @param string $char
+     * @param string $prevChar
+     * @return string|null
+     */
+    public function read($char, $prevChar)
+    {
+        if ($prevChar == Unserialize_Parser::SYMBOL_SEMICOLON) {
+            $this->_value = self::NULL_VALUE;
+            $this->_status = self::READING_VALUE;
+            return null;
+        }
+
+        if ($this->_status == self::READING_VALUE && $char == Unserialize_Parser::SYMBOL_SEMICOLON) {
+            return $this->_value;
+        }
+        return null;
+    }
+}

Ви також можете прочитати більше про помилку з цієї публікації РІШЕНО: новий клас Unserialize_Parser видає винятки за значеннями NULL


ага! Добре, 8788 V2 потрібен для запобігання проблеми!
Ікона

Чи можете ви скажіть, на яких сторінках вашого веб-сайту ви бачили помилки "Непідтримуваний тип даних"? Намагаються повторити.
Ікона

Мені вдалося запустити це на сторінці категорії та на сторінці продукту та кошику, поки продукт знаходився у кошику.
danmentzer

@danmentzer як я можу вирішити помилку, повернути патч 9767 v1 :: помилка: justpaste.it/1e9pn
zus

@zus Щойно я побачив ваш пост, чи ви зрозуміли це з вигляду вашої пасти, я б ризикну здогадатися про три різні можливі причини вашої помилки. 1. У вас неправильна версія патчу (це найменш вірогідна) 2. Ваш патч вже не застосовується. 3. Можливо, ви вручну виправили деякі з них, і патч не любить, як ви це зробили. Вибачте, якщо це не корисно
danmentzer

11

У нас виникла проблема з цим виправленням, коли кожна сторінка на сайті починала показувати помилку "404: Сторінка не знайдена" із errors/каталогу. Після невеликого копання виявляється, що це було викликано попередженням PHP Mage_Core_Model_App::init, яке видається Mage_Core_Model_Store_Exceptionу наступних рядках з патча:

diff --git app/Mage.php app/Mage.php
index 566027d..165928d 100644
--- app/Mage.php
+++ app/Mage.php
@@ -805,7 +805,12 @@ final class Mage
         static $loggers = array();

         $level  = is_null($level) ? Zend_Log::DEBUG : $level;
-        $file = empty($file) ? 'system.log' : $file;
+        $file = empty($file) ? 'system.log' : basename($file);
+
+        // Validate file extension before save. Allowed file extensions: log, txt, html, csv
+        if (!self::helper('log')->isLogFileExtensionValid($file)) {
+            return;
+        }

         try {
             if (!isset($loggers[$file])) {
  1. Попередження PHP відбувається перед ініціалізацією магазинів
  2. Попередження отримує повідомлення mageCoreErrorHandler(), яке викликає Mage::log()вхід повідомлення у файл журналу.
  3. Mage::log() дзвінки Mage::helper('log')
  4. Mage_Log_Helper_Data::__constructвиклики Mage::getStoreConfig(), які виклики Mage::app()->getStore(), але магазини не були ініційовані і Mage_Core_Model_Store_Exceptionкинутий
  5. app/Mage.php:647 ловить виняток і повертає сторінку 404

Не дуже впевнений, що таке рішення поки що, крім виправлення попередження та / або вилучення винятку під час перевірки розширення файлу журналу. Збираюся повідомити про це Magento, щоб побачити, що вони думають.


яку версію ви використовуєте?
Ікона

1
@Icon 1.9.3.6. Я помітив, що в деяких наших найсучасніших проектах не було застосовано __construct()методу Mage_Log_Helper_Data, тому це не вплине, але як останні версії спільноти, так і підприємства.
Томаш Герулайтіс

Схоже, magento вирішив
Ікона

Чи магенто згадував, чи планують виправити цю проблему наступним патчем?
Ікона

1
@Icon Ми помітили помилки в адміністраторі, а також у фронтенд, оскільки проблема сталася під час ініціалізації Magento.
Томаш Герулайтіс

8

1. Вирішено: Недійсна проблема секретного ключа, коли користувач завантажує адміністратора

У цих патчах Magento більше не відображає “Invalid Secret Key. Please refresh the page.” message when a user loads the Admin.

Змінивши код на

app/code/core/Mage/Adminhtml/Controller/Action.php

суворий весь пароль клієнта довжиною MAX 256:

Ми вже знаємо, що мінімальна довжина пароля magento 1.x - 6 .

але в цьому патчі magento має максимальну довжину до 256.

В цьому випадку, Magento зробив зміни на функції validate() з моделі Customer класу .so, якщо хто -то має перевизначити клас і перевизначити то вони повинні додати нижче код наthat override class

if (strlen($password) && !Zend_Validate::is($password, 'StringLength', array('max' => self::MAXIMUM_PASSWORD_LENGTH))) {
   $errors[] = Mage::helper('customer')
       ->__('Please enter a password with at most %s characters.', self::MAXIMUM_PASSWORD_LENGTH);
}

Додати $this->escapeHtml()та Mage :: helper ('core') -> quoteEscape () для деяких файлів, де можлива атака XSS

Якщо хтось замінить ці файли, вам слід додати код нижче, щоб повторити клас overrider 1.app/code/core/Mage/Adminhtml/Block/Report/Review/Detail.php

замінити

$ this -> _ headerText = Mage :: helper ('звіти') -> __ ('Відгуки про% s', $ product-> getName ());

з

$ this -> _ headerText = Mage :: helper ('звіти') -> __ ('Відгуки про% s', $ this-> escapeHtml ($ product-> getName ()));

2.app/code/core/Mage/Adminhtml/Block/Report/Tag/Product/Detail.php

замінити

$ this -> _ headerText = Mage :: helper ('звіти') -> __ ('Теги подано до% s', $ product-> getName ());

з

$this->_headerText = Mage::helper('reports')->__('Tags submitted to %s', $this->escapeHtml($product->getName()));

3.app/code/core/Mage/Adminhtml/Block/Review/Edit/Form.php

замінити

'text' => '<a href="' . $this->getUrl('*/catalog_product/edit', array('id' => $product->getId())) . '" onclick="this.target=\'blank\'">' . $product->getName() . '</a>'

з

'text' => '<a href="' . $this->getUrl('*/catalog_product/edit', array('id' => $product->getId())) . '" onclick="this.target=\'blank\'">' . $this->escapeHtml($product->getName()) . '</a>'

  1. Покупка перегляду замовлення на продаж погоджується: додаток / дизайн / adminhtml / за замовчуванням / за замовчуванням / шаблон / продаж / виставлення рахунків / договір / перегляд / вкладка / info.phtml

замінити

<?php echo $this->getCustomerEmail() ?>

з

<?php echo $this->escapeHtml($this->getCustomerEmail()) ?>

  1. app / design / adminhtml / default / default / template / xmlconnect / edit / tab / content.phtml від Mage :: helper ('core') -> quoteEscape

замінити

this.pageOptions += '<option value="<?php echo $helper->jsQuoteEscape($page['value']) ?>"><?php echo $helper->jsQuoteEscape($page['label']) ?></option>';

з

$this.pageOptions += '<option value="<?php echo $helper->jsQuoteEscape($page['value']) ?>"><?php echo $helper->quoteEscape($page['label']) ?></option>';

  1. app / design / adminhtml / default / default / template / xmlconnect / edit / tab / design / image_edit.phtml від Mage :: helper ('core') -> quoteEscape

замінити

<option value="<?php echo $page['value']; ?>"><?php echo $page['label']; ?></option>

з

<option value="<?php echo $page['value']; ?>"><?php echo Mage::helper('core')->quoteEscape($page['label']); ?></option>


4
Дякую @Amit я пропустив "Вирішено: Недійсна проблема секретного ключа" у моїх програмах :). +1
Рама Чандран M

1
ласкаво просимо:) ... ти добре зробив цю справу з цього питання
Аміт Бера

7

Якщо ви вже застосували SUPEE-10358 ^ або виправили вручну app/code/core/Mage/Adminhtml/Controller/Action.phpдля проблеми " Недійсний секретний ключ ", вам потрібно буде вручну видалити цей розділ з файлу патчу:

diff --git app/code/core/Mage/Adminhtml/Controller/Action.php app/code/core/Mage/Adminhtml/Controller/Action.php
index 2a8e63f..f2ee208 100644
--- app/code/core/Mage/Adminhtml/Controller/Action.php
+++ app/code/core/Mage/Adminhtml/Controller/Action.php
@@ -186,7 +186,7 @@ class Mage_Adminhtml_Controller_Action extends Mage_Core_Controller_Varien_Actio
                 'message' => $_keyErrorMsg
             )));
         } else {
-                if ($_keyErrorMsg != ''){
+                if (!$_isValidFormKey){
                 Mage::getSingleton('adminhtml/session')->addError($_keyErrorMsg);
             }
             $this->_redirect( Mage::getSingleton('admin/session')->getUser()->getStartupPageUrl() );

Крім того, якщо ви вже виправили помилку " new-pawwsord " (введена в SUPEE-10266), app/design/adminhtml/default/default/template/backup/dialogs.phtmlтоді також видаліть цей розділ із патча:

diff --git app/design/adminhtml/default/default/template/backup/dialogs.phtml app/design/adminhtml/default/default/template/backup/dialogs.phtml
index c5a3f82..2ff57cc 100644
--- app/design/adminhtml/default/default/template/backup/dialogs.phtml
+++ app/design/adminhtml/default/default/template/backup/dialogs.phtml
@@ -158,7 +158,7 @@
                             <td class="value">
                                 <!-- This is a dummy hidden field to trick firefox from auto filling the password -->
                                 <input type="password" class="input-text no-display" name="dummy" id="dummy" />

-                                    <input type="password" name="ftp_pass" id="ftp_pass" autocomplete="new-pawwsord">
+                                    <input type="password" name="ftp_pass" id="ftp_pass" autocomplete="new-password">
                             </td>
                         </tr>
                         <tr>

^ MageSupport надав SUPEE-10358 у відповідь на квиток на підтримку EE щодо питання InvalidSecretKey


5

Проблема: патч не працює на ванілі 1.9.1.1

Редагувати 1: виправлення додано нижче.

Редагувати 2: моє виправлення більше не потрібно, Magento надав SUPEE-10497 який вирішує цю проблему.

Проблема:

# file: PATCH_SUPEE-10415_CE_1.9.1.1_v1-2017-11-27-05-47-08.sh
Checking if patch can be applied/reverted successfully...
ERROR: Patch can't be applied/reverted successfully.

checking file app/Mage.php
checking file app/code/core/Mage/Adminhtml/Block/Report/Review/Detail.php
checking file app/code/core/Mage/Adminhtml/Block/Report/Tag/Product/Detail.php
checking file app/code/core/Mage/Adminhtml/Block/Review/Add.php
checking file app/code/core/Mage/Adminhtml/Block/Review/Edit/Form.php
checking file app/code/core/Mage/Adminhtml/Controller/Action.php
checking file app/code/core/Mage/Adminhtml/Model/LayoutUpdate/Validator.php
checking file app/code/core/Mage/Adminhtml/Model/System/Config/Backend/Filename.php
checking file app/code/core/Mage/Api/Helper/Data.php
checking file app/code/core/Mage/Api/Model/Server/Adapter/Soap.php
checking file app/code/core/Mage/Api/Model/Wsdl/Config.php
checking file app/code/core/Mage/Api/Model/Wsdl/Config/Base.php
checking file app/code/core/Mage/Core/Helper/String.php
checking file app/code/core/Mage/Core/Model/File/Validator/Image.php
Hunk #1 FAILED at 90.
1 out of 1 hunk FAILED
checking file app/code/core/Mage/Core/etc/config.xml
checking file app/code/core/Mage/Core/etc/system.xml
Hunk #1 succeeded at 651 (offset 1 line).
Hunk #2 succeeded at 661 (offset 1 line).
checking file app/code/core/Mage/Customer/Model/Customer.php
checking file app/code/core/Mage/Eav/Model/Entity/Attribute/Backend/Serialized.php
checking file app/code/core/Mage/Log/Helper/Data.php
checking file app/code/core/Mage/Rule/Model/Abstract.php
checking file app/code/core/Mage/Sales/Block/Adminhtml/Billing/Agreement/Grid.php
checking file app/code/core/Zend/Form/Decorator/Form.php
checking file app/design/adminhtml/default/default/template/backup/dialogs.phtml
checking file app/design/adminhtml/default/default/template/sales/billing/agreement/view/tab/info.phtml
checking file app/design/adminhtml/default/default/template/xmlconnect/edit/tab/content.phtml
checking file app/design/adminhtml/default/default/template/xmlconnect/edit/tab/design/image_edit.phtml
checking file app/locale/en_US/Mage_Adminhtml.csv
checking file app/locale/en_US/Mage_Customer.csv
checking file js/mage/adminhtml/backup.js
checking file lib/Varien/Filter/FormElementName.php

Ванільну Magento 1.9.1.1 було завантажено з https://github.com/OpenMage/magento-mirror/archive/1.9.1.1.tar.gz

Раніше застосовані виправлення на цьому Magento 1.9.1.1:

2017-11-29 07:37:12 UTC | SUPEE-5994 | CE_1.6.0.0 | v1 | _ | n/a | SUPEE-5994_CE_1.6.0.0_v1.patch
2017-11-29 07:37:12 UTC | SUPEE-6237 | EE_1.14.2.0 | v1 | 8b216c42e2e5d2cb5d8e500fcb6690abede9df52 | Fri Jun 12 13:39:59 2015 +0300 | v1.14.2.0..HEAD
2017-11-29 07:37:12 UTC | SUPEE-6285 | CE_1.9.1.1 | v2 | 7226d88b1eeb07a5fbc4e62be189a5219457cc14 | Mon Jun 22 16:32:26 2015 +0300 | 202596e441..7226d88b1e
2017-11-29 07:37:12 UTC | SUPEE-6482 | CE_1.9.2.0 | v1 |  | Tue Jul 14 14:17:04 2015 +0300 |
2017-11-29 07:37:12 UTC | SUPEE-6788 | CE_1.9.1.1 | v1 | 2349a68440e870cd68dfa81fb982f3b7a42cd099 | Fri Oct 23 14:49:16 2015 +0300 | b240663
2017-11-29 07:37:12 UTC | SUPEE-7405-CE-1-9-1-1 | CE_1.9.1.1 | v1 | f1c57f70de3fc2bea64bbe3ddf3bdf076b750f8e | Tue Jan 19 15:29:35 2016 +0200 | 2349a68440..f1c57f70de
2017-11-29 07:37:12 UTC | SUPEE-7405 | CE_1.9.1.1 | v1.1 | 38d673b4d2b132c6df53becc9d92346aa5d9627e | Fri Feb 5 13:28:39 2016 +0200 | f1c57f70de3fc2bea64bbe3ddf3bdf076b750f8e..38d673b4d2b132c6df53becc9d92346aa5d9627e
2017-11-29 07:37:12 UTC | SUPEE-7616 | CE_1.9.2.2-CE_1.8.0.0 | v1 | 1609c0d0be86473d357346fa51f93c12b365d7a1 | Tue Dec 8 12:53:31 2015 +0200 | e1fc3c59c9587427b8a9c88655715f27afbfe970..1609c0d0be86473d357346fa51f93c12b365d7a1
2017-11-29 07:37:12 UTC | SUPEE-8167 | EE_1.14.2.0 | v1 | 87bb97f9b0b2871f842b7faabf667a81806f937e | Thu Apr 27 13:31:21 2017 +0300 | 6010eb82..87bb97f9b
2017-11-29 07:37:12 UTC | SUPEE-8788 | CE_1.9.1.1 | v2 | 8d9fad1daf5131de3430ef09b0816d3d133c8412 | Mon Sep 26 14:06:18 2016 +0300 | 38d673b4d2..8d9fad1daf
2017-11-29 07:37:12 UTC | SUPEE-8967 | EE_1.13.1.0 | v1 | 1fa53e9533f6f3a16f24d9b64dabef0ab7f965d7 | Thu Aug 18 16:32:48 2016 +0300 | 97d160644..1fa53e9533
2017-11-29 07:37:13 UTC | SUPEE-9652 | EE_1.14.3.1 | v1 | 4038f0785d828794083f53f10c01aaa6af403523 | Tue Jan 24 15:03:12 2017 +0200 | 9586981e6ca8b255014b242d50b68b88525b0754..4038f0785d828794083f53f10c01aaa6af403523
2017-11-29 07:37:13 UTC | PATCH_SUPEE-9767_CE_1.9.3.0_v2 | CE_1.9.3.0 | v2 | 6566db274beaeb9bcdb56a62e02cc2da532e618c | Thu Jun 22 04:30:03 2017 +0300 | v1.14.3.3..HEAD
2017-11-29 07:37:13 UTC | SUPEE-10336_v1.14.2.4 | CE_1.9.2.4 | v1 | 721708ecf41f0ee745b8f441a4bfe56471b493a7 | Fri Sep 8 17:55:44 2017 +0300 | cc0d87..721708e
2017-11-29 07:37:13 UTC | SUPEE-10266-CE-1.9.1.1 | CE_1.9.1.1 | v1 | f66c6bcd3c16c5ca934823e6a91b5696698e497c | Fri Sep 1 12:15:53 2017 +0300 | 8d9fad1daf5131de3430ef09b0816d3d133c8412..HEAD

Офіційне виправлення патчу SUPEE-10415 для 1.9.1.1:

  • Якщо ви встановили SUPEE-10266:
    Відновіть його за допомогою: ./PATCH_SUPEE-10266_CE_1.9.1.1_v1-2017-09-15-04-59-56.sh --revert
  • Якщо ви встановили SUPEE-10415:
    відновіть його за допомогою ./PATCH_SUPEE-10415_CE_1.9.1.1_v1-2017-11-27-05-47-08.sh --revert
  • Тепер застосуйте SUPEE-10497

Виправлення вручну для патчу SUPEE-10415 для 1.9.1.1 [застаріло]:

Відредагуйте файл PATCH_SUPEE-10415_CE_1.9.1.1_v1-2017-11-27-05-47-08.sh, замініть рядки 445-447 .

Старий:

         $fileInfo = getimagesize($filePath);
         if (is_array($fileInfo) and isset($fileInfo[2])) {
             if ($this->isImageType($fileInfo[2])) {

Нове:

         list($imageWidth, $imageHeight, $fileType) = getimagesize($filePath);
         if ($fileType) {
             if ($this->isImageType($fileType)) {

Я теж переживаю це. Здається, це проблема app/code/core/Mage/Core/Model/File/Validator/Image.php. Patch SUPEE-10415, здається, не враховує модифікації, внесені Patch SUPEE-9767 (v1 або v2)
wr125

1
Виправлення додано внизу моєї відповіді.
Jeroen Vermeulen - MageHost

1
Я не згоден. Ми взяли ванільну установку з github.com/OpenMage/magento-mirror/archive/1.9.2.2.tar.gz і змогли застосувати виправлення в цьому порядку: SUPEE-7405-CE-1-9-2-2 | CE_1.9.2.2 | v1, SUPEE-7405 | CE_1.9.2.2 | v1.1, SUPEE-7616 | CE_1.9.2.2-CE_1.8.0.0 | v1, SUPEE-8167 | EE_1.14.2.0 | v1, SUPEE-8788 | CE_1.9.2.2 | v2, SUPEE-8967 | EE_1.13.1.0 | v1, SUPEE-9652 | EE_1.14.3.1 | v1, PATCH_SUPEE-9767_CE_1.9.3.0_v2 | CE_1.9.3.0 | v2, SUPEE-10336_v1.14.2.4 | CE_1.9.2.4 | v1, SUPEE-10266-CE-1.9.2.4 | CE_1.9.2.4 | v1, SUPEE-10415-ce-1.9.2.2 | CE_1.9.2.2 | v1
Jeroen Vermeulen - MageHost

1
Вам потрібно застосувати всі попередні виправлення, дивіться мій попередній коментар. Я тестував це.
Jeroen Vermeulen - MageHost

4
Будь ласка, використовуйте SUPEE-10497 замість цього, він був нещодавно випущений для вирішення цієї проблеми 1.9.1.1. Прочитайте примітки до випуску, оскільки цей виправлення вимагає видалення SUPEE-10266 перед встановленням.
Пьотр Камінський

3

Ось повний журнал змін і що я розумію з цим журналом змін

А які файли вплинули на причину, вказану нижче

Уникнути зміни Html

файл

app/code/core/Mage/Adminhtml/Block/Report/Review/Detail.php
app/code/core/Mage/Adminhtml/Block/Report/Tag/Product/Detail.php
app/code/core/Mage/Adminhtml/Block/Review/Add.php
app/code/core/Mage/Adminhtml/Block/Review/Edit/Form.php
app/code/core/Mage/Sales/Block/Adminhtml/Billing/Agreement/Grid.php
app/design/adminhtml/default/default/template/sales/billing/agreement/view/tab/info.phtml
app/design/adminhtml/default/default/template/xmlconnect/edit/tab/content.phtml
app/design/adminhtml/default/default/template/xmlconnect/edit/tab/design/image_edit.phtml
js/mage/adminhtml/backup.js

Додайте DS замість '/'

app/code/core/Mage/Adminhtml/Model/LayoutUpdate/Validator.php

додано новий файл

app/code/core/Zend/Form/Decorator/Form.php
lib/Varien/Filter/FormElementName.php

Повний файл змін

app / code / core / Mage / Adminhtml / Model / System / Config / Backend / Filename.php

 class Mage_Adminhtml_Model_System_Config_Backend_Filename extends Mage_Core_Model_Config_Data
 {
+
+ /**
+ * Config path for system log file.
+ */
+ const DEV_LOG_FILE_PATH = 'dev/log/file';
+
+ /**
+ * Config path for exception log file.
+ */
+ const DEV_LOG_EXCEPTION_FILE_PATH = 'dev/log/exception_file';
+
+ /**
+ * Processing object before save data
+ *
+ * @return Mage_Adminhtml_Model_System_Config_Backend_Filename
+ * @throws Mage_Core_Exception
+ */
 protected function _beforeSave()
 {
- $value = $this->getValue();
- $value = basename($value);
+ $value = $this->getValue();
+ $configPath = $this->getPath();
+ $value = basename($value);
+
+ // if dev/log setting, validate log file extension.
+ if ($configPath == self::DEV_LOG_FILE_PATH || $configPath == self::DEV_LOG_EXCEPTION_FILE_PATH) {
+ if (!Mage::helper('log')->isLogFileExtensionValid($value)) {
+ throw Mage::exception('Mage_Core', Mage::helper('adminhtml')->__
+ ('Invalid file extension used for log file. Allowed file extensions: log, txt, html, csv'));
+ }
+ }
+
     $this->setValue($value);
     return $this;
 }

Додано метод getCacheId () та getServiceUrl ()

app/code/core/Mage/Api/Helper/Data.php

Доданий метод несеріалізує ()

app/code/core/Mage/Core/Helper/String.php

Використовуйте метод getServiceUrl (), створений у додатку / код / ​​core / Mage / Api / Helper / Data.php

app/code/core/Mage/Api/Model/Server/Adapter/Soap.php
app/code/core/Mage/Api/Model/Wsdl/Config/Base.php

Використовуйте метод getCacheId (), створений у додатку / код / ​​core / Mage / Api / Helper / Data.php

app/code/core/Mage/Api/Model/Wsdl/Config.php

Використовуваний вище unserialize()метод, створений у додатку / код / ​​core / Mage / Core / Helper / String.php

app/code/core/Mage/Eav/Model/Entity/Attribute/Backend/Serialized.php
app/code/core/Mage/Rule/Model/Abstract.php

Змінити коментар

app/code/core/Mage/Core/Model/File/Validator/Image.php
app/code/core/Mage/Core/etc/system.xml

Коментар додано

app/code/core/Mage/Core/etc/config.xml

Додана максимальна довжина пароля

app/code/core/Mage/Customer/Model/Customer.php

Додано розширення файлу дозволу // $ _ allowedFileExtensions = array ('log', 'txt', 'html', 'csv');

app/code/core/Mage/Log/Helper/Data.php

Не знаю, що зміниться

app/design/adminhtml/default/default/template/backup/dialogs.phtml

Список питань

SUPEE-10415 перешкоджає контролю кошика

отримати код помилки # 10415 в Paypal в Магенто


1

Спроба цього на Magento EE 1.13.0.2, і, здається, цей патч не спрацьовує, якщо встановлено SUPEE-6482.

Checking patch app/code/core/Mage/Api/Model/Server/Adapter/Soap.php...
error: while searching for:
            ->setUseSession(false);

        $wsdlUrl = $params !== null
            ? $urlModel->getUrl('*/*/*', array('_current' => true, '_query' => $params))
            : $urlModel->getUrl('*/*/*');

        if( $withAuth ) {
            $phpAuthUser = $this->getController()->getRequest()->getServer('PHP_AUTH_USER', false);

error: patch failed: app/code/core/Mage/Api/Model/Server/Adapter/Soap.php:205

SUPEE-6482 змінив лінію $phpAuthUserна

$phpAuthUser = rawurlencode($this->getController()->getRequest()->getServer('PHP_AUTH_USER', false));

$phpAuthUserЛінія відрізнялася не лише тим, що інтервал між ними раніше ->setUseSession(false);був невірним
DanCarlyon
Використовуючи наш веб-сайт, ви визнаєте, що прочитали та зрозуміли наші Політику щодо файлів cookie та Політику конфіденційності.
Licensed under cc by-sa 3.0 with attribution required.