[lkml]   [2010]   [Nov]   [15]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH/RFC] netfilter: nf_conntrack_sip: Handle quirky Cisco phones
    On 15.11.2010 04:01, Kevin Cernekee wrote:
    > On Sun, Nov 14, 2010 at 11:57 AM, Eric Dumazet <> wrote:
    >> Via: SIP/2.0/UDP;branch=xxxxxxxx
    >> Maybe a fix would be to use this "5060" port, instead of hardcoding it
    >> like you did ?
    > Just posted v2... appreciate the advice so far.
    > My new code in process_sip_request() looks for an address match + port
    > mismatch between the IP source and the Via: header. This is how it
    > tries to detect whether we are talking directly to an afflicted Cisco
    > phone. If the address doesn't match, I assume the request is passing
    > through a non-SIP-aware NAT router so there is no special handling.
    > Assuming we can reliably detect the "quirky phone" condition, is there
    > any way to just trick Netfilter into thinking the source port was 5060
    > instead of 49xxx? 3/4ths of the patch could probably be eliminated if
    > we could overwrite the port number inside tuplehash.

    The problem in doing this is that further packets from port 49xxx
    wouldn't be recognized as belonging to the same connection. If another
    packet was sent to the same destination conntrack would treat it as
    a new connection, rewrite the source port number, notice the clash and
    drop the packet.

    The same problem exists with your current patch, packets from port
    5060 to the same destination won't be recognized as belonging to the
    connection that sent the REGISTER and thus won't be able to modify the
    timeout or unregister.

    Basically we would need three-legged connections to handle this
    situation correctly. I've actually done some work to move one of
    the conntrack tuples to a ct_extend since in most situations
    (all except IPv4 NAT and ICMP packets) the tuples are symetrical
    and the second one can easily be derived, but I never managed
    to finish it - not sure what the problem was anymore, I'll see
    if I can still find those patches. With this we could simply
    attach a third tuple to a connection.

     \ /
      Last update: 2010-11-15 11:19    [W:0.022 / U:31.960 seconds]
    ©2003-2016 Jasper Spaans. hosted at Digital OceanAdvertise on this site