On Tue, Jan 19, 2010 at 11:36:49AM +1100, Dave Chinner wrote: > - xname.name = (char *)dep->name; > + xname.name = (uchar_t *)dep->name; Aren't these both unsigned after the latest version of the signedness patches are in? Maybe just put the other one in for now.