🚀 go-pugleaf

Aquila <braempje@netscape.net>:
 > In een stukje logging code gebruik ik volgend commando:
 > sprintf(buffer, "%s %.9f _%i_ --- %i %i ------- [%i:%i] (%s) %s", event,
 > timeval2double(now), id, pid, length, nid, dst_i, message, additional)
 > Nu is dit niet bepaald veilig omdat buffer een array met vaste lengte is.
 > Hoe kan ik dat veiliger maken?

Je zou snprintf() kunnen gebruiken, waarmee je een maximum grootte kunt
instellen. Eventueel kan je met snprintf() ook eerst bepalen hoe groot de
uitvoer gaat worden, om daarna genoeg geheugen te alloceren en de operatie
alsnog uit te voeren:

  /* grootte bepalen, inclusief ruimte voor \0 aan het einde */
  int size = 1 + snprintf(NULL, 0, "%s ....", ....);

  /* geheugen alloceren */
  char *buf = (char *) malloc(size);

  /* buffer vullen */
  snprintf(buf, size, "%s ....", ....);

(error-checks weggelaten)

 > In concreto: kan %s een maximum lengte opleggen?

Dat kan met bijvoorbeeld %.30s, waarmee dat veld maximaal 30 karakters in
de buffer mag innemen.

 > Kan %i een maximum lengte opleggen zonder daarvoor extra nullen te
 > printen?

Dat kan meen ik niet.

-- 
                                                                      robert