diff options
| author | Chao Fan <cfan@redhat.com> | 2015-08-26 08:58:25 +0900 |
|---|---|---|
| committer | Atsushi Kumagai <kumagai-atsushi@mxc.nes.nec.co.jp> | 2015-08-26 08:58:49 +0900 |
| commit | 0c65e3809e0201852c06289141e79348ea392869 (patch) | |
| tree | 62fd47b2a81f209927af1142a2827efc6f03ec8c /sadump_info.c | |
| parent | b18a8a66d5529655e7d4d32048e159d3142cbba6 (diff) | |
[PATCH v2] Code changes to satisfy the coverity scan.
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@redhat.com>
Diffstat (limited to 'sadump_info.c')
| -rw-r--r-- | sadump_info.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/sadump_info.c b/sadump_info.c index 47da45b..fc45145 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; |