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

Issue 69340044: code review 69340044: net: fix non-blocking connect handling on dragonfly (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years ago by jsing
Modified:
11 years ago
Reviewers:
gobot, mikio
CC:
mikio, golang-codereviews
Visibility:
Public.

Description

net: fix non-blocking connect handling on dragonfly Performing multiple connect system calls on a non-blocking socket under DragonFly BSD does not necessarily result in errors from earlier connect calls being returned, particularly if we are connecting to localhost. Instead, once netpoll tells us that the socket is ready, get the SO_ERROR socket option to see if the connection succeeded or failed. Fixes issue 7474

Patch Set 1 #

Patch Set 2 : diff -r 4cd83f2e218e https://go.googlecode.com/hg/ #

Patch Set 3 : diff -r 4cd83f2e218e https://go.googlecode.com/hg/ #

Patch Set 4 : diff -r 4cd83f2e218e https://go.googlecode.com/hg/ #

Unified diffs Side-by-side diffs Delta from patch set Stats (+22 lines, -0 lines) Patch
M src/pkg/net/fd_unix.go View 1 2 3 1 chunk +22 lines, -0 lines 0 comments Download

Messages

Total messages: 4
jsing
Hello mikioh.mikioh@gmail.com (cc: golang-codereviews@googlegroups.com), I'd like you to review this change to https://go.googlecode.com/hg/
11 years ago (2014-03-05 12:47:32 UTC) #1
mikio
LGTM
11 years ago (2014-03-05 13:04:21 UTC) #2
jsing
*** Submitted as https://code.google.com/p/go/source/detail?r=2c2ecbe5ae2a *** net: fix non-blocking connect handling on dragonfly Performing multiple connect ...
11 years ago (2014-03-05 13:07:26 UTC) #3
gobot
11 years ago (2014-03-05 13:18:57 UTC) #4
Message was sent while issue was closed.
This CL appears to have broken the netbsd-amd64-bsiegert builder.
Sign in to reply to this message.

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