cvs commit: src/sys/nfsclient bootp_subr.c
Christian S.J. Peron
csjp at FreeBSD.org
Thu Jun 29 23:07:49 UTC 2006
Yar Tikhiy wrote:
> yar 2006-06-29 15:37:39 UTC
>
> FreeBSD src repository
>
> Modified files:
> sys/nfsclient bootp_subr.c
> Log:
> Use the elegant TAILQ_FOREACH() in place of a hand-rolled for() loop.
>
> Revision Changes Path
> 1.65 +1 -3 src/sys/nfsclient/bootp_subr.c
>
>
>
Just as a point of curiosity, in some places, people are using for loops
to avoid invalid pointer dereferences in the event that one of the queue
items is removed by the code in the loop while iterating. Are you using
TAILQ_FOREACH_SAFE() in such places if required?
--
Christian S.J. Peron
csjp at FreeBSD.ORG
FreeBSD Committer
FreeBSD Security Team
More information about the cvs-src
mailing list