svn commit: r213793 - in head/sys/dev: ce cp
John Baldwin
jhb at freebsd.org
Wed Oct 13 20:03:12 UTC 2010
On Wednesday, October 13, 2010 1:59:42 pm Jung-uk Kim wrote:
> On Wednesday 13 October 2010 01:27 pm, Roman Divacky wrote:
> > On Wed, Oct 13, 2010 at 05:17:50PM +0000, Rui Paulo wrote:
> > > Author: rpaulo
> > > Date: Wed Oct 13 17:17:50 2010
> > > New Revision: 213793
> > > URL: http://svn.freebsd.org/changeset/base/213793
> > >
> > > Log:
> > > Don't do a logical AND of the result of strcmp() with a
> > > constant.
> > >
> > > Found with: clang
> > >
> > > Modified:
> > > head/sys/dev/ce/if_ce.c
> > > head/sys/dev/cp/if_cp.c
> > >
> > > Modified: head/sys/dev/ce/if_ce.c
> > > =================================================================
> > >============= --- head/sys/dev/ce/if_ce.c Wed Oct 13 17:16:08
> > > 2010 (r213792) +++ head/sys/dev/ce/if_ce.c Wed Oct 13 17:17:50
> > > 2010 (r213793) @@ -1313,7 +1313,7 @@ static int ce_ioctl (struct
> > > cdev *dev, u IFP2SP(d->ifp)->pp_flags &= ~(PP_FR);
> > > IFP2SP(d->ifp)->pp_flags |= PP_KEEPALIVE;
> > > d->ifp->if_flags |= PP_CISCO;
> > > - } else if (! strcmp ("fr", (char*)data) && PP_FR) {
> > > + } else if (! strcmp ("fr", (char*)data)) {
> >
> > this is wrong I think... the PP_FR was used for compiling in/out
> > support for something.. see the comment:
> >
> > /* If we don't have Cronyx's sppp version, we don't have fr support
> > via sppp */ #ifndef PP_FR
> > #define PP_FR 0
> > #endif
> >
> > note that PP_FR is used in some other places as a flag. I guess
> > that by compiling with something like make -DPP_FR=42 some magic
> > would happen.
> >
> > anyway - this does not look like a bug but like an intent, please
> > revert.
>
> I think the attached patch should do.
This is certainly more obvious and readable.
--
John Baldwin
More information about the svn-src-all
mailing list