opendev/gerrit
Code Issues Proposed changes

ChangeUpdate: Sanitize identities in footers

Browse Source 
Change-Id: I3fb35eeae1fd62b778b3ac08eda1f039b20f95b5
This commit is contained in:
Dave Borowitz
2016-05-28 19:57:35 -04:00
parent 136aab1f0a
commit 74c35d77a9
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
gerrit-server/src/main/java/com/google/gerrit/server/notedb/ChangeUpdate.java
View File

@@ -675,8 +675,11 @@ public class ChangeUpdate extends AbstractChangeUpdate {
private StringBuilder addIdent(StringBuilder sb, Account.Id accountId) {
Account account = accountCache.get(accountId).getAccount();
PersonIdent ident = newIdent(account, when);
sb.append(ident.getName()).append(" <").append(ident.getEmailAddress())
.append('>');
PersonIdent.appendSanitized(sb, ident.getName());
sb.append(" <");
PersonIdent.appendSanitized(sb, ident.getEmailAddress());
sb.append('>');
return sb;
}