On 19 April 2015 at 02:17, Sergey Kandaurov <[email protected]> wrote: > On 19 April 2015 at 12:07, Eitan Adler <[email protected]> wrote: >> Author: eadler >> Date: Sun Apr 19 09:07:57 2015 >> New Revision: 281734 >> URL: https://svnweb.freebsd.org/changeset/base/281734 >> >> Log: >> identd: remove redundant zeroing >> se_rpc_lowvers was set to 0 twice, so remove one of them >> I can not find any other variable which they may have been a typo of. >> >> Reported by: gcc5.1 >> >> Modified: >> head/usr.sbin/inetd/inetd.c >> >> Modified: head/usr.sbin/inetd/inetd.c >> ============================================================================== >> --- head/usr.sbin/inetd/inetd.c Sun Apr 19 08:45:37 2015 (r281733) >> +++ head/usr.sbin/inetd/inetd.c Sun Apr 19 09:07:57 2015 (r281734) >> @@ -1752,8 +1752,7 @@ more: >> memmove(sep->se_proto, sep->se_proto + 4, >> strlen(sep->se_proto) + 1 - 4); >> sep->se_rpc = 1; >> - sep->se_rpc_prog = sep->se_rpc_lowvers = >> - sep->se_rpc_lowvers = 0; >> + sep->se_rpc_prog = sep->se_rpc_lowvers = 0; >> memcpy(&sep->se_ctrladdr4, bind_sa4, >> sizeof(sep->se_ctrladdr4)); >> if ((versp = strrchr(sep->se_service, '/'))) { >> > > se_rpc_highvers ? > They both seem to be used prezeroed in a later sscanf(). hmmm, likely. I was looking for a different variable, not a different member :\
I'll fix -- Eitan Adler Source, Ports, Doc committer Bugmeister, Ports Security teams _______________________________________________ [email protected] mailing list http://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "[email protected]"
