Merge "Add comment explaining BpBinder::linkToDeath invalid operation" into main am: 4212ea2 am: 93806e1

Treehugger Robot · android-build-merge-worker-robot · commit bcbec20c069c · 2025-03-18T11:32:12.000-07:00
Original change: https://android-review.googlesource.com/c/platform/frameworks/native/+/3524953 Change-Id: Idd2ba75fed12813dd397d8d1da2ef67f40f5d977 Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
diff --git a/libs/binder/Binder.cpp b/libs/binder/Binder.cpp
@@ -445,6 +445,9 @@ status_t BBinder::linkToDeath(
     const sp<DeathRecipient>& /*recipient*/, void* /*cookie*/,
     uint32_t /*flags*/)
 {
+    // BBinder::linkToDeath is invalid because this process owns this binder.
+    // The DeathRecipient is called on BpBinders when the process owning the
+    // binder node dies.
     return INVALID_OPERATION;
 }
 

Original file line number	Diff line number	Diff line change
`@@ -445,6 +445,9 @@ status_t BBinder::linkToDeath(`
`445`	`445`	`const sp<DeathRecipient>& /recipient/, void* /cookie/,`
`446`	`446`	`uint32_t /flags/)`
`447`	`447`	`{`
	`448`	`+ // BBinder::linkToDeath is invalid because this process owns this binder.`
	`449`	`+ // The DeathRecipient is called on BpBinders when the process owning the`
	`450`	`+ // binder node dies.`
`448`	`451`	`return INVALID_OPERATION;`
`449`	`452`	`}`
`450`	`453`