On Sep 27,  3:44pm, m...@m00nbsd.net (Maxime Villard) wrote:
-- Subject: Re: CVS commit: src/sys/dev/pci

| This fix is wrong.
| 
| -     memset(p, sizeof(struct twa_param_9k *), 10);
| +     memset(p, 0, sizeof(*p));
| 
| p is a [10] array, your memset only initializes the first pointer.
| 
| Seriously, guy, ...

Yes, this is the case where *p is wrong. Thank c for making this inconsistent!

christos

Reply via email to