diff options
| author |   Lars Immisch <lars@ibp.de> | 2007-01-30 14:24:57 -0800 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2007-01-30 14:24:57 -0800 |
| commit | 7da5bfbb12e327b3a347ee3e076957cd6564eb56 (patch) | |
| tree | ea964be487ae79c645224237994628af491b19ec /net/ipv4 | |
| parent | [NETFILTER]: xt_connbytes: fix division by zero (diff) | |
| download | linux-dev-7da5bfbb12e327b3a347ee3e076957cd6564eb56.tar.xz linux-dev-7da5bfbb12e327b3a347ee3e076957cd6564eb56.zip | |
[NETFILTER]: SIP conntrack: fix skipping over user info in SIP headers
When trying to skip over the username in the Contact header, stop at the
end of the line if no @ is found to avoid mangling following headers.
We don't need to worry about continuation lines because we search inside
a SIP URI.
Fixes Netfilter Bugzilla #532.
Signed-off-by: Lars Immisch <lars@ibp.de>
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/ipv4')
| -rw-r--r-- | net/ipv4/netfilter/ip_conntrack_sip.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/net/ipv4/netfilter/ip_conntrack_sip.c b/net/ipv4/netfilter/ip_conntrack_sip.c index 3a26d63eed88..571d27e20910 100644 --- a/net/ipv4/netfilter/ip_conntrack_sip.c +++ b/net/ipv4/netfilter/ip_conntrack_sip.c @@ -283,8 +283,14 @@ static int skp_epaddr_len(const char *dptr, const char *limit, int *shift) { int s = *shift; - for (; dptr <= limit && *dptr != '@'; dptr++) + /* Search for @, but stop at the end of the line. + * We are inside a sip: URI, so we don't need to worry about + * continuation lines. */ + while (dptr <= limit && + *dptr != '@' && *dptr != '\r' && *dptr != '\n') { (*shift)++; + dptr++; + } if (*dptr == '@') { dptr++; |