riched20: Swap check for NULL ptr and first usage (Coverity).

Signed-off-by: Marcus Meissner <marcus@jet.franken.de>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Marcus Meissner 2016-10-19 09:26:00 +02:00 committed by Alexandre Julliard
parent af5b414000
commit c1a8fbbb77
1 changed files with 3 additions and 1 deletions

View File

@ -156,7 +156,7 @@ static ME_String *para_num_get_str( ME_Paragraph *para, WORD num )
{ {
/* max 4 Roman letters (representing '8') / decade + '(' + ')' */ /* max 4 Roman letters (representing '8') / decade + '(' + ')' */
ME_String *str = ME_MakeStringEmpty( 20 + 2 ); ME_String *str = ME_MakeStringEmpty( 20 + 2 );
WCHAR *p = str->szData; WCHAR *p;
static const WCHAR fmtW[] = {'%', 'd', 0}; static const WCHAR fmtW[] = {'%', 'd', 0};
static const WORD letter_base[] = { 1, 26, 26 * 26, 26 * 26 * 26 }; static const WORD letter_base[] = { 1, 26, 26 * 26, 26 * 26 * 26 };
/* roman_base should start on a '5' not a '1', otherwise the 'total' code will need adjusting. /* roman_base should start on a '5' not a '1', otherwise the 'total' code will need adjusting.
@ -177,6 +177,8 @@ static ME_String *para_num_get_str( ME_Paragraph *para, WORD num )
if (!str) return NULL; if (!str) return NULL;
p = str->szData;
if ((para->fmt.wNumberingStyle & 0xf00) == PFNS_PARENS) if ((para->fmt.wNumberingStyle & 0xf00) == PFNS_PARENS)
*p++ = '('; *p++ = '(';