Re: Aric Stewart : usp10: Update diacritical script handling for non OpenType Itemization.

[Phil Krylov]

Hello Aric,

On Tue, Feb 5, 2013 at 11:25 PM, Alexandre Julliard  wrote:
> Module: wine
> Branch: master
> Commit: ef4b7c5948e43d4ceeed98ba4ddbf6c222259d36
> URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ef4b7c5948e43d4ceeed98ba4d...
>
> Author: Aric Stewart 
> Date:   Tue Feb  5 08:57:44 2013 -0600
>
> usp10: Update diacritical script handling for non OpenType Itemization.

(snip)
> -
>                for (j = i-1; j >= 0 &&  scripts[j] == first_script && pwcInChars[j] != Numeric_space; j--)
> -
>                {
> -
>                    WORD original = scripts[j];
> -
>                    if (original == Script_Ideograph || original == Script_Kana || original == Script_Yi || original == Script_CJK_Han || original == Script_Bopomofo)
> -
>                    {
> -
>                        asian = TRUE;
> -
>                        break;
> -
>                    }
> -
>                    if (original != Script_MathAlpha && scriptInformation[scripts[j]].props.fComplex)
> -
>                        break;
> -
>                    scripts[j] = scripts[i];
> -
>                    if (original == Script_Punctuation2)
> -
>                        break;
> -
>                }
> -
>                if (scriptInformation[scripts[j]].props.fComplex || asian)
> -
>                    scripts[i] = scripts[j];

I am not sure about the logic of this, so I can't commit a good patch,
but in the last statement you definitely access scripts[-1] when j
reaches the value of 0. Which may cause a crash here or (in my case)
later.

-- Ph.

Aric Stewart : usp10: Update diacritical script handling for non OpenType Itemization.

[Aric Stewart]

On 3/7/13 4:57 AM, Phil Krylov wrote:
> Hello Aric,
>
> On Tue, Feb 5, 2013 at 11:25 PM, Alexandre Julliard  wrote:
> > Module: wine
> > Branch: master
> > Commit: ef4b7c5948e43d4ceeed98ba4ddbf6c222259d36
> > URL:    http://source.winehq.org/git/wine.git/?a=commit;h=ef4b7c5948e43d4ceeed98ba4d...
> >
> > Author: Aric Stewart 
> > Date:   Tue Feb  5 08:57:44 2013 -0600
> >
> > usp10: Update diacritical script handling for non OpenType Itemization.
>
> (snip)
> > -
> >                for (j = i-1; j >= 0 &&  scripts[j] == first_script && pwcInChars[j] != Numeric_space; j--)
> > -
> >                {
> > -
> >                    WORD original = scripts[j];
> > -
> >                    if (original == Script_Ideograph || original == Script_Kana || original == Script_Yi || original == Script_CJK_Han || original == Script_Bopomofo)
> > -
> >                    {
> > -
> >                        asian = TRUE;
> > -
> >                        break;
> > -
> >                    }
> > -
> >                    if (original != Script_MathAlpha && scriptInformation[scripts[j]].props.fComplex)
> > -
> >                        break;
> > -
> >                    scripts[j] = scripts[i];
> > -
> >                    if (original == Script_Punctuation2)
> > -
> >                        break;
> > -
> >                }
> > -
> >                if (scriptInformation[scripts[j]].props.fComplex || asian)
> > -
> >                    scripts[i] = scripts[j];
>
> I am not sure about the logic of this, so I can't commit a good patch,
> but in the last statement you definitely access scripts[-1] when j
> reaches the value of 0. Which may cause a crash here or (in my case)
> later.
>
> -- Ph.

Hi there,

I am just getting back from a vacation so I will review this and your other patch early next week when I am fully back.

Thanks for doing work here!

-aric

Aric Stewart : usp10: Update diacritical script handling for non OpenType Itemization.

[Phil Krylov]

Hello,

On Fri, Mar 15, 2013 at 12:52 AM, Aric Stewart  wrote:
> I am just getting back from a vacation so I will review this and your other patch early next week when I am fully back.
>
> Thanks for doing work here!

Thanks for replying, wasn't sure you saw my posts. And of course
thanks for your work too.

-- Ph.