Subject: Re: CVS commit: src
To: None <simonb@wasabisystems.com>
From: Jun-ichiro itojun Hagino <itojun@itojun.org>
List: source-changes
Date: 09/25/2003 06:20:58
> Jun-ichiro itojun Hagino wrote:
> 
> > Module Name:	src
> > Committed By:	itojun
> > Date:		Thu Sep  4 09:17:13 UTC 2003
> > 
> > Modified Files:
> > 
> > 	src/distrib/sets/lists/comp: mi
> > 	src/sys/netinet: Makefile in_pcb.c in_pcb.h raw_ip.c tcp_input.c
> > 	    tcp_subr.c tcp_usrreq.c tcp_var.h udp_usrreq.c
> > 	src/sys/netinet6: icmp6.c in6_pcb.c in6_pcb.h in6_proto.c in6_src.c
> > 	    raw_ip6.c udp6_usrreq.c udp6_var.h
> > Added Files:
> > 
> > 	src/sys/netinet: in_pcb_hdr.h
> 
> @@ -1226,43 +1219,30 @@
>  void
>  tcp_drain()
>  {
> +       struct inpcb_hdr *inph;
>         struct inpcb *inp;
>         struct tcpcb *tp;
>  
>         /*
>          * Free the sequence queue of all TCP connections.
>          */
> -       inp = CIRCLEQ_FIRST(&tcbtable.inpt_queue);
> +       inph = CIRCLEQ_FIRST(&tcbtable.inpt_queue);
>         if (inp)                                                /* XXX */
> 
> With this change, "inp" is now used unused...  Should
> that if () be checking inph instead?

	oops, yes.  will commit a fix.

itojun