Good fix, my mistake.

Acked-by: Joe Eykholt <[email protected]>

Bhanu Gollapudi wrote:
> Avoid infinite loop while processing FIP ELS or discovery
> advertisement with non-critical descriptors.
> 
> Signed-off-by: Bhanu Prakash Gollapudi <[email protected]>
> ---
>  drivers/scsi/fcoe/libfcoe.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/scsi/fcoe/libfcoe.c b/drivers/scsi/fcoe/libfcoe.c
> index 50aaa4b..9d08f4d 100644
> --- a/drivers/scsi/fcoe/libfcoe.c
> +++ b/drivers/scsi/fcoe/libfcoe.c
> @@ -703,7 +703,7 @@ static int fcoe_ctlr_parse_adv(struct fcoe_ctlr *fip,
>                       /* standard says ignore unknown descriptors >= 128 */
>                       if (desc->fip_dtype < FIP_DT_VENDOR_BASE)
>                               return -EINVAL;
> -                     continue;
> +                     break;
>               }
>               desc = (struct fip_desc *)((char *)desc + dlen);
>               rlen -= dlen;
> @@ -885,7 +885,7 @@ static void fcoe_ctlr_recv_els(struct fcoe_ctlr *fip, 
> struct sk_buff *skb)
>                       /* standard says ignore unknown descriptors >= 128 */
>                       if (desc->fip_dtype < FIP_DT_VENDOR_BASE)
>                               goto drop;
> -                     continue;
> +                     break;
>               }
>               desc = (struct fip_desc *)((char *)desc + dlen);
>               rlen -= dlen;

_______________________________________________
devel mailing list
[email protected]
http://www.open-fcoe.org/mailman/listinfo/devel

Reply via email to