)]}'
{
  "commit": "c479487680b42b6cbe42a44aaffe9ff1ab733c69",
  "tree": "4a60df9eb32ebd0eafb18623806cf8f8c0f42c6f",
  "parents": [
    "a1aafc5c2bea98c81f5155f2ea0d1ca31a133012"
  ],
  "author": {
    "name": "Carmen Jackson",
    "email": "carmenjackson@google.com",
    "time": "Wed Jun 23 11:41:15 2021 -0700"
  },
  "committer": {
    "name": "Android Build Coastguard Worker",
    "email": "android-build-coastguard-worker@google.com",
    "time": "Thu Jun 24 23:53:32 2021 +0000"
  },
  "message": "Add Binder.clearCallingIdentity to TracingServiceProxy\n\nThe TracingServiceProxy is called by traced, which runs as UID 9999 and\ntherefore doesn\u0027t have the required permissions to start a foreground service.\n\nSo, clear that calling identity so that the identity checked for this\npermission is system_server, which does have the correct permissions.\n\nWe\u0027ll ensure that no other processes can utilize this path via selinux\nrules.\n\nBug: 191391382\nTest: Manually tested that before this change, I saw an\n\u0027ActivityManager: startForegroundService() not allowed\u0027 error when\ntaking a bugreport while a trace is running, while after this change the\nbugreport was taken successfully with no errors, and the trace was\nincluded in the bugreport.\n\nChange-Id: I4ae68047d588dfc87225ddf41288dc4093a71313\nMerged-In: I472fe8acc2e59e93afd8475f51b5f347cd3ccc5d\n(cherry picked from commit 1a856d556c829c20f2891d1f6a4545aa65baedb8)\n",
  "tree_diff": [
    {
      "type": "modify",
      "old_id": "8f227489740f84af07f09af2e19f9cedb1c14196",
      "old_mode": 33188,
      "old_path": "services/core/java/com/android/server/tracing/TracingServiceProxy.java",
      "new_id": "ff2f08bc4a50a495896671c2b07d7790befd9e91",
      "new_mode": 33188,
      "new_path": "services/core/java/com/android/server/tracing/TracingServiceProxy.java"
    }
  ]
}