Re: Assertion failes in w32uniscribe.c

[Jason Rumney]

Juanma Barranquero wrote:
> On Fri, Apr 4, 2008 at 10:14 AM, Jason Rumney <address@hidden> wrote:
>
>   
> >  (with old code, after the crash):
> >  frame 2
> >  pp otf_spec
> >
> >  (with new code):
> >  break w32uniscribe.c:569
> >  break w32uniscribe.c:618
> >  run -q
> >  pp otf_spec
> >     
>
> Are you assuming that after adding this:
>
>   /* Check the spec is in the right format.  */
>   if (!CONSP (otf_spec) || Flength (otf_spec) < 3)
>     return 0;
>
> it does not crash? Because it still does.
>   

I was assuming that the other part of the change would be what prevented 
this particular crash, since it was already getting past the point above:

     /* If features is not a cons, this font spec is messed up.  */
     if (!CONSP (features[i]))
       goto no_support;