Add comment explaining BpBinder::linkToDeath invalid operation

devinmoore-goog · devinmoore-goog · commit 7ba43785c9a7 · 2025-03-04T13:44:24.000-08:00
Test: none
Bug: none
Change-Id: I5e55018314079b895084d2ae06a197b9d9e54480
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`