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