1.9.2.0中新的Varien_Image_Adapter_Gd2析构函数的ImportExport问题

有人可以解释一下，Magento CE 1.9.1.0和1.9.2.0之间引入的以下代码有什么用吗？

class Varien_Image_Adapter_Gd2:

public function __construct()
{
    // Initialize shutdown function
    register_shutdown_function(array($this, 'destruct'));
}

/**
 * Destroy object image on shutdown
 */
public function destruct()
{
    @imagedestroy($this->_imageHandler);
}

添加这两个功能后，我们使用ImportExport接口导入产品库图像的操作停止。该错误是由于内存限制（结果是最大打开文件大小限制）引起的。

我的想法是，导入将无法正确关闭打开的文件。

我还看到destruct()引入了一些空函数（Mage_ImportExport_Model_Import_Adapter_Abstract）-但是扩展这些函数以匹配父级逻辑无济于事。

看起来他们试图确保销毁图像资源，但却引入了内存泄漏。老实说，我想不出这段代码的正当理由，但是我可以解释一下已更改的内容：

本来imagedestroy()应该在解扰器中调用__destruct()

function __destruct()
{
    @imagedestroy($this->_imageHandler);
}

每当PHP垃圾回收器销毁未使用的对象（即，内存中不再引用的对象）时，就会调用析构函数。

现在，imagedestroy()它在关闭函数中被调用，并且由于这是对Varien_Image_Adapter_Gd2对象方法的回调，因此直到最后都无法进行垃圾回收。这样，所有图像资源将保持打开状态，直到脚本执行完成。

我的Magento 1.9.2.0遇到相同的问题...

我只能通过改变得到这个工作Varien_Image_Adapter_Gd2中/lib/Varien/Image/Adapter/Gd2.php，如下所示：

public function __construct()
{
    // Initialize shutdown function
    // register_shutdown_function(array($this, 'destruct'));
}

/**
 * Destroy object image on shutdown
 */
public function __destruct()
{
    @imagedestroy($this->_imageHandler);
}

• 用register_shutdown_function删除行（或注释掉）
• 将函数名称destruct更改为__destruct

我已将memory_limit设置回1G（以前我提高到了32GB），现在它可以工作了...

该项目以modman友好的方式实现了上述程序。只需在composer上安装它，就可以了。

这是通过反序列化解决安全问题的一部分。__destruct之类的魔术方法在序列化方面存在固有的问题。

我们已经看到有人提出利用漏洞利用序列化和__destruct在文件系统中创建文件的漏洞-进行此更改（在其他地方会看到更多类似的更改）是为了避免这种情况。

它是否会导致内存泄漏或只是在脚本完成之前使用更多的内存？

/security/77549/is-php-unserialize-exploitable-without-any-interesting-methods

因此，我确实在Magento上提出了一个错误，其中包括一个“解决方案”，该解决方案应处理图像导入过程中的内存使用问题。

该解决方案可以在https://github.com/sitewards/import_image_memory_leak_fix的 github上找到但基本思想是。

修复Mage_Catalog_Helper_Image::validateUploadFile实际destruct在图像处理器上调用该方法的问题。可悲的是，默认值Varien_Image似乎不处理，destruct因此我们不得不添加自己的类。

<?php
/**
 * @category    Sitewards
 * @package     Sitewards_ImportImageMemoryLeakFix
 * @copyright   Copyright (c) Sitewards GmbH (http://www.sitewards.com/)
 */
class Sitewards_ImportImageMemoryLeakFix_Model_Destructable_Image extends Varien_Image
{
    /**
     * Constructor,
     * difference from original constructor - we register a destructor here.
     *
     * @param string $sFileName
     * @param Varien_Image_Adapter $oAdapter Default value is GD2
     */
    public function __construct($sFileName = null, $oAdapter = Varien_Image_Adapter::ADAPTER_GD2)
    {
        parent::__construct($sFileName, $oAdapter);

        // Initialize shutdown function
        register_shutdown_function(array($this, 'destruct'));
    }

    /**
     * Destroy object image on shutdown
     */
    public function destruct()
    {
        $oAdapter = $this->_getAdapter();
        if (method_exists($oAdapter, 'destruct')) {
            $oAdapter->destruct();
        } else {
            Mage::log('Image can not be destructed properly, adapter doesn\'t support the method.');
        }
    }
}

然后重写助手。

<?xml version="1.0"?>
<config>
    <modules>
        <Sitewards_ImportImageMemoryLeakFix>
            <version>0.1.0</version>
        </Sitewards_ImportImageMemoryLeakFix>
    </modules>
    <global>
        <models>
            <sitewards_importimagememoryleakfix>
                <class>Sitewards_ImportImageMemoryLeakFix_Model</class>
            </sitewards_importimagememoryleakfix>
        </models>
        <helpers>
            <catalog>
                <rewrite>
                    <image>Sitewards_ImportImageMemoryLeakFix_Helper_Catalog_Helper_Image</image>
                </rewrite>
            </catalog>
        </helpers>
    </global>
</config>

新函数将调用新的可销毁图像类。

<?php
/**
 * @category    Sitewards
 * @package     Sitewards_ImportImageMemoryLeakFix
 * @copyright   Copyright (c) Sitewards GmbH (http://www.sitewards.com/)
 */
class Sitewards_ImportImageMemoryLeakFix_Helper_Catalog_Helper_Image extends Mage_Catalog_Helper_Image
{
    /**
     * Check - is this file an image
     *
     * Difference from original method - we destroy the image object here,
     * i.e. we are not wasting memory, without that fix product import with images
     * easily goes over 4Gb on memory with just couple hundreds of products.
     *
     * @param string $sFilePath
     *
     * @return bool
     * @throws Mage_Core_Exception
     */
    public function validateUploadFile($sFilePath) {
        if (!getimagesize($sFilePath)) {
            Mage::throwException($this->__('Disallowed file type.'));
        }

        /** @var Sitewards_ImportImageMemoryLeakFix_Model_Destructable_Image $oImageProcessor */
        $oImageProcessor = Mage::getModel('sitewards_importimagememoryleakfix/destructable_image', $sFilePath);
        $sMimeType       = $oImageProcessor->getMimeType();
        $oImageProcessor->destruct();

        return $sMimeType !== null;
    }
}