Messages in this thread | | | Date | Thu, 06 Apr 2006 07:45:23 +0800 | From | Yi Yang <> | Subject | Re: [2.6.16 PATCH] Filessytem Events Reporter V2 |
| |
Carlos Silva wrote: > On Wed, 2006-04-05 at 21:21 +0800, Yi Yang wrote: > >> <snip> >> +static void cleanup_dead_listener(listener * x) >> +{ >> + pid_filter * p = NULL, * pq = NULL; >> + uid_filter * u = NULL, * uq = NULL; >> + gid_filter * g = NULL, * gq = NULL; >> + >> + if (p == NULL) >> + return; >> <snip> >> > > I think you ment "if (x == NULL)" here. Otherwise, the condition will always be true. > btw, I'm not reviewing your code, just stumbled across this while reading it. > Yes, it is my mistake, thank you. - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |