On 8/30/05, Sridhar Samudrala <sri@xxxxxxxxxx> wrote: > On Tue, 2005-08-30 at 22:45 +0200, Jesper Juhl wrote: > > Since the patch to add a NULL short-circuit to crypto_free_tfm() went in, > > there's no longer any need for callers of that function to check for NULL. > > This patch removes the redundant NULL checks and also a few similar checks > > for NULL before calls to kfree() that I ran into while doing the > > crypto_free_tfm bits. > > > > I've posted similar patches in the past, but was asked to first until the > > short-circuit patch moved from -mm to mainline - and since it is now > > firmly there in 2.6.13 I assume there's no problem there anymore. > > I was also asked previously to make the patch against mainline and not -mm, > > so this patch is against 2.6.13. > > > > Feedback, ACK, NACK, etc welcome. > > sctp change looks fine. > A similar check in sctp_endpoint_destroy() can also be removed. > Thanks, I'll remember that and either update the patch or send a small incremental one later. -- Jesper Juhl <jesper.juhl@xxxxxxxxx> Don't top-post http://www.catb.org/~esr/jargon/html/T/top-post.html Plain text mails only, please http://www.expita.com/nomime.html - : send the line "unsubscribe linux-net" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
