suhosin and @ini_set(‘memory_limit’, -1);

Hello there ??

thanks for such a great plugin I’ve been using for years.

I’m using suhosin extension and also fail2ban with a suhosin filter and jail.
so each time suhosin throws a warning it gets logged and after a few repeating warnings the client gets locked out via fail2ban

in your code you do a few @ini_set(‘memory_limit’, -1); and normally you do check first if the suhosin extension is set. see code package.module.attach_to_post.php

        if (!extension_loaded('suhosin')) {
            @ini_set('memory_limit', -1);
        }

which is really nice as suhosin will not log an error ??

But in package.module.nextgen_data.php there’s still a memory_limit without checking for suhosin extension. see line 4095

@ini_set('memory_limit', -1);

same in ngglegacy/lib/gd.thumbnail.inc.php on line 148

@ini_set('memory_limit', -1);

perhaps you could add the suhosin check also there?

            if (!extension_loaded('suhosin')) {
                @ini_set('memory_limit', -1);
            }

this would be awesome as I then wouldn’t need to manually update on each plugin update ??

Thanks again for such an awesome plugin!

Greetings
Becki