Package: bidentd
Version: 1.1.4-1.1
Severity: important
Tags: upstream patch

Dear Maintainer,

it seems that the conntrack support in bidentd is broken. It reads from
/proc/net/ip_conntrack, checking that the protocol field of each line is
"TCP". However, on my system, these lines contain "tcp" instead. I
presume this changed in some past kernel version.

I've attached a patch that simply makes comparison case insensitive, so
old and new systems should both work.

I'll also send this report to upstream, but given the upcoming freeze
for Jessie, it might be good to include the patch into the Debian
version first?


On a related note: It seems there is a new upstream version available,
that integrated the single patch currently in the debian version.

Gr.

Matthijs
Description: Fix conntrack parsing for kernels using lowercase protocol names
Author: Matthijs Kooijman <matth...@stdin.nl>
Forwarded: yes

--- a/main.cc
+++ b/main.cc
@@ -169,7 +169,7 @@ static int Resolve(void)
             in_addr masqaddr = { byteswap(masqhost) };
             msyslog(ll_debug, " (masq parse) proto = %s, masqhost = %s, l = %u, r = %u, masqport = %u",
                     proto, inet_ntoa(masqaddr), l, r, masqport);
-            if(strcmp(proto, "TCP"))continue;
+            if(strcasecmp(proto, "TCP"))continue;
             if(r == remoteport && masqport == localport)
                 return Forward(masqhost, r, l);
         }

Reply via email to