e107inc/e107

Consider simplifying this complex logical expression.

Open

                if($this->VerifyResultLines($connection,"220")>0
                && $this->PutLine($connection,"HELO $localhost")
                && $this->VerifyResultLines($connection,"250")>0
                && $this->PutLine($connection,"MAIL FROM: <$localuser@$localhost>")
                && $this->VerifyResultLines($connection,"250")>0

Suspicious type null of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

                    $this->OutputDebug("Resolving host name \"".$hosts[$host]."\"...");

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (str2) is ?string but \strcmp() takes string

Open

            && !strcmp(@gethostbyname($this->exclude_address),$ip))

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (subject) is null but \preg_match() takes string

Open

            if(preg_match('/^[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}\.[0-9]{1,3}$/',$domain))

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (hostname) is ?string but \fsockopen() takes string

Open

            if(($connection=($this->timeout ? @fsockopen($ip,25,$errno,$error,$this->timeout) : @fsockopen($ip,25))))

• Exclude checks
  • Disable engine
  • Disable check

Invalid offset 0 of array type array{}

Open

                $mxhosts[$weights[$host]]=$hosts[$host];

• Exclude checks
  • Disable engine
  • Disable check

Argument 1 (hostname) is null but \gethostbyname() takes string

Open

                if(!strcmp($ip=@gethostbyname($domain),$domain))

• Exclude checks
  • Disable engine
  • Disable check

Suspicious type null of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

                        $this->OutputDebug("Could not resolve host name \"".$hosts[$host]."\".");

• Exclude checks
  • Disable engine
  • Disable check

When appending to a value of type array{}, found an array access index of type null, but expected the index to be of type int|string

Open

                $mxhosts[$weights[$host]]=$hosts[$host];

• Exclude checks
  • Disable engine
  • Disable check

Suspicious type null of a variable or expression used to build a string. (Expected type to be able to cast to a string)

Open

                    $this->OutputDebug("Host address of \"".$hosts[$host]."\" is the exclude address");

• Exclude checks
  • Disable engine
  • Disable check

Argument 2 (str2) is null but \strcmp() takes string

Open

                if(!strcmp($ip=@gethostbyname($domain),$domain))

• Exclude checks
  • Disable engine
  • Disable check