In article <pine.bsf.4.05.9903061057250.11342-100...@janus.syracuse.net>,
Brian Feldman  <gr...@unixhelp.org> wrote:
> I make a habit of checking out any interesting kernel changes in full, and one
> thing I found in the recent diff by Mr. Wollman for kern_timeout.c is:
> 
> +void
> +callout_init(c)
> +       struct  callout *c;
> +{
> +       bzero(c, sizeof c);
>  }
> 
> That doesn't look correct, does it?

Agreed.  I think it should be "sizeof *c".

John
-- 
  John Polstra                                               j...@polstra.com
  John D. Polstra & Co., Inc.                        Seattle, Washington USA
  "Nobody ever went broke underestimating the taste of the American public."
                                                            -- H. L. Mencken


To Unsubscribe: send mail to majord...@freebsd.org
with "unsubscribe freebsd-current" in the body of the message

Reply via email to