[GitHub] [incubator-nuttx] qinwei2004 commented on a diff in pull request #7597: sched/task/task_getpid: getpid should return process id not thread id

[GitBox]

qinwei2004 commented on code in PR #7597:
URL: https://github.com/apache/incubator-nuttx/pull/7597#discussion_r1023404552


##########
fs/procfs/fs_procfsproc.c:
##########
@@ -1484,7 +1484,7 @@ static int proc_open(FAR struct file *filep, FAR const 
char *relpath,
 
   if (strncmp(relpath, "self", 4) == 0)
     {
-      tmp = (unsigned long)getpid();    /* Get the PID of the calling task */
+      tmp = (unsigned long)gettid();    /* Get the PID of the calling task */

Review Comment:
   it seem the variable tmp is defined as unsigned long
   So I think we should keep this



-- 
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-unsubscr...@nuttx.apache.org

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