ShellCheck - shell script analysis tool
version: 0.10.0
license: GNU General Public License, version 3
website: https://www.shellcheck.net
Redirect to /tmp/tmp.GX8Fgeh3e0 and /tmp/tmp.RkLK49EGcF
Tree base:
2bbb932cbe72 ("netconsole: resume previously deactivated target")
Now at:
dec31e493873 ("selftests: netconsole: validate target resume")
 ====== Checking before the patch ======
Checking tools/testing/selftests/drivers/net/lib/sh/lib_netcons.sh - bf920798bfe9ad648c492dab7e7cb69c49e9d7fc40353062d15ff0dc93f5036d


In lib_netcons.sh line 66:
	local SRCIF_IFIDX=$(cat /sys/class/net/"$SRCIF"/ifindex)
              ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values.


In lib_netcons.sh line 67:
	local DSTIF_IFIDX=$(cat /sys/class/net/"$DSTIF"/ifindex)
              ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values.

For more information:
  https://www.shellcheck.net/wiki/SC2155 -- Declare and assign separately to ...

 ====== Checking the tree with the patch ======
Checking tools/testing/selftests/drivers/net/lib/sh/lib_netcons.sh - bf920798bfe9ad648c492dab7e7cb69c49e9d7fc40353062d15ff0dc93f5036d


In lib_netcons.sh line 66:
	local SRCIF_IFIDX=$(cat /sys/class/net/"$SRCIF"/ifindex)
              ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values.


In lib_netcons.sh line 67:
	local DSTIF_IFIDX=$(cat /sys/class/net/"$DSTIF"/ifindex)
              ^---------^ SC2155 (warning): Declare and assign separately to avoid masking return values.


In lib_netcons.sh line 203:
function cleanup_netcons() {
^-- SC2120 (warning): cleanup_netcons references arguments, but none are ever passed.


In lib_netcons.sh line 224:
	cleanup_netcons
        ^-------------^ SC2119 (info): Use cleanup_netcons "$@" if function's $1 should mean script's $1.


In lib_netcons.sh line 440:
	cleanup_netcons
        ^-------------^ SC2119 (info): Use cleanup_netcons "$@" if function's $1 should mean script's $1.

For more information:
  https://www.shellcheck.net/wiki/SC2120 -- cleanup_netcons references argume...
  https://www.shellcheck.net/wiki/SC2155 -- Declare and assign separately to ...
  https://www.shellcheck.net/wiki/SC2119 -- Use cleanup_netcons "$@" if funct...

Checking tools/testing/selftests/drivers/net/netcons_resume.sh - 0d892e98b523ab3cd61407b6550c98c1da8d1a82e6f1bab1e03a008e0a815570


In netcons_resume.sh line 46:
	set_network
        ^---------^ SC2119 (info): Use set_network "$@" if function's $1 should mean script's $1.


In netcons_resume.sh line 71:
	set_network
        ^---------^ SC2119 (info): Use set_network "$@" if function's $1 should mean script's $1.


In netcons_resume.sh line 97:
exit "${ksft_pass}"
      ^----------^ SC2154 (warning): ksft_pass is referenced but not assigned.

For more information:
  https://www.shellcheck.net/wiki/SC2154 -- ksft_pass is referenced but not a...
  https://www.shellcheck.net/wiki/SC2119 -- Use set_network "$@" if function'...