I have received a backtrace for an aborted multi-tty session from Ami
Fischman that is relevant to the trunk as well.

The abort was caused by Emacs entering the debugger during redisplay
because of a property list on a face symbol that was not a well-formed
Lisp list (see frames #23-26 in the backtrace at the bottom).

Beside the (rare) crash condition, there are other symptoms that are
easier to reproduce.  For example (in a recent CVS):

	emacs -q
	(setplist 'default (append (symbol-plist 'default) (cons 'foo 'bar)))
	C-h i
        <Redisplay loops>

Recursive aliases are fun as well:

        emacs -q
	(put 'foo 'face-alias 'default) 
        (put 'default 'face-alias 'foo)
        C-h i
        <unquitable loop>

The patch below fixes all these problems by protecting the call to
Fget inside resolve_face_name and limiting the number of indirections
during a single face lookup.