Re: [PATCH 2/3] tools/libxenstat: don't skip the /proc/net/dev check after a failure

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

On Fri, 16 Oct 2026 09:12:58 +0000, Olivier Lambert 
<olivier.lambert@xxxxxxxxxx> wrote:
> xenstat_collect_networks() opens /proc/net/dev once and validates its
> header before using it. If reading the header fails, or the header isn't
> the expected one, an error is returned but the stream is left open in
> priv->procnetdev. The next call then finds the stream already open,
> skips the validation and parses the file anyway.
> 
> Close the stream and clear the pointer on both error paths, so that the
> validation is done again on the next call.
> 
> [...]

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

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