Commit:     f53dc67c5e7babafe239b93a11678b0e05bead51
Parent:     276f2edc52e309b38a216245952e05880e182c83
Author:     Gerrit Renker <[EMAIL PROTECTED]>
AuthorDate: Wed Nov 28 08:35:08 2007 +0000
Committer:  David S. Miller <[EMAIL PROTECTED]>
CommitDate: Mon Jan 28 14:55:12 2008 -0800

    [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 <[EMAIL PROTECTED]>
    Signed-off-by: Arnaldo Carvalho de Melo <[EMAIL PROTECTED]>
    Signed-off-by: David S. Miller <[EMAIL PROTECTED]>
 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 33ce737..7caa7f5 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 
         * 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;
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at

Reply via email to