Small bug in 1.00.05 upgrade script?

[omega]

Hi,

while I was checking the 1.00.05 upgrade script, I've found one update step the script is doing which was not taking place on my system.

The code is this:

##Change interface config on smb.conf
smb_check=`cat /app/etc/samba/smb.conf|grep 'interfaces = eth\*,wlan\*,lo,bond\*'|wc -l`
if [ $smb_check -eq 0 ]; then
echo "upgrade interface options in smb.conf" >> debug
cp /app/etc/samba/smb.conf /app/etc/samba/smb.conf.default /tmp/upgrade
sed -r "s/interfaces = eth\*,wlan\*,lo/interfaces = eth*,wlan*,lo,bond*/" /tmp/upgrade/smb.conf > /tmp/smb.conf.p1 > /app/etc/samba/smb.conf
sed -r "s/interfaces = eth\*,wlan\*,lo/interfaces = eth*,wlan*,lo,bond*/" /tmp/upgrade/smb.conf.default > /tmp/smb.conf.p1 > /app/etc/samba/smb.conf.default
rm -rf /tmp/smb.conf.p1
fi


In other words, the interface "bond*" is added to the "interface = " statement in smb.conf.

Can somebody still running a firmware prior to 1.00.05 post the line conatining the "interface =" statement?

I don't know what the bond interface is, I think it is not needed on the N5200 and I have no idea if this bug is really serious (indeed I don't think so).
But I've changed my smb.conf manually and added the bond* interface and I wanted to let you know about this issue.

Andreas

[jamis]

Typically, bond interfaces are used to 'trunk' or bind the network interfaces together virtually into one channel. Basically so that multiple interfaces appear as one for more aggregate throughput.