[rtems commit] ftpd: Restructure chroot() handling.

Sebastian Huber sebh at rtems.org
Wed Oct 10 11:59:06 UTC 2018


Module:    rtems
Branch:    master
Commit:    84a5921d33a5190779285017675e99e6b4aaf32f
Changeset: http://git.rtems.org/rtems/commit/?id=84a5921d33a5190779285017675e99e6b4aaf32f

Author:    Sebastian Huber <sebastian.huber at embedded-brains.de>
Date:      Tue Oct  9 08:12:46 2018 +0200

ftpd: Restructure chroot() handling.

Remove superfluous setting of errno = 0.

Update #3530.

---

 cpukit/ftpd/ftpd.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/cpukit/ftpd/ftpd.c b/cpukit/ftpd/ftpd.c
index 580e55c..9c84ff5 100644
--- a/cpukit/ftpd/ftpd.c
+++ b/cpukit/ftpd/ftpd.c
@@ -1872,7 +1872,6 @@ session(rtems_task_argument arg)
   while (1)
   {
     rtems_event_set set;
-    int rv;
 
     rtems_event_receive(FTPD_RTEMS_EVENT, RTEMS_EVENT_ANY, RTEMS_NO_TIMEOUT,
       &set);
@@ -1882,14 +1881,11 @@ session(rtems_task_argument arg)
         && chroot(ftpd_root) == 0);
 
     /*
-     * The chdir() must immediatly follow the chroot(), otherwise static
+     * The chdir() must immediately follow the chroot(), otherwise static
      * analysis tools may complain about a security issue.
-    */
-    rv = chroot_made ? chdir("/") : -1;
-
-    errno = 0;
+     */
 
-    if (rv == 0)
+    if (chroot_made && chdir("/") == 0)
     {
       send_reply(info, 220, FTPD_SERVER_MESSAGE);
 




More information about the vc mailing list