Ar Mer, 2006-06-21 am 10:17 +0400, ysgrifennodd Kirill Smelkov: > 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 Alan - : 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