something != "" can be true also when the left value does not exist
authorMiklos Tirpak <miklos@iptel.org>
Wed, 23 Apr 2008 14:25:18 +0000 (14:25 +0000)
committerMiklos Tirpak <miklos@iptel.org>
Wed, 23 Apr 2008 14:25:18 +0000 (14:25 +0000)
etc/ser-oob.cfg

index 4f481e8..5f93181 100644 (file)
@@ -646,7 +646,7 @@ route[RTPPROXY]
        }
 
        # turn the RTP proxy on for INVITEs and UPDATEs
-       if (((method=="INVITE" || method == "UPDATE") && @msg.body != "")  && !isflagset(FLAG_RTP_PROXY)) {
+       if (((method=="INVITE" || method == "UPDATE") && @msg.body)  && !isflagset(FLAG_RTP_PROXY)) {
                force_rtp_proxy('r');
                append_hf("P-RTP-Proxy: YES\r\n");
                setflag(FLAG_RTP_PROXY);
@@ -1216,8 +1216,7 @@ onreply_route[REPLY_ROUTE]
        # if a NAT is involved and this is the final positive reply
        # which contains a body, start to use the RTP proxy
        if (isflagset(FLAG_RTP_PROXY) &&
-               status=~"(18[03])|(2[0-9][0-9])" &&
-               @msg.body != "") {
+               status=~"(18[03])|(2[0-9][0-9])" && @msg.body) {
                force_rtp_proxy('r');
        }