edje: Fix coverity warning
authorJean-Philippe Andre <jp.andre@samsung.com>
Wed, 13 Jul 2016 06:44:58 +0000 (15:44 +0900)
committerJean-Philippe Andre <jp.andre@samsung.com>
Wed, 13 Jul 2016 06:44:58 +0000 (15:44 +0900)
CID 1355234 Logically dead code

The indicated dead code may have performed some action;
that action will never occur.
In eet_data_descriptor_element_add: Code can never be
reached because of a logical contradiction (CWE-561)

Solution: use explicit range within valid values with <= and >=
rather than excluded values with > and <

src/lib/eet/eet_data.c

index 0df96af..35ef7fb 100644 (file)
@@ -2223,9 +2223,8 @@ eet_data_descriptor_element_add(Eet_Data_Descriptor *edd,
     */
    if ((group_type > EET_G_UNKNOWN)
        && (group_type < EET_G_LAST)
-       && (((type > EET_T_UNKNOW) && (type < EET_T_STRING))
-           || ((type > EET_T_NULL) && (type < EET_T_VALUE))
-           || ((type > EET_T_VALUE) && (type < EET_T_LAST)))
+       && (((type >= EET_T_CHAR) && (type <= EET_T_ULONG_LONG))
+           || ((type >= EET_T_F32P32) && (type <= EET_T_F8P24)))
        && (!subtype))
      {
         subtype = calloc(1, sizeof (Eet_Data_Descriptor));