Update NEWS

[strace] / ipc_msgctl.c

diff --git a/ipc_msgctl.c b/ipc_msgctl.c
index a97dd1a1f6b01875eaf7a31d7a47192b6c56fb46..a7352cb9a301f412d4dde0b53e3fa08fe3ee9910 100644 (file)
--- a/ipc_msgctl.c
+++ b/ipc_msgctl.c
@@ -31,11 +31,21 @@
  */
 
 #include "defs.h"
-#include "ipc_defs.h"
 
-#include <sys/msg.h>
 #include DEF_MPERS_TYPE(msqid_ds_t)
+
+#include "ipc_defs.h"
+
+#ifdef HAVE_SYS_MSG_H
+/* The C library generally exports the struct the current kernel expects. */
+# include <sys/msg.h>
 typedef struct msqid_ds msqid_ds_t;
+#elif defined HAVE_LINUX_MSG_H
+/* The linux header might provide the right struct. */
+# include <linux/msg.h>
+typedef struct msqid64_ds msqid_ds_t;
+#endif
+
 #include MPERS_DEFS
 
 #include "xlat/msgctl_flags.h"
@@ -43,6 +53,7 @@ typedef struct msqid_ds msqid_ds_t;
 static void
 print_msqid_ds(struct tcb *tcp, const long addr, int cmd)
 {
+       /* TODO: We don't properly decode old compat ipc calls. */
        if (cmd & IPC_64)
                cmd &= ~IPC_64;
        msqid_ds_t msqid_ds;
@@ -56,7 +67,7 @@ print_msqid_ds(struct tcb *tcp, const long addr, int cmd)
                printuid("uid=", msqid_ds.msg_perm.uid);
                printuid(", gid=", msqid_ds.msg_perm.gid);
                tprints(", mode=");
-               tprints(sprintmode(msqid_ds.msg_perm.mode));
+               print_numeric_umode_t(msqid_ds.msg_perm.mode);
 
                if (cmd != IPC_STAT) {
                        tprints("}, ...}");
@@ -86,7 +97,7 @@ print_msqid_ds(struct tcb *tcp, const long addr, int cmd)
 SYS_FUNC(msgctl)
 {
        if (entering(tcp)) {
-               tprintf("%lu, ", tcp->u_arg[0]);
+               tprintf("%d, ", (int) tcp->u_arg[0]);
                PRINTCTL(msgctl_flags, tcp->u_arg[1], "MSG_???");
                tprints(", ");
        } else {