Work at SourceForge, help us to make it a better place! We have an immediate need for a Support Technician in our San Francisco or Denver office.

Close

#103 Bug in security check of importANN.php

closed-fixed
Eloi George
None
5
2004-11-12
2004-11-11
Steven Levin
No

I observed this problem in the current CVS version of
the module.

The security check at the top of importANN.php looks
like this:

if(!$_SESSION['OBJ_user']->isDeity())
header('location:./../index.php');
exit();

The if statement has more than 2 line in its then
clause so brackets are required. Otherwise the script
just exits everytime.

Should be:

if(!$_SESSION['OBJ_user']->isDeity()) {
header('location:./../index.php');
exit();
}

Discussion

  • Eloi George
    Eloi George
    2004-11-12

    • assigned_to: nobody --> adarkling
    • status: open --> closed-fixed
     
  • Eloi George
    Eloi George
    2004-11-12

    Logged In: YES
    user_id=619893

    Thanks for catching that!

    Fixed in CVS and version 2.9