On Wednesday, October 12, 2011 12:55:49 AM Dan Carpenter wrote:
> On Tue, Oct 11, 2011 at 05:20:11PM -0400, Paul Moore wrote:
> > > - if (doi_def == NULL || doi_def->doi == CIPSO_V4_DOI_UNKNOWN)
> > > + if (doi_def->doi == CIPSO_V4_DOI_UNKNOWN)
> > > goto doi_add_return;
> > > for (iter = 0; iter < CIPSO_V4_TAG_MAXCNT; iter++) {
> > > switch (doi_def->tags[iter]) {
> >
> > I'd prefer to keep the NULL check in there as it does afford a little
> > bit of extra safety and this is management code after all, not
> > per-packet processing code, so the extra check should have no
> > observable performance impact.
>
> The dereferences on the lines before mean we would Oops before
> reaching the check ...
Thanks for pointing that out, I missed that when looking at your patch.
--
paul moore
www.paul-moore.com
--
To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
