mirror of
https://github.com/apache/nuttx.git
synced 2026-05-27 19:36:35 +08:00
task_getpid.c:coverity HIS_metric_violation: RETURN
Use a local ret variable and return once at the end. This keeps the IDLE_PROCESS_ID fallback behavior unchanged. Addresses Coverity HIS_metric_violation: RETURN. Signed-off-by: fangxinyong <fangxinyong@xiaomi.com>
This commit is contained in:
@@ -56,6 +56,7 @@
|
|||||||
pid_t nxsched_getpid(void)
|
pid_t nxsched_getpid(void)
|
||||||
{
|
{
|
||||||
FAR struct tcb_s *rtcb;
|
FAR struct tcb_s *rtcb;
|
||||||
|
pid_t ret = IDLE_PROCESS_ID;
|
||||||
|
|
||||||
/* Get the TCB at the head of the ready-to-run task list. That
|
/* Get the TCB at the head of the ready-to-run task list. That
|
||||||
* will usually be the currently executing task. There is are two
|
* will usually be the currently executing task. There is are two
|
||||||
@@ -71,12 +72,8 @@ pid_t nxsched_getpid(void)
|
|||||||
{
|
{
|
||||||
/* Yes.. Return the Process ID */
|
/* Yes.. Return the Process ID */
|
||||||
|
|
||||||
return rtcb->group->tg_pid;
|
ret = rtcb->group->tg_pid;
|
||||||
}
|
}
|
||||||
|
|
||||||
/* We must have been called earlier in the start up sequence from the
|
return ret;
|
||||||
* start-up/IDLE thread before the ready-to-run list has been initialized.
|
|
||||||
*/
|
|
||||||
|
|
||||||
return IDLE_PROCESS_ID;
|
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user