When _fs_load_glyphs calls fs_send_open_font with FontReopen set, it
passes a NULL name and namelen of 0, since fs_send_open_font is going
to reuse the previous name.

This overly restrictive check was added in XFree86 4.3.99.12:
http://cvsweb.xfree86.org/cvsweb/xc/lib/font/fc/fserve.c.diff?r1=3.23&r2=3.24
http://cvsweb.xfree86.org/cvsweb/xc/lib/font/fc/fserve.c?rev=3.24&content-type=text/vnd.viewcvs-markup

Signed-off-by: Alan Coopersmith <[email protected]>
---
 src/fc/fserve.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/fc/fserve.c b/src/fc/fserve.c
index 3ca3912..3585951 100644
--- a/src/fc/fserve.c
+++ b/src/fc/fserve.c
@@ -1533,7 +1533,7 @@ fs_send_open_font(pointer client, FontPathElementPtr fpe, 
Mask flags,
     if (conn->blockState & FS_GIVE_UP)
        return BadFontName;
 
-    if (namelen <= 0 || namelen > sizeof (buf) - 1)
+    if (namelen < 0 || namelen > sizeof (buf) - 1)
        return BadFontName;
 
     /*
-- 
1.7.9.2

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to