MFC r305389:
authordim <dim@FreeBSD.org>
Wed, 7 Sep 2016 19:25:08 +0000 (19:25 +0000)
committerdim <dim@FreeBSD.org>
Wed, 7 Sep 2016 19:25:08 +0000 (19:25 +0000)
commitbe91514db3ffa8ef84f912275d8c612f4a1a8407
treeb71eec3c5938e28b090b740a31ad53c14a28f94d
parent83216cdbb72b65e649ebf7582ea72cb089229b41
MFC r305389:

With clang 3.9.0, compiling sys/netinet/igmp.c results in the following
warning:

sys/netinet/igmp.c:546:21: error: implicit conversion from 'int' to 'char' changes value from 148 to -108 [-Werror,-Wconstant-conversion]
        p->ipopt_list[0] = IPOPT_RA;    /* Router Alert Option */
                         ~ ^~~~~~~~
sys/netinet/ip.h:153:19: note: expanded from macro 'IPOPT_RA'
#define IPOPT_RA                148             /* router alert */
                                ^~~

This is because ipopt_list is an array of char, so IPOPT_RA is wrapped
to a negative value.  It would be nice to change ipopt_list to an array
of u_char, but it changes the signature of the public struct ipoption,
so add an explicit cast to suppress the warning.

Reviewed by: imp
Differential Revision: https://reviews.freebsd.org/D7777
sys/netinet/igmp.c