[GitHub] [dolphinscheduler] github-code-scanning[bot] commented on a diff in pull request #12533: [Bug] [API] The workflow definition and the tenant in the workflow instance are inconsistent.

[GitBox <gi...@apache.org>]

github-code-scanning[bot] commented on code in PR #12533:
URL: https://github.com/apache/dolphinscheduler/pull/12533#discussion_r1004769143


##########
dolphinscheduler-api/src/main/java/org/apache/dolphinscheduler/api/service/impl/ProcessInstanceServiceImpl.java:
##########
@@ -241,7 +241,10 @@
         ProcessDefinition processDefinition =
                 processService.findProcessDefinition(processInstance.getProcessDefinitionCode(),
                         processInstance.getProcessDefinitionVersion());
-
+        Tenant tenant = tenantMapper.queryById(processDefinition.getTenantId());

Review Comment:
   ## Dereferenced variable may be null
   
   Variable [processDefinition](1) may be null at this access as suggested by [this](2) null guard.
   
   [Show more details](https://github.com/apache/dolphinscheduler/security/code-scanning/2096)



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@dolphinscheduler.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org