Changeset 46577995 in mainline for kernel/generic/src/ipc/ops/dataread.c

Timestamp:

2018-01-04T20:50:52Z (8 years ago)

Author:

Jiří Zárevúcky <zarevucky.jiri@…>

Children:

e211ea04

Parents:

facacc71

git-author:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:47:53)

git-committer:

Jiří Zárevúcky <zarevucky.jiri@…> (2018-01-04 20:50:52)

Message:

Use errno_t in all uspace and kernel code.

Change type of every variable, parameter and return value that holds an
<errno.h> constant to either errno_t (the usual case), or sys_errno_t
(some places in kernel). This is for the purpose of self-documentation,
as well as for type-checking with a bit of type definition hackery.

After this commit, HelenOS is free of code that mixes error codes with non-error
values on the assumption that error codes are negative.

File:

• kernel/generic/src/ipc/ops/dataread.c (modified) (4 diffs)

kernel/generic/src/ipc/ops/dataread.c

@@ -41 +41 @@
 #include <config.h>
 
-static int request_preprocess(call_t *call, phone_t *phone)
+static errno_t request_preprocess(call_t *call, phone_t *phone)
 {
         size_t size = IPC_GET_ARG2(call->data);
@@ -57 +57 @@
 }
 
-static int answer_preprocess(call_t *answer, ipc_data_t *olddata)
+static errno_t answer_preprocess(call_t *answer, ipc_data_t *olddata)
 {
         assert(!answer->buffer);
@@ -76 +76 @@
                                 
                         answer->buffer = malloc(size, 0);
-                        int rc = copy_from_uspace(answer->buffer,
+                        errno_t rc = copy_from_uspace(answer->buffer,
                             (void *) src, size);
                         if (rc) {
@@ -95 +95 @@
 }
 
-static int answer_process(call_t *answer)
+static errno_t answer_process(call_t *answer)
 {
         if (answer->buffer) {
                 uintptr_t dst = IPC_GET_ARG1(answer->data);
                 size_t size = IPC_GET_ARG2(answer->data);
-                int rc;
+                errno_t rc;
 
                 rc = copy_to_uspace((void *) dst, answer->buffer, size);