acassis commented on PR #3728:
URL: https://github.com/apache/nuttx-apps/pull/3728#issuecomment-5297354199

   @Abhishekmishra2808 this is not a valid setuid test:
   
   ```
   login: root
   password:
   User Logged-in!
   
   NuttShell (NSH) NuttX-13.0.0
   nsh# id
   uid=0(root) gid=0(root)
   nsh# ls -l /bin/sudo
    -rwsr-xr-x    root    root           0 /bin/sudo
   nsh# /bin/hello
   Hello, World!!
   ```
   
   A valid test should be something like:
   
   ```
   login: root
   password:
   User Logged-in!
   
   NuttShell (NSH) NuttX-13.0.0
   nsh# id
   uid=0(root) gid=0(root)
   nsh# ls -l /bin/sudo
    -rwsr-xr-x    root    root           0 /bin/sudo
   nsh# ls -l /bin/hello
    -rwxr--r--    root    root           0 /bin/hello
   nsh# su user
   nsh$ /bin/hello
   Error: you don't have permission
   nsh$ sudo /bin/hello
   Hello, World!!
   ```
   
   Something like that, but `user` should be in the sudoers or other config 
that allow it to run as root


-- 
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