Bug #1426
Feature #1361: Add support for socket options TCP_KEEPCNT, TCP_KEEPIDLE, TCP_KEEPINTVL
1361 missed two codepaths
| Status: | Resolved | Start date: | 2011-08-29 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Dan McDonald | % Done: | 100% | |
| Category: | kernel | |||
| Target version: | - | |||
| Difficulty: | Bite-size | Tags: |
Description
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.
acceptor.c 
- Test program
(2.02 KB)
History
#1
Updated by Dan McDonald about 5 years ago
- File acceptor.c added
Attached is a test program (acceptor.c) that'll open a server-side socket. Once you telnet to that port (default == 2112), it'll accept one connection, and compare parent vs. child values of the tcp_t fields in question.
#2
Updated by Garrett D'Amore about 5 years ago
- % Done changed from 80 to 100
Resolved in:
changeset: 13443:76ab5b799560
tag: tip
user: Dan McDonald <danmcd@nexenta.com>
date: Tue Aug 30 09:23:32 2011 -0700
description:
1426 1361 missed two codepaths
Reviewed by: Garrett D'Amore <garrett@nexenta.com>
Reviewed by: Richard Lowe<richlowe@richlowe.net>
Reviewed by: Albert Lee<trisk@nexenta.com>
Approved by: Garrett D'Amore <garrett@nexenta.com>
#3
Updated by Garrett D'Amore about 5 years ago
- Status changed from New to Resolved
Also available in: Atom