netdev
[Top] [All Lists]

[PATCH 4/7] [IPV4]: Fix DST leak in icmp_push_reply()

To: linux-kernel@xxxxxxxxxxxxxxx, stable@xxxxxxxxxx, Ollie Wild <aaw@xxxxxxxxxxxx>
Subject: [PATCH 4/7] [IPV4]: Fix DST leak in icmp_push_reply()
From: Chris Wright <chrisw@xxxxxxxx>
Date: Fri, 26 Aug 2005 12:17:59 -0700
Cc: Justin Forbes <jmforbes@xxxxxxxxxxx>, Zwane Mwaikambo <zwane@xxxxxxxxxxxxxxxx>, "Theodore Ts'o" <tytso@xxxxxxx>, Randy Dunlap <rdunlap@xxxxxxxxxxxx>, Chuck Wolber <chuckw@xxxxxxxxxxxxxxxx>, torvalds@xxxxxxxx, akpm@xxxxxxxx, alan@xxxxxxxxxxxxxxxxxxx, Maillist netdev <netdev@xxxxxxxxxxx>, Patrick McHardy <kaber@xxxxxxxxx>, "David S. Miller" <davem@xxxxxxxxxxxxx>, Chris Wright <chrisw@xxxxxxxx>
References: <20050826191755.052951000@localhost.localdomain>
Sender: netdev-bounce@xxxxxxxxxxx
-stable review patch.  If anyone has any  objections, please let us know.
------------------

Based upon a bug report and initial patch by
Ollie Wild.

Signed-off-by: Patrick McHardy <kaber@xxxxxxxxx>
Signed-off-by: "David S. Miller" <davem@xxxxxxxxxxxxx>
Signed-off-by: Chris Wright <chrisw@xxxxxxxx>
---
 net/ipv4/icmp.c |   12 ++++++------
 1 files changed, 6 insertions(+), 6 deletions(-)

Index: linux-2.6.12.y/net/ipv4/icmp.c
===================================================================
--- linux-2.6.12.y.orig/net/ipv4/icmp.c
+++ linux-2.6.12.y/net/ipv4/icmp.c
@@ -349,12 +349,12 @@ static void icmp_push_reply(struct icmp_
 {
        struct sk_buff *skb;
 
-       ip_append_data(icmp_socket->sk, icmp_glue_bits, icmp_param,
-                      icmp_param->data_len+icmp_param->head_len,
-                      icmp_param->head_len,
-                      ipc, rt, MSG_DONTWAIT);
-
-       if ((skb = skb_peek(&icmp_socket->sk->sk_write_queue)) != NULL) {
+       if (ip_append_data(icmp_socket->sk, icmp_glue_bits, icmp_param,
+                          icmp_param->data_len+icmp_param->head_len,
+                          icmp_param->head_len,
+                          ipc, rt, MSG_DONTWAIT) < 0)
+               ip_flush_pending_frames(icmp_socket->sk);
+       else if ((skb = skb_peek(&icmp_socket->sk->sk_write_queue)) != NULL) {
                struct icmphdr *icmph = skb->h.icmph;
                unsigned int csum = 0;
                struct sk_buff *skb1;

--

<Prev in Thread] Current Thread [Next in Thread>
  • [PATCH 4/7] [IPV4]: Fix DST leak in icmp_push_reply(), Chris Wright <=