David Dawes
dawes****@XFree*****
2003年 8月 14日 (木) 23:58:18 JST
On Thu, Aug 14, 2003 at 09:32:40PM +0900, Chisato Yamauchi wrote: >From: David Dawes <dawes****@XFree*****> >Subject: Re: (patch seq: 5752) A patch for preventing XAA crash >Date: Wed, 13 Aug 2003 21:41:28 -0400 >Message-ID: <20030****@xfree*****> > >> Could you get some more information from XAACacheTile() that might show >> why pCache is NULL? > > We did not find the reason. > >> The values of size and the pCachePriv fields that get used might help. > > Here is "crash.log" and modified xaaPCache.c. > > http://www.mozilla.gr.jp/~kazhik/tmp/ > > This is one of the results investigated by Kazuhiko. >There may be something help in it. Thanks for the detailed log! It shows what I thought might be happening -- the number of cache slots being zero for the type of slot chosen: XAAInitPixmapCache; FreePixmapCachePrivate Num256++; 1 Num256++; 2 Num256++; 3 Num256--; 2 Num256--; 1 Num256--; 0 XAACacheTile() [1] *current: 0; max: 0; pCache: 0; w: c8; h: c8; size: c8 [2] *current: 1; max: 0; pCache: 0 Since it is possible for zero slots of a particular type to be available, the XAACache* functions need to be able to handle this. Returning the NULL value in that case, and checking for it looks reasonable to me, but Mark might have a better idea. Whatever the best solution is, it looks like it will need to be applied to other places too. David -- David Dawes Founder/committer/developer The XFree86 Project www.XFree86.org/~dawes