I'm just curious: why was the NULL-return checking on ch_realloc() added? I deliberately didn't check the return value because ch_realloc() checks for a failed BER allocation and asserts and exits if an allocation fails. AFAICT, the logging will never be called. john