Android/frameworks_base
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

ContextHubService: Hack around 32-bit app ID

Browse Source 
am: 1983f9b05b

Change-Id: I9b87a8f35d2f97dcea4bbd73fd2783e1ece26d33
This commit is contained in:
Greg Kaiser
2016-09-27 23:46:08 +00:00
committed by android-build-merger
parent f411051195 1983f9b05b
commit 40c0435026
1 changed files with 35 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

51
core/java/android/hardware/location/ContextHubService.java
View File

@@ -18,6 +18,8 @@ package android.hardware.location;
import java.io.FileDescriptor;
import java.io.PrintWriter;
import java.nio.ByteBuffer;
import java.nio.ByteOrder;
import java.util.ArrayList;
import java.util.HashMap;
@@ -146,6 +148,36 @@ public class ContextHubService extends IContextHubService.Stub {
return mContextHubInfo[contextHubHandle];
}
// TODO(b/30808791): Remove this when NanoApp's API is correctly treating
// app IDs as 64-bits.
private static long parseAppId(NanoApp app) {
// NOTE: If this shifting seems odd (since it's actually "ONAN"), note
// that it matches how this is defined in context_hub.h.
final int HEADER_MAGIC =
(((int)'N' << 0) |
((int)'A' << 8) |
((int)'N' << 16) |
((int)'O' << 24));
final int HEADER_MAGIC_OFFSET = 4;
final int HEADER_APP_ID_OFFSET = 8;
ByteBuffer header = ByteBuffer.wrap(app.getAppBinary())
.order(ByteOrder.LITTLE_ENDIAN);
try {
if (header.getInt(HEADER_MAGIC_OFFSET) == HEADER_MAGIC) {
// This is a legitimate nanoapp header. Let's grab the app ID.
return header.getLong(HEADER_APP_ID_OFFSET);
}
} catch (IndexOutOfBoundsException e) {
// The header is undersized. We'll fall through to our code
// path below, which handles being unable to parse the header.
}
// We failed to parse the header. Even through it's probably wrong,
// let's give NanoApp's idea of our ID. This is at least consistent.
return app.getAppId();
}
@Override
public int loadNanoApp(int contextHubHandle, NanoApp app) throws RemoteException {
checkPermissions();
@@ -162,27 +194,14 @@ public class ContextHubService extends IContextHubService.Stub {
msgHeader[HEADER_FIELD_MSG_TYPE] = MSG_LOAD_NANO_APP;
long appId = app.getAppId();
// TODO(b/30808791): Remove this hack when the NanoApp API is fixed.
// Due to a bug in the NanoApp API, only the least significant four
// bytes of the app ID can be stored. The most significant five
// bytes of a normal app ID are the "vendor", and thus the most
// significant of the bytes we have is the least significant byte
// of the vendor. In the case that byte is the ASCII value for
// lower-case 'L', we assume the vendor is supposed to be "Googl"
// and fill in the four most significant bytes accordingly.
// TODO(b/30808791): Remove this hack when the NanoApp API is fixed,
// and getAppId() returns a 'long' instead of an 'int'.
if ((appId >> 32) != 0) {
// We're unlikely to notice this warning, but at least
// we can avoid running our hack logic.
Log.w(TAG, "Code has not been updated since API fix.");
} else {
// Note: Lower-case 'L', not the number 1.
if (((appId >> 24) & 0xFF) == (long)'l') {
// Assume we're a Google nanoapp.
appId |= ((long)'G') << 56;
appId |= ((long)'o') << 48;
appId |= ((long)'o') << 40;
appId |= ((long)'g') << 32;
}
appId = parseAppId(app);
}
msgHeader[HEADER_FIELD_LOAD_APP_ID_LO] = (int)(appId & 0xFFFFFFFF);