Feature #1361: Add support for socket options TCP_KEEPCNT, TCP_KEEPIDLE, TCP_KEEPINTVL
1361 missed two codepaths
The changes for bug 1361 missed two codepaths (and this code reviewer takes full responsibility for missing them during review).
1.) The tcp_reinit_values() function needs to mention every field in tcp_t, according to the header file. It actually doesn't, but our new fields should be mentioned in there.
2.) tcp_init_values() has a (parent != NULL) case (i.e. sockets created by accept(3xn)) which doesn't copy over the new values.
Updated by Garrett D'Amore about 9 years ago
- % Done changed from 80 to 100
user: Dan McDonald <email@example.com>
date: Tue Aug 30 09:23:32 2011 -0700
1426 1361 missed two codepaths
Reviewed by: Garrett D'Amore <firstname.lastname@example.org>
Reviewed by: Richard Lowe<email@example.com>
Reviewed by: Albert Lee<firstname.lastname@example.org>
Approved by: Garrett D'Amore <email@example.com>