diff options
author | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-04-23 14:42:58 +0200 |
---|---|---|
committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2012-06-06 13:57:43 +0200 |
commit | 8250503f218be1a3fa434e905631b1636cf6d033 (patch) | |
tree | b23dfdca86e25ad19ff70a4799b8f7a09389a82d /cpukit/librpc/include/rpc/rpc_msg.h | |
parent | network/bootp: PR2031: Add and use header file (diff) | |
download | rtems-8250503f218be1a3fa434e905631b1636cf6d033.tar.bz2 |
librpc: PR2066: Fix for short enums
The XDR library has a problem on architectures with short enums like the
default ARM EABI. Short enums means that the size of the enum type is
variable and the smallest integer type to hold all enum values will be
selected. For many enums this is char. The XDR library uses int32_t
for enum_t. There are several evil casts from an enum type to enum_t
which leads to invalid memory accesses on short enum architectures. A
workaround is to add appropriate dummy enum values.
Diffstat (limited to '')
-rw-r--r-- | cpukit/librpc/include/rpc/rpc_msg.h | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/cpukit/librpc/include/rpc/rpc_msg.h b/cpukit/librpc/include/rpc/rpc_msg.h index 1668fbf7ee..63a1f360a0 100644 --- a/cpukit/librpc/include/rpc/rpc_msg.h +++ b/cpukit/librpc/include/rpc/rpc_msg.h @@ -58,12 +58,14 @@ struct rpc_err; /* forward */ enum msg_type { CALL=0, - REPLY=1 + REPLY=1, + _MSG_TYPE = 0xffffffff }; enum reply_stat { MSG_ACCEPTED=0, - MSG_DENIED=1 + MSG_DENIED=1, + _REPLY_STAT = 0xffffffff }; enum accept_stat { @@ -72,12 +74,14 @@ enum accept_stat { PROG_MISMATCH=2, PROC_UNAVAIL=3, GARBAGE_ARGS=4, - SYSTEM_ERR=5 + SYSTEM_ERR=5, + _ACCEPT_STAT = 0xffffffff }; enum reject_stat { RPC_MISMATCH=0, - AUTH_ERROR=1 + AUTH_ERROR=1, + _REJECT_STAT = 0xffffffff }; /* |