On Thu, Jul 18, 2013 at 11:39:08 -0400, Christos Zoulas wrote: > - don't use snprintf on a user-provided buffer This: - (void)snprintf(buf, sizeof(buf), fmt, escpage, suffix->s); + fixstring(buf, sizeof(buf), fmt, escpage); seems wrong, as it loses suffix->s, doesn't it? And fixstring is a terrible name. -uwe