Re: [PATCH v2] audit: Send netlink ACK before setting connection in auditd_set

From: Chris Riches
Date: Wed Oct 18 2023 - 08:49:35 EST



On 18/10/2023 13:11, Rinat Gadelshin wrote:
-static void audit_receive(struct sk_buff *skb)
+static void audit_receive(struct sk_buff *skb)
  {
      struct nlmsghdr *nlh;
+    bool ack;
Maybe we should initialize 'ack' as 'true' here?
That doesn't feel particularly useful to me. In fact, I think it's actually clearer
uninitialised as a never-used initialisation could look like an actually-used default.
We're guaranteed to initialise before use.

- Chris