lkml.org 
[lkml]   [2019]   [Feb]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] pstore: Avoid to write records with null size
On Thu, Jan 31, 2019 at 6:26 PM Yue Hu <huyue2@yulong.com> wrote:
>
> From 57cb7141b227b7f75a6e4afb8319f803510cc07f Mon Sep 17 00:00:00 2001
> From: Yue Hu <huyue2@yulong.com>
> Date: Thu, 31 Jan 2019 18:12:46 +0800
> Subject: [PATCH] pstore: Avoid to write records with null size
>
> Sometimes pstore_console_write() will write record with null size
> to persistent ram zone, that is unnecessary. It will only increase
> resource consumtion. ramoops_write_kmsg_hdr() has same logic if
> null hdr happened. So let's avoid those cases.
>
> Signed-off-by: Yue Hu <huyue2@yulong.com>
> ---
> fs/pstore/platform.c | 3 +++
> fs/pstore/ram.c | 6 ++++--
> 2 files changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/fs/pstore/platform.c b/fs/pstore/platform.c
> index 7486e48..5e9b817 100644
> --- a/fs/pstore/platform.c
> +++ b/fs/pstore/platform.c
> @@ -504,6 +504,9 @@ static void pstore_console_write(struct console *con, const char *s, unsigned c)
> {
> struct pstore_record record;
>
> + if (!c)
> + return;
> +
> pstore_record_init(&record, psinfo);
> record.type = PSTORE_TYPE_CONSOLE;
>
> diff --git a/fs/pstore/ram.c b/fs/pstore/ram.c
> index 0bb086b..7e188f6 100644
> --- a/fs/pstore/ram.c
> +++ b/fs/pstore/ram.c
> @@ -364,8 +364,10 @@ static size_t ramoops_write_kmsg_hdr(struct persistent_ram_zone *prz,
> (time64_t)record->time.tv_sec,
> record->time.tv_nsec / 1000,
> record->compressed ? 'C' : 'D');
> - WARN_ON_ONCE(!hdr);
> - len = hdr ? strlen(hdr) : 0;
> + if (WARN_ON_ONCE(!hdr))
> + return 0;
> +
> + len = strlen(hdr);
> persistent_ram_write(prz, hdr, len);
> kfree(hdr);

Applied, thanks!

--
Kees Cook

\
 
 \ /
  Last update: 2019-02-12 21:08    [W:0.033 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site