Messages in this thread | ![/](/images/icornerl.gif) | | From | Segher Boessenkool <> | Subject | Re: [PATCH] include/linux/slab.h: new KFREE() macro. | Date | Mon, 1 Jan 2007 04:01:07 +0100 |
| |
> +#define KFREE(x) \ > + do { \ > + kfree(x); \ > + x = NULL; \ > + } while(0)
This doesn't work correctly if "x" has side effects -- double evaluation. Use a temporary variable instead, or better, an inline function. A better name wouldn't hurt either (kfree_and_zero()?)
Segher
- 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/
| ![\](/images/icornerr.gif) |