actions/itgmania212121
1
0
Fork 0
Code Issues Pull Requests Actions 1 Packages Projects Releases Wiki Activity

Strip the newlines off of the XML before the Warn(). Also, we should never pass user-supplied data as a format string.

Browse Source
This commit is contained in:
Steve Checkoway
2007-12-15 23:21:30 +00:00
parent 05c9c69fbb
commit 4873e924d7
1 changed files with 3 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
stepmania/src/Background.cpp
+3 -1
View File
@@ -723,7 +723,9 @@ void BackgroundImpl::Layer::UpdateCurBGChange( const Song *pSong, float fLastMus
if( iter == m_BGAnimations.end() ) if( iter == m_BGAnimations.end() )
{ {
XNode *pNode = change.m_def.CreateNode(); XNode *pNode = change.m_def.CreateNode();
LOG->Warn( "Tried to switch to a background that was never loaded\n" + XmlFileUtil::GetXML(pNode) ); RString xml = XmlFileUtil::GetXML( pNode );
Trim( xml );
LOG->Warn( "Tried to switch to a background that was never loaded:\n%s", xml.c_str() );
SAFE_DELETE( pNode ); SAFE_DELETE( pNode );
return; return;
} }