[Devel] [PATCH RHEL7] kmod/whitelist: rename {,fs-}binfmt_misc

Kirill Tkhai ktkhai at virtuozzo.com
Mon May 23 08:29:51 PDT 2016


On 23.05.2016 18:25, Dmitry Safonov wrote:
> On 05/23/2016 06:25 PM, Kirill Tkhai wrote:
>> On 23.05.2016 18:15, Dmitry Safonov wrote:
>>> Somehow I managed to miss it in the first time.
>>>
>>> Fixes: commit 8b9a6bef982e ("ve/kmod: list of allowed to autoload in CT
>>> modules")
>>>
>>> https://jira.sw.ru/browse/PSBM-41216
>>>
>>> Cc: Kirill Tkhai <ktkhai at virtuozzo.com>
>>> Signed-off-by: Dmitry Safonov <dsafonov at virtuozzo.com>
>>> ---
>>>  kernel/kmod.c | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>> diff --git a/kernel/kmod.c b/kernel/kmod.c
>>> index 14879c5dd816..4e53fefab960 100644
>>> --- a/kernel/kmod.c
>>> +++ b/kernel/kmod.c
>>> @@ -377,7 +377,7 @@ static inline int module_payload_iptable_allowed(const char *module)
>>>
>>>  /* ve0 allowed modules */
>>>  static const char * const ve0_allowed_mod[] = {
>>> -    "binfmt_misc",
>>> +    "fs-binfmt_misc",
>>
>> Where it is loaded from? In search_binary_handler() I see "binfmt_misc" without prefix.
> 
> Yeah.
> 
> get_fs_type:
> if (!fs && (request_module("fs-%.*s", len, name) == 0))
> 
> Adds 'fs-' prefix on mount.

Shouldn't we keep them both?


More information about the Devel mailing list