Hi, On 23 September 2011 00:08, Peter Hutterer <peter.hutterer at who-t.net> wrote: > lib->classes = Xmalloc(size_classes((xXIAnyInfo*)ptr, lib->num_classes)); I think this should be nclasses instead of lib->num_classes, no? With that: Reviewed-by: Daniel Stone <daniel at fooishbar.org> Cheers, Daniel