Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(177)

Side by Side Diff: src/net/tcpsockopt_openbsd.go

Issue 141730043: code review 141730043: net: don't set wrong option for controlling tcp keepali... (Closed)
Patch Set: diff -r 2a642ae2e6186583899ae9889655d414867e1f68 https://go.googlecode.com/hg/ Created 9 years, 6 months ago
Left:
Right:
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View unified diff | Download patch
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright 2009 The Go Authors. All rights reserved. 1 // Copyright 2009 The Go Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style 2 // Use of this source code is governed by a BSD-style
3 // license that can be found in the LICENSE file. 3 // license that can be found in the LICENSE file.
4 4
5 // TCP socket options for openbsd
6
7 package net 5 package net
8 6
9 import ( 7 import (
10 "os"
11 "syscall" 8 "syscall"
12 "time" 9 "time"
13 ) 10 )
14 11
15 // Set keep alive period.
16 func setKeepAlivePeriod(fd *netFD, d time.Duration) error { 12 func setKeepAlivePeriod(fd *netFD, d time.Duration) error {
17 » if err := fd.incref(); err != nil { 13 » // OpenBSD has no user-settable per-socket TCP keepalive
18 » » return err 14 » // options.
19 » } 15 » return syscall.EPROTONOSUPPORT
20 » defer fd.decref()
21
22 » // The kernel expects seconds so round to next highest second.
23 » d += (time.Second - time.Nanosecond)
24 » secs := int(d.Seconds())
25
26 » return os.NewSyscallError("setsockopt", syscall.SetsockoptInt(fd.sysfd, syscall.IPPROTO_TCP, syscall.SO_KEEPALIVE, secs))
27 } 16 }
OLDNEW
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b