The ifplugd init script in /etc/init.d/ifplugd uses backwards (wrong!) logic in determining whether IGNORE_FAIL_POSITIVE==yes should result in passing the -F option to ifplugd. The -F option results in ignoring failures (and treating them as success). However, the current script essentially says (pseudocode): IGNORE_FAIL_POSITIVE || args += "-F" which can also be stated as: if (!IGNORE_FAIL_POSITIVE) args += "-F" It should say IGNORE_FAIL_POSITIVE && args += "-F" which can also be stated as: if (IGNORE_FAIL_POSITIVE) args += "-F" All that needs to be done is to change the "||" to an "&&" on the line that tests IGNORE_FAIL_POSITIVE in /etc/init.d/ifplugd. Reproducible: Always Steps to Reproduce: 1. Set IGNORE_FAIL_POSITIVE="no" in /etc/conf.d/ifplugd 2. Start ifplug: run /etc/init.d/ifplugd start 3. Examine the options that ifplugd has been started with: ps axuw | grep ifplugd Actual Results: The output of 'ps' shows the '-F' option being passed to ifplugd, which should not be the case. Expected Results: The ifplugd program should NOT be executed with the '-F' command line option.
Created attachment 124672 [details, diff] Patch to fix the /etc/init.d/ifplugd script, changing the OR to AND
ifplugd doesn't ship with an init script anymore as baselayout now controls it directly.
