]> granicus.if.org Git - strace/commitdiff
mknod.c: make use of RVAL_DECODED
authorDmitry V. Levin <ldv@altlinux.org>
Sun, 19 Jul 2015 23:07:17 +0000 (23:07 +0000)
committerDmitry V. Levin <ldv@altlinux.org>
Mon, 20 Jul 2015 01:42:01 +0000 (01:42 +0000)
* mknod.c (decode_mknod): Change to return void.
Update for RVAL_DECODED.
(sys_mknod, sys_mknodat): Update callers.  Update for RVAL_DECODED.
[SPARC || SPARC64] (xmknod): Update for RVAL_DECODED.

mknod.c

diff --git a/mknod.c b/mknod.c
index dbca736d800262579cad3a43f3ee45028212e936..a7aa774678c3d0aa6a05138f2c7f7398cc840ea3 100644 (file)
--- a/mknod.c
+++ b/mknod.c
 # include <sys/mkdev.h>
 #endif
 
-static int
+static void
 decode_mknod(struct tcb *tcp, int offset)
 {
        int mode = tcp->u_arg[offset + 1];
 
-       if (entering(tcp)) {
-               printpath(tcp, tcp->u_arg[offset]);
-               tprintf(", %s", sprintmode(mode));
-               switch (mode & S_IFMT) {
-               case S_IFCHR:
-               case S_IFBLK:
+       printpath(tcp, tcp->u_arg[offset]);
+       tprintf(", %s", sprintmode(mode));
+       switch (mode & S_IFMT) {
+       case S_IFCHR:
+       case S_IFBLK:
 #if defined(SPARC) || defined(SPARC64)
-                       if (current_personality == 1)
-                               tprintf(", makedev(%lu, %lu)",
-                               (unsigned long) ((tcp->u_arg[offset + 2] >> 18) & 0x3fff),
-                               (unsigned long) (tcp->u_arg[offset + 2] & 0x3ffff));
-                       else
+               if (current_personality == 1)
+                       tprintf(", makedev(%lu, %lu)",
+                       (unsigned long) ((tcp->u_arg[offset + 2] >> 18) & 0x3fff),
+                       (unsigned long) (tcp->u_arg[offset + 2] & 0x3ffff));
+               else
 #endif /* SPARC || SPARC64 */
-                               tprintf(", makedev(%lu, %lu)",
-                               (unsigned long) major(tcp->u_arg[offset + 2]),
-                               (unsigned long) minor(tcp->u_arg[offset + 2]));
-                       break;
-               default:
-                       break;
-               }
+                       tprintf(", makedev(%lu, %lu)",
+                       (unsigned long) major(tcp->u_arg[offset + 2]),
+                       (unsigned long) minor(tcp->u_arg[offset + 2]));
+               break;
        }
-       return 0;
 }
 
 SYS_FUNC(mknod)
 {
-       return decode_mknod(tcp, 0);
+       decode_mknod(tcp, 0);
+
+       return RVAL_DECODED;
 }
 
 SYS_FUNC(mknodat)
 {
-       if (entering(tcp))
-               print_dirfd(tcp, tcp->u_arg[0]);
-       return decode_mknod(tcp, 1);
+       print_dirfd(tcp, tcp->u_arg[0]);
+       decode_mknod(tcp, 1);
+
+       return RVAL_DECODED;
 }
 
 #if defined(SPARC) || defined(SPARC64)
@@ -57,21 +55,20 @@ SYS_FUNC(xmknod)
 {
        int mode = tcp->u_arg[2];
 
-       if (entering(tcp)) {
-               tprintf("%ld, ", tcp->u_arg[0]);
-               printpath(tcp, tcp->u_arg[1]);
-               tprintf(", %s", sprintmode(mode));
-               switch (mode & S_IFMT) {
-               case S_IFCHR:
-               case S_IFBLK:
-                       tprintf(", makedev(%lu, %lu)",
-                               (unsigned long) ((tcp->u_arg[3] >> 18) & 0x3fff),
-                               (unsigned long) (tcp->u_arg[3] & 0x3ffff));
-                       break;
-               default:
-                       break;
-               }
+       tprintf("%ld, ", tcp->u_arg[0]);
+       printpath(tcp, tcp->u_arg[1]);
+       tprintf(", %s", sprintmode(mode));
+       switch (mode & S_IFMT) {
+       case S_IFCHR:
+       case S_IFBLK:
+               tprintf(", makedev(%lu, %lu)",
+                       (unsigned long) ((tcp->u_arg[3] >> 18) & 0x3fff),
+                       (unsigned long) (tcp->u_arg[3] & 0x3ffff));
+               break;
+       default:
+               break;
        }
-       return 0;
+
+       return RVAL_DECODED;
 }
 #endif /* SPARC || SPARC64 */