Re: [PATCH] error: Strip trailing '\n' from an error string argument

[Peter Maydell]

On Wed, 12 Mar 2025 at 14:35, Markus Armbruster <arm...@redhat.com> wrote:
>
> Tracked down with scripts/coccinelle/err-bad-newline.cocci.
>
> Signed-off-by: Markus Armbruster <arm...@redhat.com>
> ---
>  net/vmnet-common.m | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/net/vmnet-common.m b/net/vmnet-common.m
> index 54d900ba67..ab33ce2b0c 100644
> --- a/net/vmnet-common.m
> +++ b/net/vmnet-common.m
> @@ -94,7 +94,7 @@ ssize_t vmnet_receive_common(NetClientState *nc,
>
>      if_status = vmnet_write(s->vmnet_if, &packet, &pkt_cnt);
>      if (if_status != VMNET_SUCCESS) {
> -        error_report("vmnet: write error: %s\n",
> +        error_report("vmnet: write error: %s",
>                       vmnet_status_map_str(if_status));
>          return -1;
>      }
> --

Reviewed-by: Peter Maydell <peter.mayd...@linaro.org>

thanks
-- PMM