Revert "BUG/MINOR: clock: fix a few occurrences of 'now' being used in place of 'date'"

This reverts commit aadcfc9ea6.

The parts affecting the DeviceAtlas addon were wrong actually, the
"now" variable was a local time_t in a file that's not compiled with
the haproxy binary (dadwsch). Only the fix to the calltrace is correct,
so better revert and fix the only one in a separate commit. No backport
is needed.
This commit is contained in:
Willy Tarreau 2023-04-27 18:12:06 +02:00
parent 64651d1769
commit fe1b3b8777
2 changed files with 4 additions and 4 deletions

View File

@ -44,7 +44,7 @@ static void dadwlog(dw_config_t cfg, const char* msg)
{ {
time_t now = time(NULL); time_t now = time(NULL);
char buf[26] = {0}; char buf[26] = {0};
ctime_r(&date, buf); ctime_r(&now, buf);
buf[24] = 0; buf[24] = 0;
fprintf(stderr, "%s: %s\n", buf, msg); fprintf(stderr, "%s: %s\n", buf, msg);
} }
@ -63,7 +63,7 @@ static dw_status_t dadwnot(void *a, dw_config_t *cfg)
jsond = da_getdatacreation(&o->atlas); jsond = da_getdatacreation(&o->atlas);
dwgetfinalp(o->dcfg.info, atlasp, sizeof(atlasp)); dwgetfinalp(o->dcfg.info, atlasp, sizeof(atlasp));
ctime_r(&jsond, jsondbuf); ctime_r(&jsond, jsondbuf);
ctime_r(&date, buf); ctime_r(&now, buf);
jsondbuf[24] = 0; jsondbuf[24] = 0;
buf[24] = 0; buf[24] = 0;
@ -150,7 +150,7 @@ int main(int argc, char **argv)
if (!dset) { if (!dset) {
time_t now = time(NULL); time_t now = time(NULL);
struct tm *cnow = gmtime(&date); struct tm *cnow = gmtime(&now);
memcpy(&global_deviceatlassch.o.dcfg.info.rtm, cnow, offsetof(struct tm, tm_mday)); memcpy(&global_deviceatlassch.o.dcfg.info.rtm, cnow, offsetof(struct tm, tm_mday));
} }

View File

@ -76,7 +76,7 @@ static int open_trace()
if (!log) if (!log)
log = stderr; log = stderr;
now_ptr = &date; now_ptr = &now;
if (getenv("HAPROXY_TRACE_FAST") != NULL) { if (getenv("HAPROXY_TRACE_FAST") != NULL) {
fast_time = 1; fast_time = 1;
now_ptr = &trace_now; now_ptr = &trace_now;