inetd daytime crash

tfonz · 01-06-2016, 03:43 AM

64-bit inetd crashes (segfault) when connecting to the daytime service.

Recompiling with attached patch fixes it. I didn't find any upstream, so I post the patch here.

If <time.h> is not included, the type of ctime(3) defaults to int which is 32 bits. But on 64-bit, ctime returns a 64 bits pointer. When cast to int, the upper 32 bits are overwritten with the sign of the lower 32 bits, and inetd segfaults in sprintf (inetd.c, line 2103).

MarcT · 01-07-2016, 12:35 PM

Good catch, though what do you use daytime for?
I guess most users have it disabled in inetd.conf.

tfonz · 01-08-2016, 03:00 AM

Quote:

Originally Posted by MarcT

Good catch, though what do you use daytime for?

Nothing important. It's sometimes useful when testing software I develop.

But since the bug crashes inetd, it can (if enabled) affect other services started via inetd.

rworkman · 01-11-2016, 12:12 AM

Queued for volkerdi; thanks!

gus3 · 01-12-2016, 12:58 PM

Thank you tfonz, that one had me stumped as well.