Skip to content

Commit 4273257

Browse files
committed
A few more fixes to specify charset
1 parent 4130858 commit 4273257

File tree

4 files changed

+11
-10
lines changed

4 files changed

+11
-10
lines changed

src/main/java/com/beowulfe/hap/impl/pairing/FinalPairHandler.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -75,17 +75,17 @@ private HttpResponse createResponse() throws Exception {
7575

7676
EdsaSigner signer = new EdsaSigner(authInfo.getPrivateKey());
7777

78-
byte[] material = ByteUtils.joinBytes(okm, authInfo.getMac().getBytes(), signer.getPublicKey());
78+
byte[] material = ByteUtils.joinBytes(okm, authInfo.getMac().getBytes(StandardCharsets.UTF_8), signer.getPublicKey());
7979

8080
byte[] proof = signer.sign(material);
8181

8282
Encoder encoder = TypeLengthValueUtils.getEncoder();
83-
encoder.add(MessageType.USERNAME, authInfo.getMac().getBytes());
83+
encoder.add(MessageType.USERNAME, authInfo.getMac().getBytes(StandardCharsets.UTF_8));
8484
encoder.add(MessageType.PUBLIC_KEY, signer.getPublicKey());
8585
encoder.add(MessageType.SIGNATURE, proof);
8686
byte[] plaintext = encoder.toByteArray();
8787

88-
ChachaEncoder chacha = new ChachaEncoder(hkdf_enc_key, "PS-Msg06".getBytes());
88+
ChachaEncoder chacha = new ChachaEncoder(hkdf_enc_key, "PS-Msg06".getBytes(StandardCharsets.UTF_8));
8989
byte[] ciphertext = chacha.encodeCiphertext(plaintext);
9090

9191
encoder = TypeLengthValueUtils.getEncoder();

src/main/java/com/beowulfe/hap/impl/pairing/PairVerificationManager.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -72,17 +72,17 @@ private HttpResponse stage1(Stage1Request request) throws Exception {
7272
byte[] proof = new EdsaSigner(authInfo.getPrivateKey()).sign(material);
7373

7474
HKDFBytesGenerator hkdf = new HKDFBytesGenerator(new SHA512Digest());
75-
hkdf.init(new HKDFParameters(sharedSecret, "Pair-Verify-Encrypt-Salt".getBytes(),
76-
"Pair-Verify-Encrypt-Info".getBytes()));
75+
hkdf.init(new HKDFParameters(sharedSecret, "Pair-Verify-Encrypt-Salt".getBytes(StandardCharsets.UTF_8),
76+
"Pair-Verify-Encrypt-Info".getBytes(StandardCharsets.UTF_8)));
7777
hkdfKey = new byte[32];
7878
hkdf.generateBytes(hkdfKey, 0, 32);
7979

8080
Encoder encoder = TypeLengthValueUtils.getEncoder();
81-
encoder.add(MessageType.USERNAME, authInfo.getMac().getBytes());
81+
encoder.add(MessageType.USERNAME, authInfo.getMac().getBytes(StandardCharsets.UTF_8));
8282
encoder.add(MessageType.SIGNATURE, proof);
8383
byte[] plaintext = encoder.toByteArray();
8484

85-
ChachaEncoder chacha = new ChachaEncoder(hkdfKey, "PV-Msg02".getBytes());
85+
ChachaEncoder chacha = new ChachaEncoder(hkdfKey, "PV-Msg02".getBytes(StandardCharsets.UTF_8));
8686
byte[] ciphertext = chacha.encodeCiphertext(plaintext);
8787

8888
encoder = TypeLengthValueUtils.getEncoder();
@@ -104,7 +104,7 @@ private HttpResponse stage2(Stage2Request request) throws Exception {
104104

105105
byte[] clientLtpk = authInfo.getUserPublicKey(authInfo.getMac()+new String(clientUsername, StandardCharsets.UTF_8));
106106
if (clientLtpk == null) {
107-
throw new Exception("Unknown user: "+new String(clientUsername));
107+
throw new Exception("Unknown user: "+new String(clientUsername, StandardCharsets.UTF_8));
108108
}
109109

110110
Encoder encoder = TypeLengthValueUtils.getEncoder();

src/main/java/com/beowulfe/hap/impl/pairing/PairingUpdateController.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@ public HttpResponse handle(HttpRequest request) throws IOException {
2929
authInfo.createUser(authInfo.getMac()+new String(username, StandardCharsets.UTF_8), ltpk);
3030
} else if (method == 4) { //Remove pairing
3131
byte[] username = d.getBytes(MessageType.USERNAME);
32-
authInfo.removeUser(authInfo.getMac()+new String(username));
32+
authInfo.removeUser(authInfo.getMac()+new String(username, StandardCharsets.UTF_8));
3333
if (!authInfo.hasUser()) {
3434
advertiser.setDiscoverable(true);
3535
}

src/main/java/com/beowulfe/hap/impl/responses/InternalServerErrorResponse.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
package com.beowulfe.hap.impl.responses;
22

33
import java.nio.ByteBuffer;
4+
import java.nio.charset.StandardCharsets;
45

56
import com.beowulfe.hap.impl.http.HttpResponse;
67

@@ -19,7 +20,7 @@ public int getStatusCode() {
1920

2021
@Override
2122
public ByteBuffer getBody() {
22-
return ByteBuffer.wrap(e.getClass().getName().getBytes());
23+
return ByteBuffer.wrap(e.getClass().getName().getBytes(StandardCharsets.UTF_8));
2324
}
2425

2526
public Exception getException() {

0 commit comments

Comments
 (0)