Formatierter Zeit String in C++ gibt falsches Ergebnis aus

Question

Ich habe diese Funktion in C++

void scSystemTimeF(JObject *c, void *) {       
    char *fmtstr = (char*)c->getParameter("str")->getString().c_str();
    time_t currentTime = time(NULL);
    struct tm* timeInfo = localtime(&currentTime); 
    char fmt[50]; 
    strftime(fmt, sizeof(fmt), fmtstr, timeInfo);
    c->getReturnVar()->setString(fmt); 
}

Wenn ich als Formatstring das angebe

"%d.%m.%y %H:%M"

kommt das richtige Ergebnis

"05 10 2025 15 54"

Wenn ich aber auch die Sekunden auch angeben will

"%d.%m.%y %H:%M:%S"

kommt als Ausgabe plötzlich

"/etc/localtime"

Wtf wie kommt das und wie fixe ich das?

Answer 1

Du castest std::string::c_str() zu char*, aber c_str() liefert einen temporären Zeiger, der ungültig wird, sobald das std::string-Objekt zerstört wird. Wenn getString() ein temporäres Objekt ist, zeigt fmtstr ins Leere und strftime() bekommt Müll. Der Format Specifier %S ist nur ein Trigger, der die Länge verändert oder den Speicherzugriff verschiebt - und dadurch wird der Fehler sichtbar. Es ist nicht %S selbst, sondern der ungültige Zeiger. Ändere die Zeile

char *fmtstr = (char*)c->getParameter("str")->getString().c_str();

zu

std::string fmtstr = c->getParameter("str")->getString();

Und die Zeile

strftime(fmt, sizeof(fmt), fmtstr, timeInfo);

ändere zu

strftime(fmt, sizeof(fmt), fmtstr.c_str(), timeInfo);

Du kannst auch den Buffer vergrößern falls du einmal längere Ausgaben hast (z.B mit Name des Wochentag o.ä).