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