Ack.

Note. I do not think the patch about error reporting is to be blamed. We should
say thank you, it opened up an old bug, which used to be ignored. Also it opened
up that modern gcc creates completely crooked undebuggable code. :-)

In this case the patch conceals actual bug, but I believe it is OK.
If we will continue to fall here we will not get any more useful information.


On Fri, Aug 11, 2023 at 7:11 PM Yuriy Vasilev
<[email protected]> wrote:
>
> Pass addr with PCS_ADDRTYPE_NONE to netaddr_cmp will lead to BUG. Let's
> skip these errors.
>
> Affects: #VSTOR-73400
>
> Signed-off-by: Yuriy Vasilev <[email protected]>
> ---
>  fs/fuse/kio/pcs/pcs_fuse_kdirect.c | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/fs/fuse/kio/pcs/pcs_fuse_kdirect.c 
> b/fs/fuse/kio/pcs/pcs_fuse_kdirect.c
> index 706da8e536c9..e122a6d1f6cb 100644
> --- a/fs/fuse/kio/pcs/pcs_fuse_kdirect.c
> +++ b/fs/fuse/kio/pcs/pcs_fuse_kdirect.c
> @@ -1386,6 +1386,9 @@ void fuse_rpc_error_account(struct fuse_error_metrics 
> *metrics,
>                 return;
>         }
>
> +       if (addr->type == PCS_ADDRTYPE_NONE)
> +               return;
> +
>         metric = NULL;
>         mutex_lock(&metrics->mutex);
>         list_for_each_entry(entry, &metrics->fuse_rpc_error_metric_list, 
> list) {
> --
> 2.34.1
>
> _______________________________________________
> Devel mailing list
> [email protected]
> https://lists.openvz.org/mailman/listinfo/devel

_______________________________________________
Devel mailing list
[email protected]
https://lists.openvz.org/mailman/listinfo/devel

Reply via email to