drivers: partition: correct some static analysis tools issues
authorYann Gautier <yann.gautier@st.com>
Fri, 9 Nov 2018 17:21:04 +0000 (18:21 +0100)
committerYann Gautier <yann.gautier@st.com>
Fri, 9 Nov 2018 17:22:01 +0000 (18:22 +0100)
cppcheck:
[drivers/partition/gpt.c:19] -> [drivers/partition/gpt.c:19]:
 (warning) Either the condition 'str_in!=((void*)0)' is redundant
 or there is possible null pointer dereference: name.

sparse:
drivers/partition/gpt.c:39:9:
 warning: Using plain integer as NULL pointer

Signed-off-by: Yann Gautier <yann.gautier@st.com>
drivers/partition/gpt.c

index 9cc917d33c18a6c5ac51944753534552814f7d82..0c51e62a87864b223fa163f9d1c6ee720ec0113b 100644 (file)
 
 static int unicode_to_ascii(unsigned short *str_in, unsigned char *str_out)
 {
-       uint8_t *name = (uint8_t *)str_in;
+       uint8_t *name;
        int i;
 
-       assert((str_in != NULL) && (str_out != NULL) && (name[0] != '\0'));
+       assert((str_in != NULL) && (str_out != NULL));
+
+       name = (uint8_t *)str_in;
+
+       assert(name[0] != '\0');
 
        /* check whether the unicode string is valid */
        for (i = 1; i < (EFI_NAMELEN << 1); i += 2) {
@@ -36,7 +40,7 @@ int parse_gpt_entry(gpt_entry_t *gpt_entry, partition_entry_t *entry)
 {
        int result;
 
-       assert((gpt_entry != 0) && (entry != 0));
+       assert((gpt_entry != NULL) && (entry != NULL));
 
        if ((gpt_entry->first_lba == 0) && (gpt_entry->last_lba == 0)) {
                return -EINVAL;