mirror of
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git
synced 2024-09-12 13:55:32 +00:00
infiniband: shut up a maybe-uninitialized warning
Some configurations produce this harmless warning when built with gcc -Wmaybe-uninitialized: infiniband/core/cma.c: In function 'cma_get_net_dev': infiniband/core/cma.c:1242:12: warning: 'src_addr_storage.sin_addr.s_addr' may be used uninitialized in this function [-Wmaybe-uninitialized] I previously reported this for the powerpc64 defconfig, but have now reproduced the same thing for x86 as well, using gcc-5 or higher. The code looks correct to me, and this change just rearranges it by making sure we alway initialize the entire address structure to make the warning disappear. My first approach added an initialization at the time of the declaration, which Doug commented may be too costly, so I hope this version doesn't add overhead. Link: http://arm-soc.lixom.net/buildlogs/mainline/v4.7-rc6/buildall.powerpc.ppc64_defconfig.log.passed Link: https://patchwork.kernel.org/patch/9212825/ Acked-by: Haggai Eran <haggaie@mellanox.com> Signed-off-by: Arnd Bergmann <arnd@arndb.de> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
This commit is contained in:
parent
beae2c9eb5
commit
c50e90d0d2
1 changed files with 28 additions and 26 deletions
|
@ -1094,47 +1094,47 @@ static void cma_save_ib_info(struct sockaddr *src_addr,
|
|||
}
|
||||
}
|
||||
|
||||
static void cma_save_ip4_info(struct sockaddr *src_addr,
|
||||
struct sockaddr *dst_addr,
|
||||
static void cma_save_ip4_info(struct sockaddr_in *src_addr,
|
||||
struct sockaddr_in *dst_addr,
|
||||
struct cma_hdr *hdr,
|
||||
__be16 local_port)
|
||||
{
|
||||
struct sockaddr_in *ip4;
|
||||
|
||||
if (src_addr) {
|
||||
ip4 = (struct sockaddr_in *)src_addr;
|
||||
ip4->sin_family = AF_INET;
|
||||
ip4->sin_addr.s_addr = hdr->dst_addr.ip4.addr;
|
||||
ip4->sin_port = local_port;
|
||||
*src_addr = (struct sockaddr_in) {
|
||||
.sin_family = AF_INET,
|
||||
.sin_addr.s_addr = hdr->dst_addr.ip4.addr,
|
||||
.sin_port = local_port,
|
||||
};
|
||||
}
|
||||
|
||||
if (dst_addr) {
|
||||
ip4 = (struct sockaddr_in *)dst_addr;
|
||||
ip4->sin_family = AF_INET;
|
||||
ip4->sin_addr.s_addr = hdr->src_addr.ip4.addr;
|
||||
ip4->sin_port = hdr->port;
|
||||
*dst_addr = (struct sockaddr_in) {
|
||||
.sin_family = AF_INET,
|
||||
.sin_addr.s_addr = hdr->src_addr.ip4.addr,
|
||||
.sin_port = hdr->port,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
static void cma_save_ip6_info(struct sockaddr *src_addr,
|
||||
struct sockaddr *dst_addr,
|
||||
static void cma_save_ip6_info(struct sockaddr_in6 *src_addr,
|
||||
struct sockaddr_in6 *dst_addr,
|
||||
struct cma_hdr *hdr,
|
||||
__be16 local_port)
|
||||
{
|
||||
struct sockaddr_in6 *ip6;
|
||||
|
||||
if (src_addr) {
|
||||
ip6 = (struct sockaddr_in6 *)src_addr;
|
||||
ip6->sin6_family = AF_INET6;
|
||||
ip6->sin6_addr = hdr->dst_addr.ip6;
|
||||
ip6->sin6_port = local_port;
|
||||
*src_addr = (struct sockaddr_in6) {
|
||||
.sin6_family = AF_INET6,
|
||||
.sin6_addr = hdr->dst_addr.ip6,
|
||||
.sin6_port = local_port,
|
||||
};
|
||||
}
|
||||
|
||||
if (dst_addr) {
|
||||
ip6 = (struct sockaddr_in6 *)dst_addr;
|
||||
ip6->sin6_family = AF_INET6;
|
||||
ip6->sin6_addr = hdr->src_addr.ip6;
|
||||
ip6->sin6_port = hdr->port;
|
||||
*dst_addr = (struct sockaddr_in6) {
|
||||
.sin6_family = AF_INET6,
|
||||
.sin6_addr = hdr->src_addr.ip6,
|
||||
.sin6_port = hdr->port,
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
|
@ -1159,10 +1159,12 @@ static int cma_save_ip_info(struct sockaddr *src_addr,
|
|||
|
||||
switch (cma_get_ip_ver(hdr)) {
|
||||
case 4:
|
||||
cma_save_ip4_info(src_addr, dst_addr, hdr, port);
|
||||
cma_save_ip4_info((struct sockaddr_in *)src_addr,
|
||||
(struct sockaddr_in *)dst_addr, hdr, port);
|
||||
break;
|
||||
case 6:
|
||||
cma_save_ip6_info(src_addr, dst_addr, hdr, port);
|
||||
cma_save_ip6_info((struct sockaddr_in6 *)src_addr,
|
||||
(struct sockaddr_in6 *)dst_addr, hdr, port);
|
||||
break;
|
||||
default:
|
||||
return -EAFNOSUPPORT;
|
||||
|
|
Loading…
Reference in a new issue