[Libreoffice] [PATCH] printf without format considered harmful
Cédric Bosdonnat
cedric.bosdonnat.ooo at free.fr
Mon Oct 4 02:01:10 PDT 2010
Hello Norbert,
Thanks for your patch. I just pushed it.
Regards,
--
Cedric
On Sun, 2010-10-03 at 14:24 -0500, Norbert Thiebaud wrote:
> [PATCH] printf without format considered harmful
>
> svtools/bmpmaker/bmp.cxx | 2 +-
> svtools/bmpmaker/bmpsum.cxx | 2 +-
> svtools/bmpmaker/g2g.cxx | 2 +-
> 3 files changed, 3 insertions(+), 3 deletions(-)
>
> basic/source/app/msgedit.cxx | 2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/svtools/bmpmaker/bmp.cxx b/svtools/bmpmaker/bmp.cxx
> index b5de769..78c8c94 100644
> --- a/svtools/bmpmaker/bmp.cxx
> +++ b/svtools/bmpmaker/bmp.cxx
> @@ -161,7 +161,7 @@ void BmpApp::Message( const String& rText, BYTE cExit )
>
> ByteString aText( rText, RTL_TEXTENCODING_UTF8 );
> aText.Append( "\r\n" );
> - fprintf( stderr, aText.GetBuffer() );
> + fprintf( stderr, "%s", aText.GetBuffer() );
> }
>
> // -----------------------------------------------------------------------------
> diff --git a/svtools/bmpmaker/bmpsum.cxx b/svtools/bmpmaker/bmpsum.cxx
> index 574f7f6..3a5eef6 100644
> --- a/svtools/bmpmaker/bmpsum.cxx
> +++ b/svtools/bmpmaker/bmpsum.cxx
> @@ -167,7 +167,7 @@ void BmpSum::Message( const String& rText, BYTE nExitCode )
>
> ByteString aText( rText, RTL_TEXTENCODING_UTF8 );
> aText.Append( "\r\n" );
> - fprintf( stderr, aText.GetBuffer() );
> + fprintf( stderr, "%s", aText.GetBuffer() );
> }
>
> // -----------------------------------------------------------------------------
> diff --git a/svtools/bmpmaker/g2g.cxx b/svtools/bmpmaker/g2g.cxx
> index dbdb9ac..7dc52d8 100644
> --- a/svtools/bmpmaker/g2g.cxx
> +++ b/svtools/bmpmaker/g2g.cxx
> @@ -125,7 +125,7 @@ void G2GApp::Message( const String& rText, BYTE nExitCode )
>
> ByteString aText( rText, RTL_TEXTENCODING_UTF8 );
> aText.Append( "\r\n" );
> - fprintf( stderr, aText.GetBuffer() );
> + fprintf( stderr, "%s", aText.GetBuffer() );
> }
>
> // -----------------------------------------------------------------------------
> diff --git a/basic/source/app/msgedit.cxx b/basic/source/app/msgedit.cxx
> index bb22390..1dbade0 100644
> --- a/basic/source/app/msgedit.cxx
> +++ b/basic/source/app/msgedit.cxx
> @@ -217,7 +217,7 @@ void MsgEdit::AddAnyMsg( TTLogMsg *LogMsg )
> // restore Original Msg
> LogMsg->aDebugData.aMsg = aOriginalMsg;
>
> - printf( ByteString( aPrintMsg, RTL_TEXTENCODING_UTF8
> ).GetBuffer() );
> + printf( "%s", ByteString( aPrintMsg,
> RTL_TEXTENCODING_UTF8 ).GetBuffer() );
> }
> }
> }
> --
> _______________________________________________
> LibreOffice mailing list
> LibreOffice at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/libreoffice
More information about the LibreOffice
mailing list