Messages in this thread | | | Subject | Re: [patch] f2fs: use _safe() version of list_for_each | From | Jaegeuk Kim <> | Date | Mon, 21 Jan 2013 18:20:01 +0900 |
| |
2013-01-21 (월), 00:32 -0800, Dmitry Torokhov: > On Sun, Jan 20, 2013 at 06:02:58PM +0300, Dan Carpenter wrote: > > This is calling list_del() inside a loop which is a problem when we try > > move to the next item on the list. I've converted it to use the _safe > > version. And also, as a cleanup, I've converted it to use > > list_for_each_entry instead of list_for_each. > > > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > --- > > Static analysis stuff. Untested. Please review carefully. > > Makes sense to me. > > Reviewed-by: Dmitry Torokhov <dmitry.torokhov@gmail.com> >
No doubt, applied. Thanks,
-- Jaegeuk Kim Samsung [unhandled content-type:application/pgp-signature] | |