From 43990d32dc2d14e946c2b93d0810b33e87424d0a Mon Sep 17 00:00:00 2001 From: Benny Prijono Date: Fri, 4 Apr 2008 10:50:21 +0000 Subject: Ticket #522: Enable keep-alive for UDP transport even when STUN is not configured git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@1908 74dad513-b988-da41-8d7b-12977e46ad98 --- pjsip/src/pjsua-lib/pjsua_acc.c | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) (limited to 'pjsip/src') diff --git a/pjsip/src/pjsua-lib/pjsua_acc.c b/pjsip/src/pjsua-lib/pjsua_acc.c index e5b18ed1..2258cdd9 100644 --- a/pjsip/src/pjsua-lib/pjsua_acc.c +++ b/pjsip/src/pjsua-lib/pjsua_acc.c @@ -777,11 +777,20 @@ static void update_keep_alive(pjsua_acc *acc, pj_bool_t start, pj_status_t status; /* Only do keep-alive if: - * - STUN is enabled in global config, and * - ka_interval is not zero in the account, and * - transport is UDP. + * + * Previously we only enabled keep-alive when STUN is enabled, since + * we thought that keep-alive is only needed in Internet situation. + * But it has been discovered that Windows Firewall on WinXP also + * needs to be kept-alive, otherwise incoming packets will be dropped. + * So because of this, now keep-alive is always enabled for UDP, + * regardless of whether STUN is enabled or not. + * + * Note that this applies only for UDP. For TCP/TLS, the keep-alive + * is done by the transport layer. */ - if (pjsua_var.stun_srv.ipv4.sin_family == 0 || + if (/*pjsua_var.stun_srv.ipv4.sin_family == 0 ||*/ acc->cfg.ka_interval == 0 || param->rdata->tp_info.transport->key.type != PJSIP_TRANSPORT_UDP) { -- cgit v1.2.3