modules/lcr: Fixed to/from_gw tests when proto parameter is 0 (ANY)
authorJuha Heinanen <jh@tutpro.com>
Wed, 4 Jul 2012 14:15:43 +0000 (17:15 +0300)
committerJuha Heinanen <jh@tutpro.com>
Wed, 4 Jul 2012 14:15:43 +0000 (17:15 +0300)
- Also, updated README regarding handling of NULL value in lcr_gw
  transport column.

modules/lcr/README
modules/lcr/doc/lcr_admin.xml
modules/lcr/lcr_mod.c

index 3676a55..178f265 100644 (file)
@@ -268,8 +268,8 @@ Chapter 1. Admin Guide
    scheme, IP address, port, and transport protocol.
 
    Valid URI scheme values are NULL = sip, 1 = sip and 2 = sips. Currently
-   valid transport protocol values are NULL = none, 1 = udp, 2 = tcp, 3 =
-   tls, and 4 = sctp.
+   valid transport protocol values are NULL and 0 = none, 1 = udp, 2 =
+   tcp, 3 = tls, and 4 = sctp.
 
    As a side effect of gateway selection, selected gateway's tag and flags
    (that may contain information about the gateway and its capabilities)
index 39aa980..d498904 100644 (file)
        </para>
         <para>
         Valid URI scheme values are NULL = sip, 1 = sip and 2
-       = sips.  Currently valid transport protocol values are NULL =
+       = sips.  Currently valid transport protocol values are NULL and 0 =
        none, 1 = udp, 2 = tcp, 3 = tls, and 4 = sctp.
        </para>
        <para>
index 370e973..3bdcc98 100644 (file)
@@ -2309,8 +2309,7 @@ static int do_from_gw(struct sip_msg* _m, unsigned int lcr_id,
 
     /* Store tag and flags and return result */
     if ((res != NULL) &&
-       ((res->transport == transport) ||
-        ((res->transport == PROTO_NONE) && (transport == PROTO_UDP)))) {
+       ((transport == PROTO_NONE) || (res->transport == transport))) {
        LM_DBG("request game from gw\n");
        if (tag_avp_param) {
            val.s.s = res->tag;
@@ -2499,8 +2498,7 @@ static int do_to_gw(struct sip_msg* _m, unsigned int lcr_id,
 
     /* Return result */
     if ((res != NULL) &&
-       ((res->transport == transport) ||
-        ((transport == PROTO_NONE) && (res->transport == PROTO_UDP)))) {
+       ((transport == PROTO_NONE) || (res->transport == transport))) {
        LM_DBG("request goes to gw\n");
        return 1;
     } else {