i think i found the bug that causes incorrect setting of dp_translate
attrs pvar value.

current code in dp_update function has this:

        no_change = (dest->type == PVT_NONE) || (!repl->s) || (!repl->len);

        if (no_change)
                goto set_attr_pvar;

        memset(&val, 0, sizeof(pv_value_t));
        val.flags = PV_VAL_STR;
        val.rs = *repl;

and then

set_attr_pvar:

        if(!attr_pvar)
                return 0;

        val.rs = *attrs;

if there is no change (like is the case in my failing test), val is not
initialized and its type can be anything.

i'll commit fix to trunk.  let me know if it looks ok, i'll pick it also
to 3.3.

-- juha

_______________________________________________
SIP Express Router (SER) and Kamailio (OpenSER) - sr-users mailing list
sr-users@lists.sip-router.org
http://lists.sip-router.org/cgi-bin/mailman/listinfo/sr-users

Reply via email to