Created attachment 181055 [details] git patch We have found and fixed bugs using PVS-Studio tool. PVS-Studio is a static code analyzer for C, C++ and C#: https://www.viva64.com/en/pvs-studio/ Analyzer warnings: V764 Possible incorrect order of arguments passed to 'sati_scsi_fixed_sense_data_construct' function: 'scsi_status' and 'response_code'. sati_util.c 621 V764 Possible incorrect order of arguments passed to 'sati_scsi_descriptor_sense_data_construct' function: 'scsi_status' and 'response_code'. sati_util.c 626
I think your patch is wrong. While there indeed a confusion, it looks like wrong parameter order is only in a header file in function declaration. Real use seems correct.