PHP Web Host - Quality Web Hosting For All PHP Applications Free RavenNuke(tm) Add Ons
  Login or Register
 • Home • Downloads • Your Account • Forums • 

View next topic
View previous topic


Google
 
Web RavenPHPScripts (This Site)
This forum is locked: you cannot post, reply to, or edit topics.   This topic is locked: you cannot edit posts or make replies.
Author Message
webservant
Worker
Worker


Joined: Feb 26, 2006
Posts: 206
Location: Springfield, MA

PostPosted: Sat Dec 23, 2006 12:57 pm Reply with quote Back to top

I just ugrade my test system from 2.5.03 to 2.5.04. The upgrade went smoothly, but I found an issue on the NukeSentinel Admin screen after running nsnst.php.

The following appears in the middle of the screen...
Code:
Warning: base64_decode() expects parameter 1 to be string, array given in /home/doulos/public_html/includes/nukesentinel.php on line 972
Admin Auth List
 
Warning: base64_decode() expects parameter 1 to be string, array given in /home/doulos/public_html/includes/nukesentinel.php on line 972
Scan for New Admins
 
Warning: base64_decode() expects parameter 1 to be string, array given in /home/doulos/public_html/includes/nukesentinel.php on line 972
Database Maintenance


I looked at the code around 972, and find this:
Code:
function is_god($axadmin) {
  global $db, $prefix, $aname;
  $tmpadm = base64_decode($axadmin);
  $tmpadm = explode(":", $tmpadm);
  $aname = $tmpadm[0];
  $apwd = $tmpadm[1];
  if(!empty($aname) AND !empty($apwd)) {
    $aname = trim($aname);
    $apwd = trim($apwd);
    $admrow = $db->sql_fetchrow($db->sql_query("SELECT * FROM `".$prefix."_authors` WHERE `aid`='$aname'"));
    //if((strtolower($admrow['name']) == "god" OR $admrow['radminsuper'] == 1) AND $admrow['pwd']==$apwd) { return 1; }
    if(strtolower($admrow['name']) == "god" AND $admrow['pwd']==$apwd) { return 1; }
  }
  return 0;
}


When I changed the function to be similar to is_admin in mainfile.php, it works. Here is my changed code:
Code:
function is_god($axadmin) {
  global $db, $prefix, $aname;
//  Replace with is_admin type if statement.
//  $tmpadm = base64_decode($axadmin);
//  $tmpadm = explode(":", $tmpadm);
//   $aname = $tmpadm[0];
//    $apwd = $tmpadm[1];
  if (!is_array($axadmin)) {
     $tmpadm = base64_decode($axadmin);
    $tmpadm = addslashes($tmpadm);
     $tmpadm = explode(":", $tmpadm);
   $aname = $tmpadm[0];
     $apwd = $tmpadm[1];
  } else {
   $aname = $axadmin[0];
     $apwd = $axadmin[1];
  }
  if(!empty($aname) AND !empty($apwd)) {
    $aname = trim($aname);
    $apwd = trim($apwd);
    $admrow = $db->sql_fetchrow($db->sql_query("SELECT * FROM `".$prefix."_authors` WHERE `aid`='$aname'"));
    //if((strtolower($admrow['name']) == "god" OR $admrow['radminsuper'] == 1) AND $admrow['pwd']==$apwd) { return 1; }
    if(strtolower($admrow['name']) == "god" AND $admrow['pwd']==$apwd) { return 1; }
  }
  return 0;
}
View user's profile Send private message Visit poster's website AIM Address
kguske
Site Admin


Joined: Jun 04, 2004
Posts: 6044

PostPosted: Sat Dec 23, 2006 9:15 pm Reply with quote Back to top

Haven't tried the new version yet, but we should notify Bob.
View user's profile Send private message
webservant
Worker
Worker


Joined: Feb 26, 2006
Posts: 206
Location: Springfield, MA

PostPosted: Sat Dec 23, 2006 10:09 pm Reply with quote Back to top

OK. How? Where?
View user's profile Send private message Visit poster's website AIM Address
kguske
Site Admin


Joined: Jun 04, 2004
Posts: 6044

PostPosted: Sat Dec 23, 2006 10:14 pm Reply with quote Back to top

Just did so. Thanks...
View user's profile Send private message
BobMarion
Former Admin in Good Standing


Joined: Oct 30, 2002
Posts: 1043
Location: RedNeck Land (known as Kentucky)

PostPosted: Sun Dec 24, 2006 1:56 am Reply with quote Back to top

Strage, it hasn't popped that warning up at me. What's even stranger is that has been coded like that since 2.4.2pl7. I will dive in right after X-Mas on it.
View user's profile Send private message Send e-mail Visit poster's website
webservant
Worker
Worker


Joined: Feb 26, 2006
Posts: 206
Location: Springfield, MA

PostPosted: Sun Jan 14, 2007 1:11 pm Reply with quote Back to top

Any ideas as to why this happened?
View user's profile Send private message Visit poster's website AIM Address
BobMarion
Former Admin in Good Standing


Joined: Oct 30, 2002
Posts: 1043
Location: RedNeck Land (known as Kentucky)

PostPosted: Sat Mar 03, 2007 3:46 pm Reply with quote Back to top

I'll ensure this goes into 2.5.07 as it completely slipped my mind for 2.5.06 Sad
View user's profile Send private message Send e-mail Visit poster's website
Susann
Moderator


Joined: Dec 19, 2004
Posts: 3143
Location: Germany:Moderator German NukeSentinel Support

PostPosted: Sat Mar 03, 2007 4:16 pm Reply with quote Back to top

Nobody reported ever such an issue. IŽve checked all my upgrades and couldnŽt find nothing special in NukeSentinel Admin. However, I have still a 3.2b 7.6 website IŽll check what IŽll find there after upgrade NukeSentinel.
View user's profile Send private message Visit poster's website
Display posts from previous:       
This forum is locked: you cannot post, reply to, or edit topics.   This topic is locked: you cannot edit posts or make replies.

View next topic
View previous topic
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum
Forums ©
 

All logos and trademarks in this site are property of their respective owner.
The comments are property of their posters, all the rest © 2002-2011 by Raven

You can syndicate our news using the file xml

CSE HTML Validator Helped Clean up This Page! [Valid RSS] valid RSS 2.0 Valid robots.txt Stop Spam Harvesters, Join Project Honey Pot

Website engines core code is © copyright by PHP-Nuke but has been heavily patched and modified by myself and others.
PHP-Nuke is a free software released under the GNU/GPL.


:: fisubice phpbb2 style by Daz :: PHP-Nuke theme by www.nukemods.com ::
:: fisubice Theme Modified by the RavenNuke™ Team ::

:: W3C CSS Compliance Validation :: W3C HTML 4.01 Transitional Compliance Validation ::

zerosum