I recently submitted this: http://marc.info/?l=linux-scsi&m=131228153521094&w=2 Patch to fix an oops in the cxgb3i iscsi driver. I did what I thought was some minor cleanup after testing and prior to submission. Of course that bit me, and and a result I introduced some build warnings that really don't need to be there. This patch corrects them. Apologies for the noise Signed-off-by: Neil Horman <nhorman@xxxxxxxxxxxxx> CC: Divy Le Ray <divy@xxxxxxxxxxx> CC: Steve Wise <swise@xxxxxxxxxxx> CC: Karen Xie <kxie@xxxxxxxxxxx> CC: Mike Christie <michaelc@xxxxxxxxxxx> CC: "James E.J. Bottomley" <JBottomley@xxxxxxxxxxxxx> --- drivers/scsi/cxgbi/libcxgbi.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/scsi/cxgbi/libcxgbi.c b/drivers/scsi/cxgbi/libcxgbi.c index eb5625d..0890728 100644 --- a/drivers/scsi/cxgbi/libcxgbi.c +++ b/drivers/scsi/cxgbi/libcxgbi.c @@ -475,7 +475,7 @@ static struct cxgbi_sock *cxgbi_check_route(struct sockaddr *dst_addr) struct sockaddr_in *daddr = (struct sockaddr_in *)dst_addr; struct dst_entry *dst; struct net_device *ndev; - struct cxgbi_device *cdev; + struct cxgbi_device *cdev = NULL; struct rtable *rt = NULL; struct flowi4 fl4; struct cxgbi_sock *csk = NULL; @@ -2499,7 +2499,7 @@ struct iscsi_endpoint *cxgbi_ep_connect(struct Scsi_Host *shost, return ep; release_conn: - cdev_put(&csk->cdev); + cdev_put(csk->cdev); cxgbi_sock_put(csk); cxgbi_sock_closed(csk); err_out: -- 1.7.6 -- 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