From b346c11e228f60113fb60eac1d3f9e39d089ff24 Mon Sep 17 00:00:00 2001 From: Benny Prijono Date: Tue, 8 Oct 2013 11:14:42 +0000 Subject: Re #1703 General bug fixes: fixed some wrong null checking order git-svn-id: http://svn.pjsip.org/repos/pjproject/trunk@4615 74dad513-b988-da41-8d7b-12977e46ad98 --- pjnath/src/pjnath/stun_msg.c | 8 ++++---- pjnath/src/pjnath/stun_sock.c | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) (limited to 'pjnath') diff --git a/pjnath/src/pjnath/stun_msg.c b/pjnath/src/pjnath/stun_msg.c index cce914e2..445a26ed 100644 --- a/pjnath/src/pjnath/stun_msg.c +++ b/pjnath/src/pjnath/stun_msg.c @@ -1137,11 +1137,11 @@ PJ_DEF(pj_status_t) pj_stun_string_attr_init( pj_stun_string_attr *attr, int attr_type, const pj_str_t *value) { - INIT_ATTR(attr, attr_type, value->slen); - if (value && value->slen) + INIT_ATTR(attr, attr_type, 0); + if (value && value->slen) { + attr->value.slen = value->slen; pj_strdup(pool, &attr->value, value); - else - attr->value.slen = 0; + } return PJ_SUCCESS; } diff --git a/pjnath/src/pjnath/stun_sock.c b/pjnath/src/pjnath/stun_sock.c index 333dedd9..c51527db 100644 --- a/pjnath/src/pjnath/stun_sock.c +++ b/pjnath/src/pjnath/stun_sock.c @@ -211,7 +211,7 @@ PJ_DEF(pj_status_t) pj_stun_sock_create( pj_stun_config *stun_cfg, if (stun_sock->ka_interval == 0) stun_sock->ka_interval = PJ_STUN_KEEP_ALIVE_SEC; - if (cfg && cfg->grp_lock) { + if (cfg->grp_lock) { stun_sock->grp_lock = cfg->grp_lock; } else { status = pj_grp_lock_create(pool, NULL, &stun_sock->grp_lock); -- cgit v1.2.3