linux-kernel - Re: [PATCH] aoe: Add forgotten NULL at end of attribute list in aoeblk.c

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20061116161556.GA7222@coraid.com>
Date:	Thu, 16 Nov 2006 11:15:56 -0500
From:	"Ed L. Cashin" <ecashin@...aid.com>
To:	Dennis Stosberg <dennis@...sberg.net>
Cc:	Greg Kroah-Hartman <gregkh@...e.de>, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] aoe: Add forgotten NULL at end of attribute list in aoeblk.c

On Mon, Nov 13, 2006 at 09:15:20AM +0100, Dennis Stosberg wrote:
> This caused the system to stall when the aoe module was loaded.  The
> error was introduced in commit 4ca5224f3ea4779054d96e885ca9b3980801ce13

Boy, I've been totally spoiled by the gitweb at kernel.org.  It's been
unavailable lately.

Anyway, thanks for the fix.  It looks good to me if it looks good to
Greg---after a google search it appears that your patch is a followup
to his consolidation of the attributes.

> Signed-off-by: Dennis Stosberg <dennis@...sberg.net>
> ---
> 
> The log of the caused error can be found at
> http://stosberg.net/tmp/aoe_trace.txt
> 
>  drivers/block/aoe/aoeblk.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/block/aoe/aoeblk.c b/drivers/block/aoe/aoeblk.c
> index d433f27..aa25f8b 100644
> --- a/drivers/block/aoe/aoeblk.c
> +++ b/drivers/block/aoe/aoeblk.c
> @@ -68,6 +68,7 @@ static struct attribute *aoe_attrs[] = {
>  	&disk_attr_mac.attr,
>  	&disk_attr_netif.attr,
>  	&disk_attr_fwver.attr,
> +	NULL
>  };
>  
>  static const struct attribute_group attr_group = {
> -- 
> 1.4.3.3

-- 
  Ed L Cashin <ecashin@...aid.com>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/