Changeset 302 for trunk/kernel/syscalls/sys_chmod.c

Timestamp:

Jul 31, 2017, 2:36:48 PM (7 years ago)

Author:

max@…

Message:

Style, and use hal_strcpy_from_uspace.

File:

• trunk/kernel/syscalls/sys_chmod.c (modified) (3 diffs)

trunk/kernel/syscalls/sys_chmod.c

@@ -1 +1 @@
 /*
  * sys_chmod.c - Change file access rights.
- * 
+ *
  * Author    Alain Greiner  (2016,2017)
  *
@@ -35 +35 @@
 {
     error_t     error;
-    paddr_t     paddr;
-    uint32_t    length;
     char        kbuf[CONFIG_VFS_MAX_PATH_LENGTH];
-        
-        thread_t  * this    = CURRENT_THREAD;
-        process_t * process = this->process;
 
-    // check pathname in user space
-    error = vmm_v2p_translate( false , pathname , &paddr );
+    thread_t  * this    = CURRENT_THREAD;
+    process_t * process = this->process;
 
-        if( error )
-        {
-        printk("\n[ERROR] in %s : user pathname unmapped for thread %x in process %x\n",
-               __FUNCTION__ , this->trdid , process->pid );
-                this->errno = EINVAL;
-                return -1;
-        }       
+    // get pathname copy in kernel space
+    error = hal_strcpy_from_uspace( kbuf , pathname , CONFIG_VFS_MAX_PATH_LENGTH );
 
-    // get pathname length
-    length = hal_strlen_from_uspace( pathname );
-
-    if( length >= CONFIG_VFS_MAX_PATH_LENGTH )
+    if( error )
     {
         printk("\n[ERROR] in %s : pathname too long\n", __FUNCTION__ );
-                this->errno = ENFILE;
+        this->errno = ENFILE;
         return -1;
     }
- 
-        // get pathname copy in kernel space
-    hal_copy_from_uspace( kbuf, pathname, length );
 
     // get cluster and local pointer on reference process
@@ -73 +57 @@
     // get extended pointer on cwd inode
     xptr_t cwd_xp = hal_remote_lwd( XPTR( ref_cxy , &ref_ptr->vfs_cwd_xp ) );
-   
+
     // get the cwd lock in read mode from reference process
-        remote_rwlock_rd_lock( XPTR( ref_cxy , &ref_ptr->cwd_lock ) );
+    remote_rwlock_rd_lock( XPTR( ref_cxy , &ref_ptr->cwd_lock ) );
 
     // call the relevant VFS function
-        error = vfs_chmod( cwd_xp,
+    error = vfs_chmod( cwd_xp,
                        kbuf,
                        rights );
 
     // release the cwd lock
-        remote_rwlock_rd_unlock( XPTR( ref_cxy , &ref_ptr->cwd_lock ) );
+    remote_rwlock_rd_unlock( XPTR( ref_cxy , &ref_ptr->cwd_lock ) );
 
     if( error )
-        {
+    {
         printk("\n[ERROR] in %s : cannot remove directory %s\n",
                __FUNCTION__ , kbuf );
-                this->errno = error;
-                return -1;
-        }
-   
-        return 0;
+        this->errno = error;
+        return -1;
+    }
 
-}  // end sys_chmod()   
+    return 0;
+
+}  // end sys_chmod()