]> granicus.if.org Git - sysstat/commitdiff
Fix unsigned compared against 0
authorSebastien GODARD <sysstat@users.noreply.github.com>
Fri, 1 Jul 2016 10:38:38 +0000 (12:38 +0200)
committerSebastien GODARD <sysstat@users.noreply.github.com>
Fri, 1 Jul 2016 10:38:38 +0000 (12:38 +0200)
file_magic->header_size is an unsigned int value and so cannot be
strictly less than 0 [CID#127473].

Signed-off-by: Sebastien GODARD <sysstat@users.noreply.github.com>
sa_common.c

index f4e8a1e4fcf13ed85f82f1aa75572f6d3b2f054c..266605007d28eb5f243927d70e894d037b0595c4 100644 (file)
@@ -1274,7 +1274,7 @@ int sa_open_read_magic(int *fd, char *dfile, struct file_magic *file_magic,
        if ((n != FILE_MAGIC_SIZE) ||
            (file_magic->sysstat_magic != SYSSTAT_MAGIC) ||
            ((file_magic->format_magic != FORMAT_MAGIC) && !ignore) ||
-           (file_magic->header_size < MIN_FILE_HEADER_SIZE) ||
+           (file_magic->header_size <= MIN_FILE_HEADER_SIZE) ||
            (file_magic->header_size > MAX_FILE_HEADER_SIZE) ||
            ((file_magic->header_size < FILE_HEADER_SIZE) && !ignore)) {
                /* Display error message and exit */