On Tue, 2012-08-21 at 15:54 -0500, Nathan Zimmer wrote: > + if (pdeo) { > + kfree(pdeo); > + } Just make it unconditional. kfree(NULL) is a no-op. -- dwmw2