Changeset e3c762cd in mainline for generic/src/ipc/irq.c

Timestamp:

2006-05-05T11:59:19Z (19 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

de07bcf

Parents:

22cf454d

Message:

Complete implementation of copy_from_uspace() and copy_to_uspace()
for amd64 and ia32. Other architectures still compile and run,
but need to implement their own assembly-only memcpy(), memcpy_from_uspace(),
memcpy_to_uspace() and their failover parts. For these architectures
only dummy implementations are provided.

File:

• generic/src/ipc/irq.c (modified) (3 diffs)

generic/src/ipc/irq.c

@@ -48 +48 @@
 #include <ipc/irq.h>
 #include <atomic.h>
+#include <syscall/copy.h>
 
 typedef struct {
@@ -121 +122 @@
         irq_code_t *code;
         irq_cmd_t *ucmds;
+        int rc;
 
         code = malloc(sizeof(*code), 0);
-        copy_from_uspace(code, ucode, sizeof(*code));
+        rc = copy_from_uspace(code, ucode, sizeof(*code));
+        if (rc != 0) {
+                free(code);
+                return NULL;
+        }
         
         if (code->cmdcount > IRQ_MAX_PROG_SIZE) {
@@ -131 +137 @@
         ucmds = code->cmds;
         code->cmds = malloc(sizeof(code->cmds[0]) * (code->cmdcount), 0);
-        copy_from_uspace(code->cmds, ucmds, sizeof(code->cmds[0]) * (code->cmdcount));
+        rc = copy_from_uspace(code->cmds, ucmds, sizeof(code->cmds[0]) * (code->cmdcount));
+        if (rc != 0) {
+                free(code->cmds);
+                free(code);
+                return NULL;
+        }
 
         return code;