On Tue, Dec 28, 2010 at 3:49 PM, <michael.w.pfeiffer@xxxxxxxxx> wrote: > - sprintf(buffer, B_TRANSLATE("%.2f KB"), size / 1024.0); > + snprintf(buffer, sizeof(buffer) - 1, B_TRANSLATE("%.2f KB"), > + size / 1024.0); just FYI, the - 1 is unnecessary, snprintf implicitly reserves one byte since it guarantees that the output will be NULL terminated. Regards, Rene