Hvordan tilgår jeg en variable i min funktion?

Du skal lige gemme din returnerede værdi fra din funktion. Som det er nu, så returnerer du message, fra login funktionen, men den bliver ikke 'fanget' nogle steder. Variabler deklareret og brugt inde i en funktion er lokale for den enkelte funktion, og du kan derfor ikke bare tilgå den uden at gemme den returnerede værdi.

Et eksempel på hvordan du skal løse det

 if (isset($_POST['login']))
 {
    //Brug funktion og gem evt. returnerede værdier
    $message_from_login = login($_POST['username'], $_POST['password']);
 
    //Tjek om der overhovedet er kommet en returneret værdi
    if (!empty($message_from_login))
    {
       echo $message_from_login;
    }
 }

Kode 

$username = mysql_real_escape_string(addslashes($username));

Det her er temmelig meget overkill. Addslashes gør (ca.) det samme som mysql_real_escape_string, så du dobbeltenkoder altså din streng. Nøjes med mysql_real_escape_string.

Kode 

$password = mysql_real_escape_string(addslashes($password));
$password = hash('sha512', $password);

Det her er også forkert. Man escaper strengen for at undgå SQL injection, men da du efterfølgende hasher så er det helt overflødigt (og faktisk også forkert). Nøjes med at hashe dit password.

Endnu en ting i din nye implementering. Du tjekker om $username og $password er false efter at du har behandlet dem.

Gør det som det første, da i hvert fald hash() returnerer et gyldigt hash selvom du giver en tom streng med...medmindre $username er null returnerer mysql_real_escape_string også noget gyldigt.

Jeg takker og bukker drenge.

Begge dele er rettet og virker nu.

Til dem der skulle have interesse så endte det ud med dette resultat. OBS, der anvendes en anden funktion i login funktionen, den er ikke medtaget.

 function login($username, $password)
 {
  $username = mysql_real_escape_string($username); 
  $password = hash('sha512', $password);    
  
  if(!$username||!$password) 
  {
      $msg = "Udfyld venligst begge felter.";
     return $msg;
  } else 
     {
         $login = mysql_query("SELECT * FROM user WHERE username = '$username' AND password = '$password'");
        $login_count = mysql_num_rows($login);
        if($login_count == 1)
        {
            $row = mysql_fetch_assoc($login);
             $_SESSION['user_id'] = $row['id'];
             $_SESSION['logged_in'] = true;
             
             logIt("Brugeren".$username."(id: ".$row['id'].") blev logget ind", 0);
             
             header("location: index.php");
        } else
          {
                  logIt("Forkert brugernavn eller adgangskode", 1);
                  $msg = "Forkert brugernavn eller adgangskode.";
                 return $msg;
              }
   }
 }

@Robert Larsen:
Sådan her?

 function login($username, $password)
 {
  if(!$username||!$password) 
  {
      $msg = "Udfyld venligst begge felter.";
     return $msg;
  } else 
     {
          $username = mysql_real_escape_string($username); 
           $password = hash('sha512', $password);    
          
         $login = mysql_query("SELECT * FROM user WHERE username = '$username' AND password = '$password'");
        $login_count = mysql_num_rows($login);
        if($login_count == 1)
        {
            $row = mysql_fetch_assoc($login);
             $_SESSION['user_id'] = $row['id'];
             $_SESSION['logged_in'] = true;
             
             logIt("Brugeren".$username."(id: ".$row['id'].") blev logget ind", 0);
             
             header("location: index.php");
        } else
          {
                  logIt("Forkert brugernavn eller adgangskode", 1);
                  $msg = "Forkert brugernavn eller adgangskode.";
                 return $msg;
              }
   }
 }

Præcis!