falcon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ying Zheng (JIRA)" <j...@apache.org>
Subject [jira] [Updated] (FALCON-2073) Handle with NULL corner cases
Date Tue, 12 Jul 2016 22:37:20 GMT

     [ https://issues.apache.org/jira/browse/FALCON-2073?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ying Zheng updated FALCON-2073:
-------------------------------
    Description: 
Coverity scan finds a few places where the return value of an object could be NULL but Falcon
tries to access its member without handling with the corner NULL case. For example, cluster
update could run into an error if there is no dependent entity for the cluster, see OozieWorkflowEngine::updateDependents:

{code}
        Set<Entity> affectedEntities = EntityGraph.get().getDependents(oldEntity);
        for (Entity affectedEntity : affectedEntities) {
                 ......
        }
{code}

  was:
Coverity scan finds a few places where the return value of an object could be NULL but Falcon
tries to access its member without handling with the corner NULL case. For example, in OozieWorkflowEngine::updateDependents,
we will run into an error if there is no dependency edge for the oldEntity:

{code}
        Set<Entity> affectedEntities = EntityGraph.get().getDependents(oldEntity);
        for (Entity affectedEntity : affectedEntities) {
                 ......
        }
{code}


> Handle with NULL corner cases
> -----------------------------
>
>                 Key: FALCON-2073
>                 URL: https://issues.apache.org/jira/browse/FALCON-2073
>             Project: Falcon
>          Issue Type: Bug
>            Reporter: Ying Zheng
>            Assignee: Ying Zheng
>
> Coverity scan finds a few places where the return value of an object could be NULL but
Falcon tries to access its member without handling with the corner NULL case. For example,
cluster update could run into an error if there is no dependent entity for the cluster, see
OozieWorkflowEngine::updateDependents:
> {code}
>         Set<Entity> affectedEntities = EntityGraph.get().getDependents(oldEntity);
>         for (Entity affectedEntity : affectedEntities) {
>                  ......
>         }
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message