diff options
| author | paul <paul> | 2005-01-05 08:14:13 +0000 |
|---|---|---|
| committer | paul <paul> | 2005-01-05 08:14:13 +0000 |
| commit | a24a7e1b9e145d4c855d0aa4d919a79f598c645b (patch) | |
| tree | 02236b0424330cce2c902a9a971829a3610bac25 | |
| parent | 6c20046fd7494f9f529dac1b6f79da51b6efa44d (diff) | |
2005-01-05 Paul Jakma <paul@dishone.st>
* bgp_packet.c: (bgp_write) set socket to nonblock while writing
this should be generalised. See bugzilla #102. Fix supplied by
wawa@yandex-team.ru (Vladimir Ivanov).
| -rw-r--r-- | bgpd/ChangeLog | 6 | ||||
| -rw-r--r-- | bgpd/bgp_packet.c | 8 |
2 files changed, 14 insertions, 0 deletions
diff --git a/bgpd/ChangeLog b/bgpd/ChangeLog index b9d0cd59..79414def 100644 --- a/bgpd/ChangeLog +++ b/bgpd/ChangeLog @@ -1,3 +1,9 @@ +2005-01-05 Paul Jakma <paul@dishone.st> + + * bgp_packet.c: (bgp_write) set socket to nonblock while writing + this should be generalised. See bugzilla #102. Fix supplied by + wawa@yandex-team.ru (Vladimir Ivanov). + 2004-12-08 Andrew J. Schorr <ajschorr@alumni.princeton.edu> * *.c: Change level of debug messages to LOG_DEBUG. diff --git a/bgpd/bgp_packet.c b/bgpd/bgp_packet.c index 4eb7050c..5640e2b7 100644 --- a/bgpd/bgp_packet.c +++ b/bgpd/bgp_packet.c @@ -566,10 +566,17 @@ bgp_write (struct thread *thread) while (1) { int writenum; + int val; s = bgp_write_packet (peer); if (! s) return 0; + + /* XXX: FIXME, the socket should be NONBLOCK from the start + * status shouldnt need to be toggled on each write + */ + val = fcntl (peer->fd, F_GETFL, 0); + fcntl (peer->fd, F_SETFL, val|O_NONBLOCK); /* Number of bytes to be sent. */ writenum = stream_get_endp (s) - stream_get_getp (s); @@ -577,6 +584,7 @@ bgp_write (struct thread *thread) /* Call write() system call. */ num = write (peer->fd, STREAM_PNT (s), writenum); write_errno = errno; + fcntl (peer->fd, F_SETFL, val); if (num <= 0) { /* Partial write. */ |