efl_ui_focus_manager: Add null check 52/190052/2
authorJunsuChoi <jsuya.choi@samsung.com>
Thu, 27 Sep 2018 02:13:36 +0000 (11:13 +0900)
committerGerrit Code Review <gerrit@review.ap-northeast-2.compute.internal>
Thu, 27 Sep 2018 02:39:57 +0000 (02:39 +0000)
Summary:
This commit add null check on _next and _request_subchild function.
The value 'node' is can be null.

Test Plan: N/A

Reviewers: bu5hm4n, YOhoho, Hermet

Reviewed By: Hermet

Subscribers: cedric, #reviewers, #committers

Tags: #efl

Differential Revision: https://phab.enlightenment.org/D7069

Change-Id: I9c12ebc17bd2e959a4fc388ad761ee7d171581fd

src/lib/elementary/efl_ui_focus_manager_calc.c

index a366271..ee6840c 100644 (file)
@@ -1111,6 +1111,7 @@ static Node*
 _next(Node *node)
 {
    Node *n;
+   if (!node) return NULL;
 
    //Case 1 we are having children
    //But only enter the children if it does NOT have a redirect manager
@@ -1295,6 +1296,7 @@ _request_subchild(Node *node)
 {
    //important! if there are no children _next would return the parent of node which will exceed the limit of children of node
    Node *target = NULL;
+   if (!node) return target;
 
    if (node->tree.children)
      {