linux-kernel - Re: [PATCH] hugetlbfs: add err code in initializing module

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAJd=RBC=ajApxw_+tfOoY9KpTNW2Y+_B5MxDOjBH3Y5-PDW2jg@mail.gmail.com>
Date:	Thu, 22 Mar 2012 21:26:11 +0800
From:	Hillf Danton <dhillf@...il.com>
To:	Al Viro <viro@...iv.linux.org.uk>
Cc:	LKML <linux-kernel@...r.kernel.org>,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] hugetlbfs: add err code in initializing module

On Thu, Mar 22, 2012 at 8:33 AM, Al Viro <viro@...iv.linux.org.uk> wrote:
> On Sun, Mar 11, 2012 at 01:09:59PM +0800, Hillf Danton wrote:
>> Error code is added if fail to create inode kmem cache, and newly registered
>> hugetlb FS is unregistered if fail to mount, both for unlikely corner cases.
>>
>> --- a/fs/hugetlbfs/inode.c    Sun Mar 11 12:46:38 2012
>> +++ b/fs/hugetlbfs/inode.c    Sun Mar 11 12:49:28 2012
>> @@ -1000,6 +1000,7 @@ static int __init init_hugetlbfs_fs(void
>>       hugetlbfs_inode_cachep = kmem_cache_create("hugetlbfs_inode_cache",
>>                                       sizeof(struct hugetlbfs_inode_info),
>>                                       0, 0, init_once);
>> +     error = -ENOMEM;
>>       if (hugetlbfs_inode_cachep == NULL)
>>               goto out2;
>>
>> @@ -1015,6 +1016,7 @@ static int __init init_hugetlbfs_fs(void
>>       }
>>
>>       error = PTR_ERR(vfsmount);
>> +     unregister_filesystem(&hugetlbfs_fs_type);
>
> Bloody bad idea, that...  Realistically, the proper action on failure here
> (as well as in sock_init(), etc.) is panic().  If we fail to OOM that early,
> the box is doomed anyway.
>
> Note that unregister_filesystem() in module init is *always* wrong; it's not
> an issue here (it's done too early to care about and realistically the box
> is not going anywhere - it'll panic when attempt to exec /sbin/init fails,
> if not earlier), but it's a damn bad example.
>
Thanks for your review, Al.

Due to bad idea and bad example, please issue Nack, then the patch
that was not well prepared will be dropped.

-hd
--
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/