On Wed, Apr 13, 2016 at 05:53:08PM +0300, Sagi Grimberg wrote: > Hey Dan, > > >Hello Sagi Grimberg, > > > >The patch a73c2a2f9123: "libiscsi: Use scsi helper to set information > >descriptor" from Jul 15, 2015, leads to the following static checker > >warning: > > > > drivers/scsi/libiscsi.c:858 iscsi_scsi_cmd_rsp() > > error: XXX uninitialized symbol 'sector'. > > > >drivers/scsi/libiscsi.c > > 850 ascq = session->tt->check_protection(task, §or); > > > >If "ascq" is 0x1 then there sector might not be initialized. The > >documentation is not clear on how that works. Har dee har har. The > >oldest jokes are still the best... :P > > iscsi transports that implement this callout are expected > to set the sector which is passed by reference. There is only iscsi_iser_check_protection() I think. It behaves how I described in my original email. If ib_check_mr_status() fails it returns without setting sector. > > would it make the checker happy if we set sector to 0 before > calling check_protection (although it's not needed by no means)? It looks for if there is any possible way that it could be uninitialized so that would solve the problem. regards, dan carpenter -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
