Saída do seu script em ShellCheck . Corrija os problemas:
Line 4:
echo 'java -jar /opt/portmapper/portmapper.jar -list | grep $mylanip > /opt/portmapper/0prelims'
^-- SC2046: Quote this to prevent word splitting.
^-- SC2005: Useless echo? Instead of 'echo $(cmd)', just use 'cmd'.
^-- SC2006: Use $(..) instead of legacy '..'.
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 6:
portprelim=$(cat /opt/portmapper/0prelims | grep ${mylanip}:443 | awk '{print $4}')
^-- SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 7:
if [ "$portprelim" = $mylanip:443 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 10:
if [ "$portprelim" != $mylanip:443 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 12:
java -jar /opt/portmapper/portmapper.jar -lib org.chris.portmapper.router.sbbi.SBBIRouterFactory -add -protocol TCP -internalPort 443 -externalPort 443 -ip $mylanip -description HTTPS >/dev/null ; sleep 2 ;
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 16:
portprelim=$(cat /opt/portmapper/0prelims | grep ${mylanip}:57 | awk '{print $4}')
^-- SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 17:
if [ "$portprelim" = $mylanip:57 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 20:
if [ "$portprelim" != $mylanip:57 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 22:
java -jar /opt/portmapper/portmapper.jar -lib org.chris.portmapper.router.sbbi.SBBIRouterFactory -add -protocol TCP -internalPort 57 -externalPort 57 -ip $mylanip -description PrivateTTY >/dev/null ; sleep 2 ;
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 26:
portprelim=$(cat /opt/portmapper/0prelims | grep ${mylanip}:80 | awk '{print $4}')
^-- SC2002: Useless cat. Consider 'cmd < file | ..' or 'cmd file | ..' instead.
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 28:
if [ "$portprelim" = $mylanip:80 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 31:
if [ "$portprelim" != $mylanip:80 ]; then
^-- SC2086: Double quote to prevent globbing and word splitting.
Line 33:
java -jar /opt/portmapper/portmapper.jar -lib org.chris.portmapper.router.sbbi.SBBIRouterFactory -add -protocol TCP -internalPort 80 -externalPort 80 -ip $mylanip -description HTTP >/dev/null ; sleep 2 ;
^-- SC2086: Double quote to prevent globbing and word splitting.