Since this command doesn't have any additional arguments, the comma does
not needed. Since this is the only command which lacks additional
arguments, it's better to add special case for it rather than add
printing of comma to all other commands.

* keyctl.c (SYS_FUNC(keyctl)): Add check for command not being
  KEYCTL_SESSION_TO_PARENT when printing comma dividing cmd argument
  from the rest.
---
 keyctl.c |    8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/keyctl.c b/keyctl.c
index a0bbbd0..c9081b7 100644
--- a/keyctl.c
+++ b/keyctl.c
@@ -214,7 +214,13 @@ SYS_FUNC(keyctl)
 
        if (entering(tcp)) {
                printxval(keyctl_commands, cmd, "KEYCTL_???");
-               tprints(", ");
+
+               /*
+                * For now, KEYCTL_SESSION_TO_PARENT is the only cmd without
+                * arguments.
+                */
+               if (cmd != KEYCTL_SESSION_TO_PARENT)
+                       tprints(", ");
        }
 
        switch (cmd) {
-- 
1.7.10.4


------------------------------------------------------------------------------
_______________________________________________
Strace-devel mailing list
Strace-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/strace-devel

Reply via email to