hat jemand eine ahnung was gemeint sein koennte?

Zitat

> in line 428 there is the following code:
> /* config file is ~/.bti */
> file = alloca(strlen(session->homedir) + 7); > sprintf(file, "%s/.bti",session->homedir);

Ugh, that's some horrible code, you really don't want to be reading it

> "/.bti" makes 5 chars + 1 for '\0' makes 6, but you allocate 7 bytes.
> the same thing in line 558:
> filename = alloca(strlen(session->homedir) + > strlen(session->logfile) + 3); >sprintf(filename, "%s/%s", session->homedir, session->logfile);
> the two strings + '/' + '\0' make +2, but you allocate +3
> I am sure there is something I missed, but I don't get it :-/
> any hints for a C n00b?

It's just being paranoid and allocating one extra byte just to be "safe".

Don't use it as a good example of coding style at all

Alles anzeigen

in welchem fall koennte dieses extrabyte safer sein?

oookay, das kann ich mal prinzipiell nachvollziehen.
in dem fall hab ich aber strlen + einen konstanten string/konstante anzahl an zeichen, was soll da also off-by-one werden?

aber gut, Plantschkuh! und greg k-h widerlegen wird wohl etwas schwierig werden

Zitat von Plantschkuh!

Edit: Wenn, dann würd ich es aber ganz explizit machen: Also statt foo + 7 würd ich foo + 6 + 1 schreiben. Naja, jedem seine Quirks

das muss noch viel expliziter werden :distur:

#define N (1)
#define OFF_BY_N_PREVENTION N


foo + 6 + OFF_BY_N_PREVENTION