Source-Changes-D archive

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index][Old Index]

Re: CVS commit: src/usr.bin/man



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


Home | Main Index | Thread Index | Old Index