Re: [PATCH 3/3] tools/libxenstat: fix output initialisation in parseNetDevLine()

[Baptiste Le Duc <baptiste.le-duc@xxxxxxxxxx>]

> parseNetDevLine() zeroes all the outputs it is passed before parsing the
> line, but *rxPackets is zeroed twice while *rxMcast isn't zeroed at all,
> so it would be left untouched for a line that doesn't parse.
> 
> Also, iface is cleared with memset(iface, 0, sizeof(*iface)), which only
> covers the first byte. The function isn't told the buffer size, and an
> empty string is all that's needed, so write the terminator directly.
> 
> No functional change intended: the only caller passes NULL for rxMcast.
> 
> Assisted-by: Claude-Code:claude-opus-5-5
> Signed-off-by: Olivier Lambert <olivier.lambert@xxxxxxxxxx>
> ---
The subject says "fix", but the body says no functional change. Either
it's a bug fix and wants a Fixes: tag, or it isn't, and then I'd use
Amends: 1fd93d4f8cb8 ("xenstat: Linux dom0 statistics for case we use
network bonding") and drop "fix" from the subject. Based on the changes, I 
think the latter.

-- 
Baptiste Le Duc <baptiste.le-duc@xxxxxxxxxx>