]> git.proxmox.com Git - mirror_ubuntu-disco-kernel.git/commit
tcp: Always set options to 0 before calling tcp_established_options
authorAndi Kleen <ak@linux.intel.com>
Mon, 30 Sep 2013 20:29:11 +0000 (13:29 -0700)
committerDavid S. Miller <davem@davemloft.net>
Wed, 2 Oct 2013 20:32:43 +0000 (16:32 -0400)
commit5843ef421311c34f06d5ab82bced5aebfe185b2c
tree6c0900e7c8557e22ea382554cb3bd7c88e5be9d6
parent58e4e1f6cacddb7823c44bcfb272174553f6c645
tcp: Always set options to 0 before calling tcp_established_options

tcp_established_options assumes opts->options is 0 before calling,
as it read modify writes it.

For the tcp_current_mss() case the opts structure is not zeroed,
so this can be done with uninitialized values.

This is ok, because ->options is not read in this path.
But it's still better to avoid the operation on the uninitialized
field. This shuts up a static code analyzer, and presumably
may help the optimizer.

Cc: netdev@vger.kernel.org
Signed-off-by: Andi Kleen <ak@linux.intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv4/tcp_output.c