linux-stable.git/drivers/net/fddi, branch v4.12

format-security: move static strings to const

2017-05-09T00:15:14+00:00

While examining output from trial builds with -Wformat-security enabled,
many strings were found that should be defined as "const", or as a char
array instead of char pointer.  This makes some static analysis easier,
by producing fewer false positives.

As these are all trivial changes, it seemed best to put them all in a
single patch rather than chopping them up per maintainer.

Link: http://lkml.kernel.org/r/20170405214711.GA5711@beast
Signed-off-by: Kees Cook 
Acked-by: Jes Sorensen 	[runner.c]
Cc: Tony Lindgren 
Cc: Russell King 
Cc: "Maciej W. Rozycki" 
Cc: Ralf Baechle 
Cc: Arnd Bergmann 
Cc: Greg Kroah-Hartman 
Cc: "Rafael J. Wysocki" 
Cc: Viresh Kumar 
Cc: Daniel Vetter 
Cc: Jani Nikula 
Cc: Sean Paul 
Cc: David Airlie 
Cc: Yisen Zhuang 
Cc: Salil Mehta 
Cc: Thomas Bogendoerfer 
Cc: Jiri Slaby 
Cc: Patrice Chotard 
Cc: "David S. Miller" 
Cc: James Hogan 
Cc: Paul Burton 
Cc: Matt Redfearn 
Cc: Paolo Bonzini 
Cc: Ingo Molnar 
Cc: Rasmus Villemoes 
Cc: Mugunthan V N 
Cc: Felipe Balbi 
Cc: Jarod Wilson 
Cc: Florian Westphal 
Cc: Antonio Quartulli 
Cc: Dmitry Torokhov 
Cc: Kejian Yan 
Cc: Daode Huang 
Cc: Qianqian Xie 
Cc: Philippe Reynes 
Cc: Colin Ian King 
Cc: Eric Dumazet 
Cc: Christian Gromm 
Cc: Andrey Shvetsov 
Cc: Jason Litzinger 
Cc: WANG Cong 
Signed-off-by: Andrew Morton 
Signed-off-by: Linus Torvalds

fddi: skfp: Use more common logging styles

2016-12-29T16:37:14+00:00

Several macros use non-standard styles where format and arguments
are not verified.  Convert these to a more typical fmt, ##__VA_ARGS__
use so format and arguments match as appropriate.

Miscellanea:

o Fix format and argument mismatches
o Realign and reindent misindented block
o Strip newlines from formats and add to macro defines
o Coalesce a few consecutive logging uses to more simple single uses

Signed-off-by: Joe Perches 
Signed-off-by: David S. Miller

skfp: hwmtm: Use proper logging macros, correct mismatches

2016-12-29T16:37:14+00:00

Logging macros should allow format and argument validation.
The DB_TX, DB_RX, and DB_GEN macros did not.

Update the macros and uses and add no_printk validation to the
previously compiled away #ifndef DEBUG variants.

Done with coccinelle and some typing.

Signed-off-by: Joe Perches 
Signed-off-by: David S. Miller

Replace with globally

2016-12-24T19:46:01+00:00

This was entirely automated, using the script by Al:

  PATT='^[[:blank:]]*#[[:blank:]]*include[[:blank:]]*'
  sed -i -e "s!$PATT!#include !" \
        $(git grep -l "$PATT"|grep -v ^include/linux/uaccess.h)

to do the replacement at the end of the merge window.

Requested-by: Al Viro 
Signed-off-by: Linus Torvalds

net: fddi: skfp: use %p format specifier for addresses rather than %x

2016-12-21T20:34:51+00:00

Trivial fix: Addresses should be printed using the %p format specifier
rather than using %x.

Signed-off-by: Colin Ian King 
Signed-off-by: David S. Miller

net: use core MTU range checking in misc drivers

2016-10-20T18:51:10+00:00

firewire-net:
- set min/max_mtu
- remove fwnet_change_mtu

nes:
- set max_mtu
- clean up nes_netdev_change_mtu

xpnet:
- set min/max_mtu
- remove xpnet_dev_change_mtu

hippi:
- set min/max_mtu
- remove hippi_change_mtu

