[SCSI] libfc: discovery gpn_ft parse bug
authorJoe Eykholt <jeykholt@cisco.com>
Tue, 25 Aug 2009 21:02:43 +0000 (14:02 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Thu, 10 Sep 2009 17:07:52 +0000 (12:07 -0500)
In fc_disc_gpn_ft_parse(), after fc_disc_done() is called, the
disc state is changed by setting buf_len = 0.  This is wrong
since the discovery may have restarted.   Instead, return
after calling fc_disc_done.

Also, return an error on memory allocation failure.

Signed-off-by: Joe Eykholt <jeykholt@cisco.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/libfc/fc_disc.c

index 3efdbbab9b2bbae967fd80db5004d615db286021..a2410dc744414bd4a83f6445e28f693e1ca0a166 100644 (file)
@@ -479,6 +479,7 @@ static int fc_disc_gpn_ft_parse(struct fc_disc *disc, void *buf, size_t len)
        plen = len;
        np = (struct fc_gpn_ft_resp *)bp;
        tlen = disc->buf_len;
+       disc->buf_len = 0;
        if (tlen) {
                WARN_ON(tlen >= sizeof(*np));
                plen = sizeof(*np) - tlen;
@@ -519,10 +520,12 @@ static int fc_disc_gpn_ft_parse(struct fc_disc *disc, void *buf, size_t len)
                        rdata = lport->tt.rport_create(lport, &ids);
                        if (rdata)
                                rdata->disc_id = disc->disc_id;
-                       else
+                       else {
                                printk(KERN_WARNING "libfc: Failed to allocate "
                                       "memory for the newly discovered port "
                                       "(%6x)\n", ids.port_id);
+                               error = -ENOMEM;
+                       }
                }
 
                if (np->fp_flags & FC_NS_FID_LAST) {
@@ -546,8 +549,6 @@ static int fc_disc_gpn_ft_parse(struct fc_disc *disc, void *buf, size_t len)
                        memcpy(&disc->partial_buf, np, len);
                }
                disc->buf_len = (unsigned char) len;
-       } else {
-               disc->buf_len = 0;
        }
        return error;
 }