Abhishekmishra2808 opened a new pull request, #18876:
URL: https://github.com/apache/nuttx/pull/18876

   ## Summary
   This PR fixes pseudoFS mode reporting in stat()/ls output by preserving only 
the synthesized file type bits (S_IFMT) and replacing permission bits from 
inode->i_mode instead of merging them with |= semantics. Previously, 
restrictive permission updates such as chmod 0600 could not override the 
default pseudoFS permission bits, causing stat() and ls -l to continue 
reporting modes like 0666.
   ## Impact
   
   This fixes incorrect permission reporting for pseudoFS inodes after chmod() 
updates and ensures that stat()/ls reflect the actual inode permission state.
   
   ## Testing
   
   Tested on sim:nsh
   Pre-requisite is https://github.com/apache/nuttx-apps/pull/3480 in chmod and 
chown is added in nsh
   Before fix:
   <img width="485" height="411" alt="image" 
src="https://github.com/user-attachments/assets/f8910651-7c99-49b2-b2ec-2b0cea6d8959";
 />
   
   
   After fix:
   <img width="425" height="362" alt="image" 
src="https://github.com/user-attachments/assets/f993cc89-3fdd-4dd5-bfb2-4fbb2a27d976";
 />
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to