diff options
| author | 2013-01-09 10:53:51 -0800 | |
|---|---|---|
| committer | 2013-01-09 10:53:51 -0800 | |
| commit | aa621bf26ff4c967993878b7787c45e6bdd7af6d (patch) | |
| tree | 51a0b7be44f1bd97f4075284931d954203bb291a | |
| parent | 176baa7de11be910c36b7b4dfa7826b55ec97963 (diff) | |
Use readUnsignedByte.
To remove any doubt about how java is doing its byte/integer arithmetic
use readUnsignedByte instead of readByte.
Change-Id: I5bdc200ed0cda51ad739804ac1d8d13c043424ec
| -rw-r--r-- | wifi/java/android/net/wifi/p2p/nsd/WifiP2pServiceResponse.java | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/wifi/java/android/net/wifi/p2p/nsd/WifiP2pServiceResponse.java b/wifi/java/android/net/wifi/p2p/nsd/WifiP2pServiceResponse.java index ac31663d47f9..194c982b870d 100644 --- a/wifi/java/android/net/wifi/p2p/nsd/WifiP2pServiceResponse.java +++ b/wifi/java/android/net/wifi/p2p/nsd/WifiP2pServiceResponse.java @@ -222,10 +222,10 @@ public class WifiP2pServiceResponse implements Parcelable { */ // The length equals to 3 plus the number of octets in the vendor // specific content field. And this is little endian. - int length = ((dis.readByte() & 0xff) + - ((dis.readByte() & 0xff) << 8)) - 3; + int length = (dis.readUnsignedByte() + + (dis.readUnsignedByte() << 8)) - 3; int type = dis.readUnsignedByte(); - byte transId = dis.readByte(); + int transId = dis.readUnsignedByte(); int status = dis.readUnsignedByte(); if (length < 0) { return null; |