Changeset 594 for trunk/kernel/syscalls/sys_write.c

Timestamp:

Nov 10, 2018, 2:33:26 PM (5 years ago)

Author:

alain

Message:

Fix various bugs in sys_stat() and sys_mmap() functions.
Improve debug in other functions.

File:

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

trunk/kernel/syscalls/sys_write.c

@@ -153 +153 @@
                                 count );
     }
-    else if( type == INODE_TYPE_DEV )  // check ownership & write to device
-    {
-        // check user buffer size for TXT_TX
-        if( (type == INODE_TYPE_DEV) && (count >= CONFIG_TXT_KBUF_SIZE) )
-        {
-
-#if DEBUG_SYSCALLS_ERROR
-printk("\n[ERROR] in %s : thread[%x,%x] user buffer size %x too large\n",
-__FUNCTION__ , process->pid, this->trdid, count );
-#endif
-                    this->errno = EINVAL;
-                    return -1;
-        }
-
+    else if( type == INODE_TYPE_DEV )  // write to TXT device
+    {
         // move count bytes to device
         nbytes = devfs_user_move( false,             // from buffer to device
@@ -175 +163 @@
     else  // not FILE and not DEV
     {
-        nbytes = 0;
-        assert( false , "file type %d non supported\n", type );
+
+#if DEBUG_SYSCALLS_ERROR
+printk("\n[ERROR] in %s : thread[%x,%x] / illegal inode type %\n",
+__FUNCTION__, vfs_inode_type_str( type ) );
+#endif
+                this->errno = EBADFD;
+                return -1;
     }
 
@@ -186 +179 @@
 __FUNCTION__ , process->pid, this->trdid, file_id );
 #endif
-        this->errno = error;
+        this->errno = EIO;
         return -1;
     }