summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorBen Beetle <ben.beetle@gmail.com>2014-12-05 17:29:13 -0500
committerBen Beetle <ben.beetle@gmail.com>2014-12-05 17:29:13 -0500
commitc57cc478650bfce3276a0e0dc6122d273e850b22 (patch)
tree63a97c16d7521f91ae6b66ef50adb34f167f59c2
parentd3897336634a5f3d29ba18a22a34953f726262b2 (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.java5
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;
}