jlaitine commented on code in PR #8026: URL: https://github.com/apache/nuttx/pull/8026#discussion_r1063401584
########## mm/shm/shmdt.c: ########## @@ -113,6 +91,16 @@ int shmdt(FAR const void *shmaddr) goto errout_with_errno; } + if (!group) Review Comment: the shmdt_priv is called from two places. "shmdt" is the normal detach function for user, and there the "group" always exists. shmdt_priv is also called from "munmap_shm" in shmat.c. From there the group can also be "null" in case unmap is being done at process destruction phase. -- 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