I have same question here. I has been 3 months after the patch submitted.

I got bitten by this bug today and my client's Internet was totally down
until I have time to log into the system and finally found after reboot,
the ip_forward is not set.

I have to create a file under /etc/sysctl.d to workaround this bug.

One thing I do not fully understand is today's latest git repos (26 May),
the snippet of sysctl.c is as follows:

    if (stat(DEFAULT_PRELOAD, &ts) < 0 || S_ISREG(ts.st_mode)) {
        if (!Quiet)
            printf(_("* Applying %s ...\n"), DEFAULT_PRELOAD);
        rc |= Preload(DEFAULT_PRELOAD);
    }

My limited C knowledge telling me is that if stat /etc/sysctl.conf returns
an error or /etc/sysctl.conf is a regular file, it will try to Preload the
/etc/sysctl.conf still. I couldn't understand the point of checking here.
Shouldn't be stat()==0 && S_ISREG() then we do the preloading?

Best Regards,
Bai Shi

Reply via email to