Il 26/07/2012 10:52, James Bottomley ha scritto: >> > +static unsigned int virtscsi_get_lun(u8 *lun_bytes) >> > +{ >> > + unsigned int lun = (lun_bytes[2] << 8) | lun_bytes[3]; >> > + return lun & 16383; >> > +} >> > + > Why are you rolling your own incomplete version of scsilun_to_int here?
Because scsilun_to_int does not do the AND, so it would have exactly the same bug I'm fixing. Paolo -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/