[SOLVED] Re: About coverity 988424 (svdfppt.cxx from filter module)

[Julien Nabet]

On 24/04/2013 18:25, Caolán McNamara wrote:
> On Sat, 2013-03-16 at 14:03 -0700, julien2412 wrote:
>> Hello,
>>
>> Scan coverity detected this:
>> CID 988424 (#1 of 1): Logically dead code (DEADCODE)dead_error_line:
>> Execution cannot reach this expression "nExtBuInd + nExtParaNibble<
>> aStyleTextProp9->size()" inside statement "if (nExtParaNibble&&  nExtB...".
>>
>> Indeed, we have these lines:
>>     5026     if ( nExtParaPos )
>>     5027     {
>>     5028         sal_uInt32 nExtBuInd = nMask&  0x3c00;
>>     5029         sal_uInt32  nExtParaNibble = 0;
>>     5030         if ( nExtBuInd )
>>     5031             nExtBuInd = ( aSet.mnFlags&  0x3c00 )>>  10;
>>     5032         if ( nExtBuInd<  aStyleTextProp9.size() )
>>     5033         {
>>     5034             if ( nExtParaNibble&&  ( ( nExtBuInd + nExtParaNibble )
>> <  aStyleTextProp9.size() ) )
>>     5035                 nExtBuInd += nExtParaNibble;
>>
>> (see
>> http://opengrok.libreoffice.org/xref/core/filter/source/msfilter/svdfppt.cxx#5026)
>>
>> Any idea what should be done?
> Git doesn't seem able to track back to where this code was really
> originally written across the old version control file moves and
> removes. But throwing some key words into google I can find the original
> code, and can determine it happened with
> "StyleTextProp with empty text PPT import (bnc#485630) [Thorsten]" which
> appeared to get merged to OpenOffice.org from ooo-build in Jun 2009.
>
> The original code it was based on was originally a loop where multiple
> properties were read and it made sense to increment nExtParaNibble
> between properties. Now it doesn't.
>
> So, yeah, all lines with nExtParaNibble in them or depend on those lines
> or exist only to set that value can be removed as it enters the
> condition as 0, and gets useless set to non 0 after exiting the
> condition.
Thank you Caolán, I pushed the fix on master (see 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=b5ca660e41fd81156ed62aa2c8451ed66a61eed4)

Julien