datetime: Correct the validation of parameter in _field_format_get/_field_limit_get
authorRyuan Choi <ryuan.choi@gmail.com>
Tue, 4 Feb 2014 15:53:14 +0000 (00:53 +0900)
committerRyuan Choi <ryuan.choi@gmail.com>
Tue, 4 Feb 2014 15:53:14 +0000 (00:53 +0900)
Because field_list is an array, we should check boundary instead of NULL.

src/lib/elm_datetime.c

index 9436478..e47188b 100644 (file)
@@ -693,10 +693,11 @@ _field_format_get(Evas_Object *obj,
 {
    Datetime_Field *field;
 
+   if (field_type > ELM_DATETIME_AMPM) return NULL;
+
    ELM_DATETIME_DATA_GET(obj, sd);
 
    field = sd->field_list + field_type;
-   if (!field) return NULL;
 
    return field->fmt;
 }
@@ -711,10 +712,11 @@ _field_limit_get(Evas_Object *obj,
    Datetime_Field *field;
    unsigned int idx;
 
+   if (field_type > ELM_DATETIME_AMPM) return;
+
    ELM_DATETIME_DATA_GET(obj, sd);
 
    field = sd->field_list + field_type;
-   if (!field) return;
 
    min = field->min;
    max = field->max;