riched20.dll asserts

[Aric Stewart]

Hello,

So looking into an issue with IE6 i found the following problem.

If I install IE6, and then go to a japanese web site. (for example 
http://www.mainichi.co.jp) I get given the option to download and 
install the Japanese Text Display Support. However if i click Download 
then the download reports as a failure (a message about having to set 
your security settings from High to Medium, but that is a red herring)

The issue is that the assert in riched20/run.c line 115 is being hit 
during a call to RichEditANSIWndProc for the message EM_LINELENGTH. If i 
change the assert to instead to return 0, then the certificate dialog is 
displayed and the download progresses without issue.   (there is still 
an issue with the install but that is not richedit related)

Does someone with richedit experience want to help figure out what to do 
with that assert beyond just changing it to an error trace and returning 0?

thanks
-aric

[Ulrich Czekalla]

Hi Aric,

Could you try the attached patch. I must admit I don't fully understand the
underlying data structures but I think this patch fixes the problem.

/Ulrich

On Tue, Sep 27, 2005 at 11:36:58AM -0500, Aric Stewart wrote:
> Hello,
>
> So looking into an issue with IE6 i found the following problem.
>
> If I install IE6, and then go to a japanese web site. (for example 
> http://www.mainichi.co.jp) I get given the option to download and 
> install the Japanese Text Display Support. However if i click Download 
> then the download reports as a failure (a message about having to set 
> your security settings from High to Medium, but that is a red herring)
>
> The issue is that the assert in riched20/run.c line 115 is being hit 
> during a call to RichEditANSIWndProc for the message EM_LINELENGTH. If i 
> change the assert to instead to return 0, then the certificate dialog is 
> displayed and the download progresses without issue.   (there is still 
> an issue with the install but that is not richedit related)
>
> Does someone with richedit experience want to help figure out what to do 
> with that assert beyond just changing it to an error trace and returning 0?
>
> thanks
> -aric

[Krzysztof Foltman]

Ulrich Czekalla wrote:
> Could you try the attached patch. I must admit I don't fully understand the
> underlying data structures but I think this patch fixes the problem.

I'll take a look at that, but can anyone give another (simple) test case?

Krzysztof