WPIntell

Source evidence

Malware warnings on php code for GDPR framework

The GDPR Framework By Data443 · support · 2022-02-05T14:03:00+00:00

mixedsentiment
highseverity
0.95relevance
6replies
Evidence linked to opportunitycommercial context

Proof Health

Open evidence

Commercial opportunities need traceable source links before they are treated as build-worthy.

4 / 34 rows with source links

11.8% of this page's analysis has direct source links.

0 build-decision rows missing links

0 rows here require auditable proof before promotion.

30 rows with no attached evidence

0 rows have source counts but still need direct links.

Conversation

support
creativeloves resolved
Hi I am using GDPR framework by Data 443 and my malware scanner keeps flagging these lines of code as dangerous and probable malware. Can anyone give insight into this? 1) protected function passthru($method, array $params) 2) exec(“mklink /{$mode} “.escapeshellarg($link).’ ‘.escapeshellarg($target)); 3) assert(class_exists(‘ParagonIE_Sodium_Compat’), ‘Possible filesystem/autoloader bug?’); And assert(class_exists(‘ParagonIE_Sodium_Compat’)) These are three separate ” flags” on the malware scanner. I use Cleantalk for antispam and malware scanning, and these lines of code are routinely triggered as dangerous with a “99.5%” likelihood of being malware. So, I’m wondering what’s up with that? Thanks! Hello, Thanks for bringing this to our attention. This is the first time we’ve heard of this issue. We’ll contact Cleantalk to see about resolving this false notification. Hello @data443 @creativeloves I’m a developer from CleanTalk team. I wanna share my thoughts about the issue: 1. protected function passthru($method, array $params) . Looks like it’s a false positive. Your passthru() is a method, not a native PHP function . We’ll tweak the logic. Thank you! 2. exec(“mklink /{$mode} “.escapeshellarg($link).’ ‘.escapeshellarg($target)); – is not safe to use this, because if you made a mistake in sanitizing any of these 3 parameters, it will become a security vulnerability. You could use safe alternative for this like: link() and symlink() . 3. assert(class_exists(‘ParagonIE_Sodium_Compat’)) – as the documentation says, the assertions should not be used for normal runtime operations like input parameter checks. Only for debugging. This reply was modified 4 years, 3 months ago by Safronik . Reason: spelling Hi @creativeloves @safronik , We have scrubbed the plugin and would like to provide to you to test before we release publicly. Can you provide your email or send us an email at support@data443.com so we can connect? Thank you! Thank you both @data443 and @safronik for addressing this issue! I am still getting flags every day. I have been sent a new version of the GDPR framework plugin to test and that is my next step, but before I go download that and apply it I have a quick question for Data443. Has this new version been scrubbed of said security risks, such as resolving this? “2. exec(“mklink /{$mode} “.escapeshellarg($link).’ ‘.escapeshellarg($target)); – is not safe to use this, because if you made a mistake in sanitizing any of these 3 parameters, it will become a security vulnerability. You could use safe alternative for this like: link() and symlink().” The new version we sent you has been scrubbed of the security risks you encountered. After reviewing the original problem we’ve traced the vulnerable code to a set of third-party scripts tied to an early commit. We have removed said third-party scripts in version 2.0.0 in order to remediate the vulnerability, and have already taken steps to ensure future contributions are properly vetted. This reply was modified 4 years, 3 months ago by Data443 Risk Mitigation, Inc. . The latest version has been released.

Comments

6 shown
Data443 Risk Mitigation, Inc. 2022-02-07T13:34:00+00:00

Hello, Thanks for bringing this to our attention. This is the first time we’ve heard of this issue. We’ll contact Cleantalk to see about resolving this false notification.

Safronik 2022-02-11T11:34:00+00:00

Hello @data443 @creativeloves I’m a developer from CleanTalk team. I wanna share my thoughts about the issue: 1. protected function passthru($method, array $params) . Looks like it’s a false positive. Your passthru() is a method, not a native PHP function . We’ll tweak the logic. Thank you! 2. exec(“mklink /{$mode} “.escapeshellarg($link).’ ‘.escapeshellarg($target)); – is not safe to use this, because if you made a mistake in sanitizing any of these 3 parameters, it will become a security vulnerability. You could use safe alternative for this like: link() and symlink() . 3. assert(class_exists(‘ParagonIE_Sodium_Compat’)) – as the documentation says, the assertions should not be used for normal runtime operations like input parameter checks. Only for debugging. This reply was modified 4 years, 3 months ago by Safronik . Reason: spelling

Data443 Risk Mitigation, Inc. 2022-02-16T15:43:00+00:00

Hi @creativeloves @safronik , We have scrubbed the plugin and would like to provide to you to test before we release publicly. Can you provide your email or send us an email at support@data443.com so we can connect? Thank you!

creativeloves 2022-02-17T16:49:00+00:00

Thank you both @data443 and @safronik for addressing this issue! I am still getting flags every day. I have been sent a new version of the GDPR framework plugin to test and that is my next step, but before I go download that and apply it I have a quick question for Data443. Has this new version been scrubbed of said security risks, such as resolving this? “2. exec(“mklink /{$mode} “.escapeshellarg($link).’ ‘.escapeshellarg($target)); – is not safe to use this, because if you made a mistake in sanitizing any of these 3 parameters, it will become a security vulnerability. You could use safe alternative for this like: link() and symlink().”

Data443 Risk Mitigation, Inc. 2022-02-28T13:37:00+00:00

The new version we sent you has been scrubbed of the security risks you encountered. After reviewing the original problem we’ve traced the vulnerable code to a set of third-party scripts tied to an early commit. We have removed said third-party scripts in version 2.0.0 in order to remediate the vulnerability, and have already taken steps to ensure future contributions are properly vetted. This reply was modified 4 years, 3 months ago by Data443 Risk Mitigation, Inc. .

Data443 Risk Mitigation, Inc. 2022-03-03T18:15:00+00:00

The latest version has been released.