diff options
| author | Julien Nabet <serval2412@yahoo.fr> | 2013-07-22 00:08:40 +0200 |
|---|---|---|
| committer | Fridrich Strba <fridrich@documentfoundation.org> | 2013-07-22 03:17:30 +0000 |
| commit | b11dc22caa903014c295bea62a6d86e5e12518d8 (patch) | |
| tree | acc82f300fd429cef1870803e27b3b05ab569517 | |
| parent | d3cf684146162a1528046ab2bcd7d9fa94443c69 (diff) | |
coverity#1038312: Dereference before null check
Change-Id: Ibe9babbe8610bf70a80b819e21d24c6572436092
Reviewed-on: https://gerrit.libreoffice.org/5015
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>
| -rw-r--r-- | include/sal/log-areas.dox | 4 | ||||
| -rw-r--r-- | l10ntools/source/helpmerge.cxx | 9 |
2 files changed, 9 insertions, 4 deletions
diff --git a/include/sal/log-areas.dox b/include/sal/log-areas.dox index 95ec6f1914c2..a78abda434c1 100644 --- a/include/sal/log-areas.dox +++ b/include/sal/log-areas.dox @@ -176,6 +176,10 @@ certain functionality. @li @c jfw.level1 @li @c jfw.level2 +@section l10ntools + +@li @c l10ntools + @section Math @li @c starmath diff --git a/l10ntools/source/helpmerge.cxx b/l10ntools/source/helpmerge.cxx index d79dfaea7fa2..161362e1feab 100644 --- a/l10ntools/source/helpmerge.cxx +++ b/l10ntools/source/helpmerge.cxx @@ -263,11 +263,12 @@ void HelpParser::ProcessHelp( LangHashMap* aLangHM , const OString& sCur , ResDa aLangHM->erase( sCur ); } } - else if( pResData == NULL ) + else if( pResData ) { - fprintf(stdout,"Can't find GID=%s LID=%s TYP=%s\n", - pResData->sGId.getStr(), pResData->sId.getStr(), - pResData->sResTyp.getStr()); + SAL_WARN( + "l10ntools", + "Can't find GID=" << pResData->sGId.getStr() << " LID=" + << pResData->sId.getStr() << " TYP=" << pResData->sResTyp.getStr() << "\n"); } pXMLElement->ChangeLanguageTag( OStringToOUString(sCur, RTL_TEXTENCODING_ASCII_US)); |