netdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] net: fix sparse warning (plain int as NULL)
@ 2005-06-12 22:05 Jesper Juhl
       [not found] ` <20050613.135950.48528369.davem@davemloft.net>
  0 siblings, 1 reply; 3+ messages in thread
From: Jesper Juhl @ 2005-06-12 22:05 UTC (permalink / raw)
  To: LKML; +Cc: David S. Miller, Ross Biro, netdev


Here's a patch to fix a small sparse warning in net/ipv4/tcp_input.c :
net/ipv4/tcp_input.c:4179:29: warning: Using plain integer as NULL pointer

Signed-off-by: Jesper Juhl <juhl-lkml@dif.dk>
---

 net/ipv4/tcp_input.c |    2 +-
 1 files changed, 1 insertion(+), 1 deletion(-)

--- linux-2.6.12-rc6-mm1-orig/net/ipv4/tcp_input.c	2005-06-12 15:58:58.000000000 +0200
+++ linux-2.6.12-rc6-mm1/net/ipv4/tcp_input.c	2005-06-12 23:58:41.000000000 +0200
@@ -4176,7 +4176,7 @@ int tcp_rcv_state_process(struct sock *s
 				 */
 				if (tp->rx_opt.saw_tstamp && tp->rx_opt.rcv_tsecr &&
 				    !tp->srtt)
-					tcp_ack_saw_tstamp(tp, 0, 0);
+					tcp_ack_saw_tstamp(tp, NULL, 0);
 
 				if (tp->rx_opt.tstamp_ok)
 					tp->advmss -= TCPOLEN_TSTAMP_ALIGNED;





Please CC me on replies.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] net: fix sparse warning (plain int as NULL)
       [not found]   ` <9a8748490506131421707008ed@mail.gmail.com>
@ 2005-06-13 21:31     ` David S. Miller
  2005-06-14 16:39       ` Jesper Juhl
  0 siblings, 1 reply; 3+ messages in thread
From: David S. Miller @ 2005-06-13 21:31 UTC (permalink / raw)
  To: jesper.juhl; +Cc: juhl-lkml, linux-kernel, ross.biro, netdev

From: Jesper Juhl <jesper.juhl@gmail.com>
Date: Mon, 13 Jun 2005 23:21:56 +0200

> Since tcp_ack_saw_tstamp() in 2.6.12-rc6-git6 only takes two arguments
> this patch is only relevant for -mm.

Thanks for the clarification.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [PATCH] net: fix sparse warning (plain int as NULL)
  2005-06-13 21:31     ` David S. Miller
@ 2005-06-14 16:39       ` Jesper Juhl
  0 siblings, 0 replies; 3+ messages in thread
From: Jesper Juhl @ 2005-06-14 16:39 UTC (permalink / raw)
  To: David S. Miller; +Cc: juhl-lkml, linux-kernel, ross.biro, netdev

On 6/13/05, David S. Miller <davem@davemloft.net> wrote:
> From: Jesper Juhl <jesper.juhl@gmail.com>
> Date: Mon, 13 Jun 2005 23:21:56 +0200
> 
> > Since tcp_ack_saw_tstamp() in 2.6.12-rc6-git6 only takes two arguments
> > this patch is only relevant for -mm.
> 
> Thanks for the clarification.
> 
You are welcome. Should I just send the patch on to Andrew for inclusion, or?

-- 
Jesper Juhl <jesper.juhl@gmail.com>
Don't top-post  http://www.catb.org/~esr/jargon/html/T/top-post.html
Plain text mails only, please      http://www.expita.com/nomime.html

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2005-06-14 16:39 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2005-06-12 22:05 [PATCH] net: fix sparse warning (plain int as NULL) Jesper Juhl
     [not found] ` <20050613.135950.48528369.davem@davemloft.net>
     [not found]   ` <9a8748490506131421707008ed@mail.gmail.com>
2005-06-13 21:31     ` David S. Miller
2005-06-14 16:39       ` Jesper Juhl

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).