21 Jun 2006 15:44, Alan Cox wrote: > > id->model="TRANSCEND", > > id->fw_rev="20050811TRANSCEND" > > > > note the trailing in id->fw_rev, > > These are not null terminated strings in the ident block. So if you > merely print them or test against them you'll break on 8 char long > firmware names. They may also be space rather than \0 padded if shorter This justifies strstr usage. Thanks again. -- Kirill - : send the line "unsubscribe linux-ide" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html