FUJITA Kazutoshi wrote:
> --- /sys/netinet/tcp_subr.c.ORG Thu Jul 18 19:47:04 2002
> +++ /sys/netinet/tcp_subr.c     Sun Aug 11 04:00:09 2002
> @@ -393,7 +393,8 @@
>                 bcopy((caddr_t)th, (caddr_t)nth, sizeof(struct tcphdr));
>                 flags = TH_ACK;
>         } else {
> -               m_freem(m->m_next);
> +               if (m->m_next)
> +                       m_freem(m->m_next);
>                 m->m_next = 0;
>                 m->m_data = (caddr_t)ipgen;
>                 /* m_len is set later */

NO.

It is better to know that it's not NULL before it gets there.

If you check everything everywhere to see if it's NULL before
you do anything, then you are going to speen all your time
comparing things to NULL, rather than doing real work.

-- Terry

To Unsubscribe: send mail to [EMAIL PROTECTED]
with "unsubscribe freebsd-hackers" in the body of the message

Reply via email to