CRE Loaded Community

Banner


Board index » Loaded Commerce Support » Security Issues

All times are UTC - 5 hours




Post new topic Reply to topic  [ 6 posts ] 
Author Message
 Post subject: vunerability in admin section
PostPosted: Mon Nov 09, 2009 10:33 pm 
Offline
CRE Newbie

Joined: Mon Nov 09, 2009 10:28 pm
Posts: 1
There is a massive vunerability in the admin section.

http://demos.creloaded.com/crecds/admin/orders.php/login.php?action=save%20HTTP/1.1&page=1&oID=1&action=edit

Login requirement is completely bypassed. This is present in oscommerce too, but you would think for the money I paid for this you guys could at least catch this one for me.

You MUST, MUST, MUST use basic auth on the admin section.

The oscommerce admin login is completely and utterly useless for security.


Top
 Profile  
 
 Post subject: Re: vunerability in admin section
PostPosted: Tue Nov 10, 2009 8:28 am 
Offline
CRE Expert

Joined: Thu Jul 13, 2006 12:00 am
Posts: 653
That is fixed here
http://www.creloaded.com/fdm_folder_fil ... fPath=0_69

and the newest version 6.4.0.a comes with that already in place

_________________
Regards,

------------------------------------------------------------------------
Kirk Osburne

CRE everything
WebGraphicsSource.com
------------------------------------------------------------------------


Top
 Profile  
 
 Post subject: Re: vunerability in admin section
PostPosted: Tue Nov 10, 2009 9:45 pm 
Offline
CRE Newbie

Joined: Sun Nov 01, 2009 3:23 pm
Posts: 4
Kirk wrote:
That is fixed here
http://www.creloaded.com/fdm_folder_fil ... fPath=0_69

and the newest version 6.4.0.a comes with that already in place


Just looking through the forum trying to keep patched, Since I am new to CRE, I Downloaded and installed the latest 6.04 B2B from the cre site within the last week or so. This should have this patch in it and the version.php claims it is 6.4.0.a. I did some checking to be sure and I noticed that the admin/includes/application_top.php in the Patch seems to differ from what is in the "already patched" full download release. I am wondering which is actually the correct file and what the differences between them actually mean.

I am looking at admin/includes/application_top.php

In the b2b patch downloaded files i see line 213 showing this
if ((basename($PHP_SELF) != 'sss_register.php') &&
(basename($PHP_SELF) != 'sss_validate.php') &&
(basename($PHP_SELF) != 'login.php')


In the b2b full 6.04 download line 213 reads
if ((basename($_SERVER['PHP_SELF']) != 'sss_register.php') &&
(basename($_SERVER['PHP_SELF']) != 'sss_validate.php') &&
(basename($_SERVER['PHP_SELF']) != 'login.php')

The version.php files in the patch and the full download are identical though.

What is the difference between these two lines. Which one of these correct or is there no difference?
:?:


Top
 Profile  
 
 Post subject: Re: vunerability in admin section
PostPosted: Tue Nov 10, 2009 9:54 pm 
Offline
CRE Expert

Joined: Thu Jul 13, 2006 12:00 am
Posts: 653
the patch 6.4.0a is the one with the security fix

_________________
Regards,

------------------------------------------------------------------------
Kirk Osburne

CRE everything
WebGraphicsSource.com
------------------------------------------------------------------------


Top
 Profile  
 
 Post subject: Re: vunerability in admin section
PostPosted: Tue Nov 10, 2009 10:18 pm 
Offline
CRE Newbie

Joined: Sun Nov 01, 2009 3:23 pm
Posts: 4
Kirk wrote:
the patch 6.4.0a is the one with the security fix

I confused :(

I just want to make sure I don't goof this up , so bear with me.


I downloaded "CRE_Loaded_PCI_B2B_v6.4.0.zip" (did this right now just to make sure it's the latest).

Extract the archive and then extract catalog.zip to see that

admin\includes\version.php is shows

define('INSTALLED_PATCH', '0.a');

and in admin\includes\application_top.php line 223 reads

if ((basename($_SERVER['PHP_SELF']) != 'sss_register.php') &&
(basename($_SERVER['PHP_SELF']) != 'sss_validate.php') &&
(basename($_SERVER['PHP_SELF']) != 'login.php')

I also downloaded CRE_Loaded_PCI_B2B_v6.4.0.a_Security_PHPSELF.zip
Inside the file the patch zip file admin\includes\version.php is shows

define('INSTALLED_PATCH', '0.a');

and in admin\includes\application_top.php line 223 reads

if ((basename($PHP_SELF) != 'sss_register.php') &&
(basename($PHP_SELF) != 'sss_validate.php') &&
(basename($PHP_SELF) != 'login.php')

Both the full download and the patch contain identical version.php's but the application_top.php files differ only at line 223.

The difference between the two application_top.php files being
basename($PHP_SELF) vs basename($_SERVER['PHP_SELF'])

If the patched zip is the correct one does this mean that the
the latest full download of b2b does not already include the fix?


Top
 Profile  
 
 Post subject: Re: vunerability in admin section
PostPosted: Sat May 22, 2010 10:07 am 
Offline
CRE Freak

Joined: Thu Feb 22, 2007 1:00 am
Posts: 69
The actual fix is replacing
Code:
$PHP_SELF = (isset($_SERVER['PHP_SELF']) ? $_SERVER['PHP_SELF'] : $_SERVER['SCRIPT_NAME']);

with
Code:
$PHP_SELF = $_SERVER['SCRIPT_NAME'];


in admin/includes/application_top.php. its also simple to do it your self.

You can also take extended measures to secure admin area like http://blog.wasimasif.com/hardening-pro ... dmin-area/

thanks

_________________
http://www.infolates.com/


Top
 Profile  
 
Display posts from previous:  Sort by  
Post new topic Reply to topic  [ 6 posts ] 

Board index » Loaded Commerce Support » Security Issues

All times are UTC - 5 hours


Who is online

Users browsing this forum: No registered users and 0 guests


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 post attachments in this forum

Search for:
Jump to:  
It is currently Wed Feb 08, 2012 9:08 am
Powered by phpBB © 2000, 2002, 2005, 2007 phpBB Group

Login

Top Listing

1. Cart2Cart - Shopping...
    Category: Shopping Cart Database Conversion Scripts
    
2. Points & Rewards PLUS!...
    Category: Add-Ons
    
3. Configuration Server...
    Category: Fixes
    
4. Credit Card with CCV
    Category: Payment Modules
    
5. CC7333_ATS
    Category: Templates
    
Show more...

Follow Us on Twitter

An error occurred

Oops, an error seems to have occurred. We're sorry for any inconvenience this might have caused. If the error persists, feel free to tell us about it.

CRE Loaded Community Chat hosted by CRE Loaded.

Join now


Chat about what's on your mind. More about public chats.


© CRE Loaded is a product of Chain Reaction Ecommerce, Inc. Usage & Privacy Policy