anchao merged PR #15889:
URL: https://github.com/apache/nuttx/pull/15889
--
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.apach
jasonbu commented on code in PR #15889:
URL: https://github.com/apache/nuttx/pull/15889#discussion_r1966491515
##
binfmt/binfmt_copyactions.c:
##
@@ -122,6 +122,7 @@ int binfmt_copyactions(FAR const posix_spawn_file_actions_t
**copy,
prev->flink = (FAR void *)c
nuttxpr commented on PR #15889:
URL: https://github.com/apache/nuttx/pull/15889#issuecomment-2676141653
[**\[Experimental Bot, please feedback
here\]**](https://github.com/search?q=repo%3Aapache%2Fnuttx+13552&type=issues)
Yes, this PR appears to meet the basic NuttX requiremen
acassis commented on code in PR #15889:
URL: https://github.com/apache/nuttx/pull/15889#discussion_r1966121443
##
binfmt/binfmt_copyactions.c:
##
@@ -122,6 +122,7 @@ int binfmt_copyactions(FAR const posix_spawn_file_actions_t
**copy,
prev->flink = (FAR void *)c
jasonbu opened a new pull request, #15889:
URL: https://github.com/apache/nuttx/pull/15889
## Summary
We should not modify the input actions, casing when kernel build, userspace
call posix_spawn touch kernel address.
with current code, within posix_spawn will modify the file_action
