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]
Date:   Tue, 11 Apr 2023 13:40:18 +0000
From:   patchwork-bot+netdevbpf@...nel.org
To:     Jakub Kicinski <kuba@...nel.org>
Cc:     davem@...emloft.net, netdev@...r.kernel.org, edumazet@...gle.com,
        pabeni@...hat.com, kory.maincent@...tlin.com
Subject: Re: [PATCH net-next] tools: ynl: throw a more meaningful exception if
 family not supported

Hello:

This patch was applied to netdev/net-next.git (main)
by Paolo Abeni <pabeni@...hat.com>:

On Fri,  7 Apr 2023 07:56:09 -0700 you wrote:
> cli.py currently throws a pure KeyError if kernel doesn't support
> a netlink family. Users who did not write ynl (hah) may waste
> their time investigating what's wrong with the Python code.
> Improve the error message:
> 
> Traceback (most recent call last):
>   File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 362, in __init__
>     self.family = GenlFamily(self.yaml['name'])
>                   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
>   File "/home/kicinski/devel/linux/tools/net/ynl/lib/ynl.py", line 331, in __init__
>     self.genl_family = genl_family_name_to_id[family_name]
>                        ~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^^
> KeyError: 'netdev'
> 
> [...]

Here is the summary with links:
  - [net-next] tools: ynl: throw a more meaningful exception if family not supported
    https://git.kernel.org/netdev/net-next/c/ebe3bdc4359e

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html


Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