edwindwalker / edwinwalker

Personal site
0 stars 0 forks source link

NowSecure static analysis: Context Registered Broadcast Receivers Not Protected with Permissions #35

Closed jnowsecure closed 4 months ago

jnowsecure commented 9 months ago

Finding Description

The app dynamically registers some unprotected broadcast receivers.

Broadcast receivers are an inter-process communication ingest mechanism, allowing apps to receive data in the form of intents coming from other apps. They can be statically declared in the Android Manifest or registered dynamically at runtime. The latter is known as Context-registered receivers, because their lifetime is bound to a given Context object in the app. For example, an Activity is a Context that may have broadcast receivers registered against it at runtime.

As with static receivers, dynamic ones can be exported, protected using permissions or using signature enforcement. If left unprotected, as is the case with this finding, third party apps can send data to these receivers.

The exact impact varies on case-by-case basis depending on how the app handles incoming broadcasts. Although many cases are harmless, commonly seen consequences for the affected apps are:

Steps to Reproduce

This test reverse engineers the application code and detects any context-registered broadcast receivers that have not been protected with permissions.

Business Impact

A malicious app installed on the device can attempt to send broadcast intents that trigger the unprotected broadcast receiver. These intents can modify the runtime of the app, making the app a potential phishing vector. If the sender app's broadcasts contain sensitive information, a malicious app may be able to perform intent sniffing, leaking sensitive data. Furthermore, a malicious app can register itself with high priority, if the broadcast is an ordered broadcast, to receive the broadcast first. If the malicious app is the first to receive the broadcast, it could cancel the broadcast causing a denial of service, or it could inject a malicious data into the broadcast.

Remediation Resources

The evidence table presents the class names and method names where unprotected broadcast receivers have been registered in the app's source code.

Open your source code and browse to the listed methods. Review how they construct the parameters to the registerReceiver calls, what the receivers do with the incoming data and whether they handle sensitive data.

Note that the results might contain instances from third-party code. To inspect the results from your own code first you can search for the package names that your app uses.

If the broadcast receiver handles sensitive information you need to restrict access to it. These are the options:

If your app is supposed to only send and receive broadcasts to itself, avoid both Broadcast Receivers and using LocalBroadcastManager, which is deprecated.

Instead, Google recommends using an observable pattern, such as androidx.lifecycle.LiveData or reactive streams, depending on your use case.

Note: for results pointing to third-party libraries included in your app you should consider inspecting them carefully to see if the affected broadcast receiver handles sensitive information and if that's the case, consider updating or replacing those libraries.

Risk and Regulatory Information

Severity: medium

CVSS: 4

Policy Category: Default Priority 1

Application

See more detail in the NowSecure Report

jnowsecure commented 4 months ago

Update: This finding has been dismissed by Moi, Edwin with reason "risk accepted". No additional action is required.

Powered by NowSecure Platform