Hi, On Thu, Jun 02, 2011 at 05:13:36PM +1000, Peter Hutterer wrote: > + if(HasConfineTo(client) == TRUE) > + memcpy((unsigned char *)&confine_to, (char*)&stuff[1] + stuff->mask_len * 4, 4); The first cast here is unnecessary. Cheers, Daniel