Patch "tools: ynl: don't ignore errors in NLMSG_DONE messages" has been added to the 6.8-stable tree

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is a note to let you know that I've just added the patch titled

    tools: ynl: don't ignore errors in NLMSG_DONE messages

to the 6.8-stable tree which can be found at:
    http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     tools-ynl-don-t-ignore-errors-in-nlmsg_done-messages.patch
and it can be found in the queue-6.8 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <stable@xxxxxxxxxxxxxxx> know about it.



commit d761736c76954c0c4751c4d7eb16231de9bac795
Author: Jakub Kicinski <kuba@xxxxxxxxxx>
Date:   Fri Apr 19 19:08:26 2024 -0700

    tools: ynl: don't ignore errors in NLMSG_DONE messages
    
    [ Upstream commit a44f2eb106a46f2275a79de54ce0ea63e4f3d8c8 ]
    
    NLMSG_DONE contains an error code, it has to be extracted.
    Prior to this change all dumps will end in success,
    and in case of failure the result is silently truncated.
    
    Fixes: e4b48ed460d3 ("tools: ynl: add a completely generic client")
    Signed-off-by: Jakub Kicinski <kuba@xxxxxxxxxx>
    Reviewed-by: Donald Hunter <donald.hunter@xxxxxxxxx>
    Link: https://lore.kernel.org/r/20240420020827.3288615-1-kuba@xxxxxxxxxx
    Signed-off-by: Paolo Abeni <pabeni@xxxxxxxxxx>
    Signed-off-by: Sasha Levin <sashal@xxxxxxxxxx>

diff --git a/tools/net/ynl/lib/ynl.py b/tools/net/ynl/lib/ynl.py
index 1e10512b2117c..571c2e218993f 100644
--- a/tools/net/ynl/lib/ynl.py
+++ b/tools/net/ynl/lib/ynl.py
@@ -208,6 +208,7 @@ class NlMsg:
             self.done = 1
             extack_off = 20
         elif self.nl_type == Netlink.NLMSG_DONE:
+            self.error = struct.unpack("i", self.raw[0:4])[0]
             self.done = 1
             extack_off = 4
 




[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux