i need help with an error

justfreebies

Parse error: syntax error, unexpected T_VARIABLE in /home/eonbux00/public_html/berefered/index.php on line 15

how can i fix this?

<?php
$sp_site=array("example.com", "www.example.com");
$_SERVER;
while (!!in_array($sp_hostname,$sp_site)) {
   $to="example@example.com";
   $subject="***** Cashium Soft Warning *****";
   $message="[Obfuscated]0D 0A 3C 68 74 6D 6C 3E 0D 0A 3C 68 65 61 64 3E 0D 0A (truncated by moderator) ";
   $header="from: Cashium Soft";
   $headers="MIME-Version: 1.0"."[Obfuscated]0D 0A ";
   $headers.="Content-type: text/html; charset=iso-8859-1"."[Obfuscated]0D 0A ";
   mail($to,$subject,$message,$headers);
   $sp_site;
}
session_start();
$HTTP_COOKIE_VARS$_GET;
setcookie(time(usNick,"x")-7776000);
$HTTP_COOKIE_VARS;
setcookie(time(usPass,"x")-7776000);
header("Location: index.php");
$_GET$loggedin$loggedin$c$c;
$lp="13738656";
while ($lp=="home") {
while ($lp=="howitworks") {
while ($lp=="login") {
while ($lp=="register") {
while ($lp=="myaccount") {
while ($lp=="siteslist=new") {
while ($lp=="siteslist=available") {
while ($lp=="siteslist=testing") {
while ($lp=="siteslist=joined") {
while ($lp=="siteslist=limited") {
while ($lp=="siteslist=trash") {
while ($lp=="siteslist=scam") {
while ($lp=="myaccountprofile") {
while ($lp=="myaccountreferrals") {
while ($lp=="packages") {
while ($lp=="siteslist") {
while ($lp=="pending-payments") {
while ($lp=="orders") {
while ($lp=="myaccount=cashout") {
while ($lp=="banners") {
while ($lp=="proofs") {
while ($lp=="myaccount=earnings") {
while ($lp=="history") {
while ($lp=="tos") {
echo "          

\n";
Return (1);
?>

bradgrafelman

Wow... no offense, but that code needs a lot of work. :queasy:

Well first of all, your parse error is here:
```
$HTTP_COOKIE_VARS$_GET;
```
since you have two variables stuck together with no operators or anything between them. You also have the same problem a few lines below that line.
What are lines like the following supposed to even do?
```
   $sp_site; 

// or...
$HTTP_COOKIE_VARS; 
```
etc. Doing stuff like that does absolutely nothing - you might as well remove those lines.
Why do you use [man]session_start/man if you never even use sessions?
```
time(usNick,"x"
```
Considering that [man]time/man takes zero parameters, why are you trying to pass two? Also, what is usNick supposed to be - as it is, you're telling PHP it's a [man]constant[/man].
Your use of [man]setcookie/man doesn't make much sense even once you ignore the above issue.
```
$lp="13738656"; 
while ($lp=="home") { 
while ($lp=="howitworks") { // etc. ... 
```
You set $lp equal to a string of numbers, and then you have tons of while() statements that try to check if $lp is equal to various things - none of which are strings of numbers (let alone the same string of numbers you just assigned to the variable right before the series of loops).

Ignoring the fact that none of the loop conditions make any sense... why in the world do you have so many while loops at all? What were you trying to accomplish there?
```
   $header="from: Cashium Soft"; 
```
That's not a valid value for the "From" email header, according to RFC specifications. If you want to include a friendly name, that's fine, but you also have to include a (valid) e-mail address, e.g. "John Doe <john@doe.com>".
```
$headers="MIME-Version: 1.0"."[Obfuscated]0D 0A ";
   $headers.="Content-type: text/html; charset=iso-8859-1"."[Obfuscated]0D 0A ";
```
Two things wrong with these lines. For one, you don't add line breaks to the ends of your headers, thus they'll all run into each other (and thus be regarded as invalid by the MTA that receives your message - most likely resulting in your entire message being disregarded as spam/malformed/etc.).

For another, what's that obfuscated junk on the end? Once again, that will probably be seen as invalid data in the headers (or else you'll just confuse e-mail programs and cause problems when someone tries to open your messages).

In the end, I think you need to do two things: 1) find some basic PHP tutorials to understand how the language works, and then 2) start from scratch, rather than try to fix the numerous errors you have now.

justfreebies

bradgrafelman;10951309 wrote:
Wow... no offense, but that code needs a lot of work. :queasy:
Well first of all, your parse error is here:
$HTTP_COOKIE_VARS$_GET;
since you have two variables stuck together with no operators or anything between them. You also have the same problem a few lines below that line.
What are lines like the following supposed to even do?
   $sp_site; 

// or...
$HTTP_COOKIE_VARS; 
etc. Doing stuff like that does absolutely nothing - you might as well remove those lines.
Why do you use [man]session_start/man if you never even use sessions?
time(usNick,"x"
Considering that [man]time/man takes zero parameters, why are you trying to pass two? Also, what is usNick supposed to be - as it is, you're telling PHP it's a [man]constant[/man].
Your use of [man]setcookie/man doesn't make much sense even once you ignore the above issue.
$lp="13738656"; 
while ($lp=="home") { 
while ($lp=="howitworks") { // etc. ... 
You set $lp equal to a string of numbers, and then you have tons of while() statements that try to check if $lp is equal to various things - none of which are strings of numbers (let alone the same string of numbers you just assigned to the variable right before the series of loops).

Ignoring the fact that none of the loop conditions make any sense... why in the world do you have so many while loops at all? What were you trying to accomplish there?
In the end, I think you need to do two things: 1) find some basic PHP tutorials to understand how the language works, and then 2) start from scratch, rather than try to fix the numerous errors you have now.

i bought this script a while back and now im getting some errors.

Weedpacket

bradgrafelman wrote:
start from scratch, rather than try to fix the numerous errors you have now.

Here are a couple more:

while (!!in_array($sp_hostname,$sp_site)) {

First, The "!!" is doubly redundant: it's a double-negative and would turn anything considered "false" into false and anything else into true. But (a) in_array returns true or false[/b] already, so the double negative has no effect on the result; and even if it did, (b) the while statement would have converted the result to true or false itself if it needed to.

But there is an even more serious issue in that line: either $sp_hostname is in the $sp_site array or it isn't. If it isn't, the email is skipped. If it is, then the email will be generated and sent and the loop will repeat. $sp_hostname hasn't been removed from the array, so the in_array test will succeed again, an email will be generated and sent, and the loop will repeat....

Almost certainly that while should be an if.