On Sun, Jan 06, 2008 at 03:41:49AM -0700, John R. Shannon wrote: > Line 1124 has: > > memcpy(inqbuf->vendor, "ADAPTEC ACB-4000 ", 28); > > and line 1144 has: > > memcpy(inqbuf->vendor, "EMULEX MT-02 QIC ", 28); > > yet inqbuf->vendor is declared in struct scsipi_inquiry_data as: > > char vendor[8]; and is followed by: char product[16]; char revision[4]; so the memcpy updates all 3 fields :-) David -- David Laight: david%l8s.co.uk@localhost