Re: [PATCH] pstore/ram: Fix possible NULL dereference

2012-07-20 Thread Kees Cook
On Thu, Jul 19, 2012 at 4:43 PM, Anton Vorontsov wrote: > We can dereference 'cxt->cprz' if console and dump logging are disabled > (which is unlikely, but still possible to do). This patch fixes the issue > by changing the code so that we don't dereference przs at all, we can > just calculate

Re: [PATCH] pstore/ram: Fix possible NULL dereference

2012-07-20 Thread Kees Cook
On Thu, Jul 19, 2012 at 4:43 PM, Anton Vorontsov anton.voront...@linaro.org wrote: We can dereference 'cxt-cprz' if console and dump logging are disabled (which is unlikely, but still possible to do). This patch fixes the issue by changing the code so that we don't dereference przs at all, we

[PATCH] pstore/ram: Fix possible NULL dereference

2012-07-19 Thread Anton Vorontsov
We can dereference 'cxt->cprz' if console and dump logging are disabled (which is unlikely, but still possible to do). This patch fixes the issue by changing the code so that we don't dereference przs at all, we can just calculate bufsize from console_size and record_size values. Plus, while at

[PATCH] pstore/ram: Fix possible NULL dereference

2012-07-19 Thread Anton Vorontsov
We can dereference 'cxt-cprz' if console and dump logging are disabled (which is unlikely, but still possible to do). This patch fixes the issue by changing the code so that we don't dereference przs at all, we can just calculate bufsize from console_size and record_size values. Plus, while at