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

Reply via email to