Zitat von davewood

Hab mal atoi durch stroul ersetzt, is das ne bessere Alternative?

solange du kein error-checking einfuehrst nicht wirklich. es geht darum, dass du bei "atoi" keine moeglichkeit hast den fehlerfall abzufragen, bei strtol schon. ein "strtol("fucked_up", NULL, 10)" haut dir auch "0" zurueck. es liegt dann an dir zu schauen was passiert ist. "man strtol" hat ein ausfuehrliches beispiel.

Zitat von davewood

Sowie sprintf durch snprintf

die n-varianten verwende ich meist nur, wenn ich mit user-input rechne, oder input bei dem man sich nicht sicher sein kann wie lang er ist. manchmal bin ich aber auch zu schlampig, tja. man darf halt nicht den fehler machen und sich denken "is eh a snprintf, damit bin ich auf der sicheren seite". in deinem fall haetts nichts gebracht, aber da war der '*' ja ein typo. ob man es die n-varianten bei "sicherem code" verwendet ist wohl geschmackssache. je nachdem wie vorsichtig man sein mag.

Zitat von davewood

Naja der Input kommt aus einer Regexp und is maximal 3 digits lang, also kann die zahl weder zu gross noch zu klein noch nicht-zahl sein.

vorausgesetzt pcre ist perfekt und bugfrei. ich denke das muss jeder selbst entscheiden welchen libraries er trauen mag oder nicht. darauf wollte ich aber gar nicht hinaus. es war auf den generellen fall von atoi vs. strtol bezogen.