batman-adv:
- set max_mtu
- remove batadv_interface_change_mtu
- initialization is a little async, not 100% certain that max_mtu is set
  in the optimal place, don't have hardware to test with

rionet:
- set min/max_mtu
- remove rionet_change_mtu

slip:
- set min/max_mtu
- streamline sl_change_mtu

um/net_kern:
- remove pointless ndo_change_mtu

hsi/clients/ssi_protocol:
- use core MTU range checking
- remove now redundant ssip_pn_set_mtu

ipoib:
- set a default max MTU value
- Note: ipoib's actual max MTU can vary, depending on if the device is in
  connected mode or not, so we'll just set the max_mtu value to the max
  possible, and let the ndo_change_mtu function continue to validate any new
  MTU change requests with checks for CM or not. Note that ipoib has no
  min_mtu set, and thus, the network core's mtu > 0 check is the only lower
  bounds here.

mptlan:
- use net core MTU range checking
- remove now redundant mpt_lan_change_mtu

fddi:
- min_mtu = 21, max_mtu = 4470
- remove now redundant fddi_change_mtu (including export)

fjes:
- min_mtu = 8192, max_mtu = 65536
- The max_mtu value is actually one over IP_MAX_MTU here, but the idea is to
  get past the core net MTU range checks so fjes_change_mtu can validate a
  new MTU against what it supports (see fjes_support_mtu in fjes_hw.c)

hsr:
- min_mtu = 0 (calls ether_setup, max_mtu is 1500)

f_phonet:
- min_mtu = 6, max_mtu = 65541

u_ether:
- min_mtu = 14, max_mtu = 15412

phonet/pep-gprs:
- min_mtu = 576, max_mtu = 65530
- remove redundant gprs_set_mtu

CC: netdev@vger.kernel.org
CC: linux-rdma@vger.kernel.org
CC: Stefan Richter 
CC: Faisal Latif 
CC: linux-rdma@vger.kernel.org
CC: Cliff Whickman 
CC: Robin Holt 
CC: Jes Sorensen 
CC: Marek Lindner 
CC: Simon Wunderlich 
CC: Antonio Quartulli 
CC: Sathya Prakash 
CC: Chaitra P B 
CC: Suganath Prabu Subramani 
CC: MPT-FusionLinux.pdl@broadcom.com
CC: Sebastian Reichel 
CC: Felipe Balbi 
CC: Arvid Brodin 
CC: Remi Denis-Courmont 
Signed-off-by: Jarod Wilson 
Signed-off-by: David S. Miller

net: skfb: remove obsolete -I cflag

2016-06-16T05:06:06+00:00

The skfp driver has been moved to drivers/net/fddi/skfp a long time
ago, but we still attempt to include headers from the old location,
which causes a warning when building with W=1:

cc1: error: /git/arm-soc/drivers/net/skfp: No such file or directory [-Werror=missing-include-dirs]
cc1: error: drivers/net/skfp: No such file or directory [-Werror=missing-include-dirs]

Clearly this include directive is not needed any more, so we can
just remove it now.

Signed-off-by: Arnd Bergmann 
Signed-off-by: David S. Miller

defxx: fix build warning

2016-01-25T18:51:52+00:00

We are getting many build warnings about:
'bar_start' may be used uninitialized
and
'bar_len' may be used uninitialized

They are not actually uninitialized as dfx_get_bars() will initialize
them properly. But still lets have them initialized just to satisfy the
compiler (gcc 4.8.2).

Signed-off-by: Sudip Mukherjee 
Acked-by: Maciej W. Rozycki 
Signed-off-by: David S. Miller

net/fddi: remove HWM_REVERSE() macro

2015-08-14T04:12:17+00:00

HWM_REVERSE() macro is unused, remove it.

Signed-off-by: yalin wang 
Signed-off-by: David S. Miller

fddi: print an address with %p format specifier rather than %x

2015-06-07T07:31:22+00:00

The debug is printing the struct smt_header * address using
the %x format specifier. Fix it to use %p instead.

Signed-off-by: Colin Ian King 
Signed-off-by: David S. Miller