Hi Martin,

quoted

WRITE SAME 32 command handling reads WRPROTECT at the wrong offset in 
1st octet instead of 10th octet.

Instead of twiddling all these offsets I think it would be cleaner to turn the sbc_setup_write_same() flags[] into an 'unsigned char protect'. And then fix up
sbc_check_prot() to take 'protect' as argument instead of the full CDB and indexing into that.

OK, I will prepare new version of the patch.

Another option would be passing the index but since cdb[0] is only used for a rare error message I'm not sure it's worth it.

BR,
 Dmitry