On 20 June 2017 at 12:44, Alin Serdean <[email protected]>
wrote:

> x->errline is a signed int
> (https://github.com/openvswitch/ovs/blob/934386dd129a3f15b2546562a775f2
> f43dd3aac6/tests/test-sflow.c#L88)
> Flagged by static code analysis.
>
> Signed-off-by: Alin Gabriel Serdean <[email protected]>
>
Applied, thanks!


> ---
>  tests/test-sflow.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tests/test-sflow.c b/tests/test-sflow.c
> index 6125d38..039a1b1 100644
> --- a/tests/test-sflow.c
> +++ b/tests/test-sflow.c
> @@ -675,7 +675,7 @@ print_sflow(struct ofpbuf *buf)
>          process_datagram(x);
>      } else {
>          // CATCH
> -        printf("\n>>>>> ERROR in " __FILE__ " at line %u\n", x->errline);
> +        printf("\n>>>>> ERROR in " __FILE__ " at line %d\n", x->errline);
>      }
>  }
>
> --
> 2.10.2.windows.1
> _______________________________________________
> dev mailing list
> [email protected]
> https://mail.openvswitch.org/mailman/listinfo/ovs-dev
>
_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to