[PATCH] makedumpfile: code changes to satisfy the coverity scan
Atsushi Kumagai
ats-kumagai at wm.jp.nec.com
Tue Aug 25 17:42:03 PDT 2015
Hello Chao,
>Hi Atsushi Kumagai,
>
> Apologize for my mistake again. It is my first time to post patch,
>and I am very sorry to make the big mistake.
That's Okay, I hope it was a good lesson for you.
> I have updated and reposted the patch. If there are any problems,
>please tell me and I will change it immediately.
> This time, I have compiled it and test my patch. Hope it is good.
The v2 patch looks good to me, I'll merge it into v1.5.9.
Thanks,
Atsushi Kumagai
>Thanks,
>Chao Fan
>
>----- Original Message -----
>> From: "Chao Fan" <cfan at redhat.com>
>> To: "Atsushi Kumagai" <ats-kumagai at wm.jp.nec.com>
>> Sent: Tuesday, August 25, 2015 1:07:01 PM
>> Subject: Re: [PATCH] makedumpfile: code changes to satisfy the coverity scan
>>
>> Hi Atsushi Kumagai,
>>
>> Sorry for my mistake, I will update the patch.
>> Thank you for your reply.
>>
>> Thanks,
>> Chao Fan
>>
>> ----- Original Message -----
>> > From: "Atsushi Kumagai" <ats-kumagai at wm.jp.nec.com>
>> > To: "Chao Fan" <cfan at redhat.com>
>> > Cc: kexec at lists.infradead.org
>> > Sent: Tuesday, August 25, 2015 10:43:41 AM
>> > Subject: RE: [PATCH] makedumpfile: code changes to satisfy the coverity
>> > scan
>> >
>> > Hello Chao,
>> >
>> > >Coverity scan found some uncheked return value when calling
>> > >function. So the return value of the function set_page_size
>> > >and lseek should be checked to satisfy the coverity scan.
>> > >
>> > >In the function read_sadump_header, block_size is defined as an
>> > >unsigned long value, and it will never be less than zero, so this
>> > >comparison should be removed.
>> > >
>> > >Signed-off-by: Chao Fan <cfan at redhat.com>
>> > >---
>> > > makedumpfile.c | 3 ++-
>> > > makedumpfile.h | 6 +++++-
>> > > sadump_info.c | 3 ---
>> > > 3 files changed, 7 insertions(+), 5 deletions(-)
>> > >
>> > >diff --git a/makedumpfile.c b/makedumpfile.c
>> > >index 32f5459..53efc04 100644
>> > >--- a/makedumpfile.c
>> > >+++ b/makedumpfile.c
>> > >@@ -3213,7 +3213,8 @@ initial(void)
>> > > MSG("because the cyclic mode doesn't support sadump format.\n");
>> > > }
>> > >
>> > >- set_page_size(sadump_page_size());
>> > >+ if (!set_page_size(sadump_page_size()))
>> > >+ return FALSE;
>> > >
>> > > if (!sadump_initialize_bitmap_memory())
>> > > return FALSE;
>> > >diff --git a/makedumpfile.h b/makedumpfile.h
>> > >index d2fadbd..679e075 100644
>> > >--- a/makedumpfile.h
>> > >+++ b/makedumpfile.h
>> > >@@ -1745,7 +1745,11 @@ is_dumpable(struct dump_bitmap *bitmap, mdf_pfn_t
>> > >pfn)
>> > > off_t offset;
>> > > if (pfn == 0 || bitmap->no_block != pfn/PFN_BUFBITMAP) {
>> > > offset = bitmap->offset + BUFSIZE_BITMAP*(pfn/PFN_BUFBITMAP);
>> > >- lseek(bitmap->fd, offset, SEEK_SET);
>> > >+ if (lseek(bitmap->fd, old_offset, SEEK_SET) < 0 ) {
>> >
>> > s/old_offset/offset/
>> >
>> > >+ ERRMSG("Can't seek the bitmap(%s). %s\n",
>> > >+ bitmap->file_name, strerror(errno));
>> > >+ return FALSE;
>> >
>> > You have to include print_info.h to use ERRMSG.
>> >
>> >
>> > Thanks,
>> > Atsushi Kumagai
>> >
>> > >+ }
>> > > read(bitmap->fd, bitmap->buf, BUFSIZE_BITMAP);
>> > > if (pfn == 0)
>> > > bitmap->no_block = 0;
>> > >diff --git a/sadump_info.c b/sadump_info.c
>> > >index e2c4f03..e8154f4 100644
>> > >--- a/sadump_info.c
>> > >+++ b/sadump_info.c
>> > >@@ -465,9 +465,6 @@ read_sadump_header(char *filename)
>> > > smh = si->smh_memory;
>> > >
>> > > restart:
>> > >- if (block_size < 0)
>> > >- return FALSE;
>> > >-
>> > > if (!read_device(sph, block_size, &offset))
>> > > return ERROR;
>> > >
>> > >--
>> > >2.4.3
>> >
More information about the kexec
mailing list