help please [function.eregi]: REG_EMPTY

yenz

need your help guys im getting an error on my page that says

Warning: eregi() [function.eregi]: REG_EMPTY in /home/orbital/public_html/includes/config.class.php on line 107

this error shows 2 times on the webpage im working on trying to get a ddl script up on my server

i have the error line 107 this is the value on line 107

(eregi(substr($fil[$i], 0, strlen($fil[$i])-3), $url)) {

bellow is the full source code

var $mysql_tb_dl = "downloads";        // The downloads table
  var $mysql_tb_que = "que";          // The table for downloads submitted but not reviewed
  var $mysql_tb_le = "linker";        // The links information
  var $mysql_tb_ip = "iplogg";        // Ip-logging, for anti-cheating (and for you to check if you suspect cheating)
  var $mysql_tb_sdl = "site_downloads";        // The Sitedownloads table
  var $connect = false;

function option_list() {
    for ($i=0; $i<count($this->type); $i++) {
      $return .= "<select>
  <option>".$this->type[$i]."</option>";
    }
    return $return;
}

function open() {
    $this->connect = @mysql_connect($this->mysql_host, $this->mysql_user, $this->mysql_pass)
      or die("This is the new share server! It is still being installed! For share on the old server, Go to: http:[url]www.test.com[/url] <a href=\"mailto:".$this->admin_email."\">".$this->admin_email."</a>");
    @mysql_select_db($this->mysql_db)
      or die("Failed to connect to databases, contact <a href=\"mailto:".$this->admin_email."\">".$this->admin_email."</a>");
}

function close() {
    @mysql_close($this->connect);
}

function dato() {
    $dato = date($this->date_format);
    return $dato;
}

function run($list) {
    $check_first = 1;
    for ($i=0; $i<count($list); $i++) {
      if ($check_first) {
        $hvahvor = "id = '".$list[$i]."'";
        $check_first = 0;
      } else
        $hvahvor .= " || id = '".$list[$i]."'";
    }
    return $hvahvor;
}

function page($url) {
    global $page;
    $pages=1;
    for ($starter=0; $this->total>$starter; $starter+=$this->limit) {
      if ($pages != $page)
        echo "<a href=\"".$url.$pages."\" class=\"page\">$pages</a> \n";
      else
        echo "<b>[ $pages ]</b> \n";
    $pages++;
    }
}

function blacklist($url) {
    $fil = @file("blacklist.txt");
    $go = false;
    for ($i=0; $i<count($fil); $i++) {
      if (eregi(substr($fil[$i], 0, strlen($fil[$i])-3), $url)) {
        $go = true;
        break;
      }
    }
    if ($go)
      return true;
    else
      return false;
  }
}
?>

bpat1434

remove the "@" character before the [man]file/man function call a few lines above 107 (line 104 I believe) and see what error spits out at you then.

yenz

doing that just made it worse now it says

Warning: file(blacklist.txt) [function.file]: failed to open stream: No such file or directory in /home/orbital/public_html/includes/config.class.php on line 104

Warning: eregi() [function.eregi]: REG_EMPTY in /home/orbital/public_html/includes/config.class.php on line 107

Warning: file(blacklist.txt) [function.file]: failed to open stream: No such file or directory in /home/orbital/public_html/includes/config.class.php on line 104

Warning: eregi() [function.eregi]: REG_EMPTY in /home/orbital/public_html/includes/config.class.php on line 107

and i have the blacklist.txt file in the directory

NogDog

It didn't make things worse. Rather, it allowed you to see what the problem is. It's telling you that it cannot open the file "blacklist.txt" because it cannot find it. Is the file in the same directory as the script? (If not, you'll need to provide a full or relative path to it.) Have you spelled it correctly, including upper-/lower-case letters? Does the file have the necessary permissions to be readable by the webserver user under whom PHP scripts are executed?

PS: This is why it's always a good idea to "practice safe coding," and never assuming things will work:

$blacklist = "blacklist.txt";
$fil = @file($blacklist);
if($fil === FALSE)
{
   user_error("Unable to open blacklist file for reading.", E_USER_WARNING);
}
else
{
   // code that uses data from blacklist...
}

bpat1434

doing that just made it worse

Sometimes the only way to move forward, is to go backwards....

When something doesn't work, you need to allow the errors to show, not suppress them. By removing the "@" character we aren't suppressing errors when the file "blacklist" is read (or tried to be read). Instead, now we see the cause of the problems. While it looks like it's getting worse, we're really just getting back on track 😉