On Thu, Jun 30, 2011 at 10:39:29PM +0200, Bron Gondwana wrote:
> On Sat, Feb 13, 2010 at 09:09:05PM +1100, Bron Gondwana wrote:
> > + if (config_getint(IMAPOPT_TCP_KEEPALIVE_CNT)) {
> > + r = setsockopt(be->sock, SOL_TCP, TCP_KEEPCNT, &optval, optlen);
> > + if (r < 0) {
> > + syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPCNT): %m");
> > + }
> > + }
> > + if (config_getint(IMAPOPT_TCP_KEEPALIVE_IDLE)) {
> > + r = setsockopt(be->sock, SOL_TCP, TCP_KEEPIDLE, &optval, optlen);
> > + if (r < 0) {
> > + syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPIDLE): %m");
> > + }
> > + }
> > + if (config_getint(IMAPOPT_TCP_KEEPALIVE_INTVL)) {
> > + r = setsockopt(be->sock, SOL_TCP, TCP_KEEPINTVL, &optval, optlen);
> > + if (r < 0) {
> > + syslog(LOG_ERR, "unable to setsocketopt(TCP_KEEPINTVL): %m");
> > + }
> > + }
>
> Why didn't one of you scumbags code review me and tell me
> how bogus this is??
Because it was so different from the patch that I submitted that I
assumed it addressed a different problem.
> Now I have to fix it.
>
> "keepalive interval 1 indeed"
--
-Gary Mills- -Unix Group- -Computer and Network Services-
----
Cyrus Home Page: http://www.cyrusimap.org/
List Archives/Info: http://lists.andrew.cmu.edu/pipermail/info-cyrus/
