Skip to content

Commit 148457d

Browse files
committed
doc tweaks
1 parent 2eb26ec commit 148457d

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

binder/src/main/java/io/grpc/binder/BinderChannelBuilder.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -284,19 +284,19 @@ public BinderChannelBuilder strictLifecycleManagement() {
284284
}
285285

286286
/**
287-
* Checks servers against this channel's {@link SecurityPolicy} *before* binding.
287+
* Checks servers against this Channel's {@link SecurityPolicy} *before* binding.
288288
*
289289
* <p>Android users can be tricked into installing a malicious app with the same package name as a
290290
* legitimate server. That's why we don't send calls to a server until it has been authorized by
291291
* an appropriate {@link SecurityPolicy}. But merely binding to a malicious server can enable
292-
* "keep-alive" and "background activity launch" attacks, even if security policy ultimately
292+
* "keep-alive" and "background activity launch" abuse, even if security policy ultimately
293293
* causes the grpc connection to fail. Pre-authorization is especially important for security when
294294
* the server's address isn't known in advance but rather resolved via target URI or discovered by
295295
* other means.
296296
*
297297
* <p>Note that, unlike ordinary authorization, pre-authorization is performed against the server
298-
* app's UID, not the UID of the server process. These can be different, most commonly due to
299-
* services that set `android:isolatedProcess=true`.
298+
* app's UID, not the UID of the process hosting the bound Service. These can be different, most
299+
* commonly due to services that set `android:isolatedProcess=true`.
300300
*
301301
* <p>Pre-authorization is strongly recommended but it remains optional for now because of this
302302
* behavior change and the small performance cost.

0 commit comments

Comments
 (0)