diff options
author | Ben Beetle <ben.beetle@gmail.com> | 2014-12-05 17:29:13 -0500 |
---|---|---|
committer | Ben Beetle <ben.beetle@gmail.com> | 2014-12-05 17:29:13 -0500 |
commit | c57cc478650bfce3276a0e0dc6122d273e850b22 (patch) | |
tree | 63a97c16d7521f91ae6b66ef50adb34f167f59c2 | |
parent | d3897336634a5f3d29ba18a22a34953f726262b2 (diff) |
Don't duplicate commandName in SSH log
Using an enhanced for loop caused it to include arg[0]
which duplicated commandName. Now it uses an explicit
for loop starting at index 1 to avoid this.
Change-Id: Ie1db7d41c1670ba463f8c8002560192183015113
-rw-r--r-- | gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshLog.java | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshLog.java b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshLog.java index fc80f5a9bb..82394afbcf 100644 --- a/gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshLog.java +++ b/gerrit-sshd/src/main/java/com/google/gerrit/sshd/SshLog.java @@ -269,8 +269,9 @@ class SshLog implements LifecycleListener { private String extractWhat(DispatchCommand dcmd) { String commandName = dcmd.getCommandName(); - for (String arg : dcmd.getArguments()) { - commandName = commandName + "." + arg; + String[] args = dcmd.getArguments(); + for (int i = 1; i < args.length; i++) { + commandName = commandName + "." + args[i]; } return commandName; } |