Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 3 May 2017 13:39:03 -0400
From:      Ryan Stone <rysto32@gmail.com>
To:        Alan Somers <asomers@freebsd.org>
Cc:        "src-committers@freebsd.org" <src-committers@freebsd.org>,  "svn-src-all@freebsd.org" <svn-src-all@freebsd.org>,  "svn-src-head@freebsd.org" <svn-src-head@freebsd.org>
Subject:   Re: svn commit: r317755 - head/sbin/ifconfig
Message-ID:  <CAFMmRNyHc7qgeidDTZss%2Bo=56Uc-Xgyu_voj4ybHt=mEJa6c%2Bg@mail.gmail.com>
In-Reply-To: <201705031721.v43HL2vS071819@repo.freebsd.org>
References:  <201705031721.v43HL2vS071819@repo.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Wed, May 3, 2017 at 1:21 PM, Alan Somers <asomers@freebsd.org> wrote:

> Author: asomers
> Date: Wed May  3 17:21:01 2017
> New Revision: 317755
> URL: https://svnweb.freebsd.org/changeset/base/317755
>
> Log:
>   Various Coverity fixes in ifconfig(8)
>
>   * Exit early if kldload(2) fails (1011259). This is the only change that
>     affects ifconfig's behavior.
>
>

Please revert this ASAP.  kldload is expected to fail for a number of
benign reasons and this change is likely to prevent any network
configuration from being applied to systems, breaking remote access.



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAFMmRNyHc7qgeidDTZss%2Bo=56Uc-Xgyu_voj4ybHt=mEJa6c%2Bg>