diff options
| author | Gerrit Renker <gerrit@erg.abdn.ac.uk> | 2007-11-28 08:35:08 +0000 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2008-01-28 14:55:12 -0800 |
| commit | f53dc67c5e7babafe239b93a11678b0e05bead51 (patch) | |
| tree | 50737228e796667e8c24e9a821cb11bca372516a /net/dccp/output.c | |
| parent | 276f2edc52e309b38a216245952e05880e182c83 (diff) | |
| download | linux-3.10-f53dc67c5e7babafe239b93a11678b0e05bead51.tar.gz linux-3.10-f53dc67c5e7babafe239b93a11678b0e05bead51.tar.bz2 linux-3.10-f53dc67c5e7babafe239b93a11678b0e05bead51.zip | |
[DCCP]: Use AF-independent rebuild_header routine
This fixes a nasty bug: dccp_send_reset() is called by both DCCPv4 and DCCPv6, but uses
inet_sk_rebuild_header() in each case. This leads to unpredictable and weird behaviour:
under some conditions, DCCPv6 Resets were sent, in other not.
The fix is to use the AF-independent rebuild_header routine.
Signed-off-by: Gerrit Renker <gerrit@erg.abdn.ac.uk>
Signed-off-by: Arnaldo Carvalho de Melo <acme@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/dccp/output.c')
| -rw-r--r-- | net/dccp/output.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/dccp/output.c b/net/dccp/output.c index 33ce737ef3a..7caa7f57bb7 100644 --- a/net/dccp/output.c +++ b/net/dccp/output.c @@ -391,7 +391,7 @@ int dccp_send_reset(struct sock *sk, enum dccp_reset_codes code) * FIXME: what if rebuild_header fails? * Should we be doing a rebuild_header here? */ - int err = inet_sk_rebuild_header(sk); + int err = inet_csk(sk)->icsk_af_ops->rebuild_header(sk); if (err != 0) return err; |