Closed Bug 1698174 Opened 2 years ago Closed 2 years ago

GeckoThread.getProcessName is not compatible with isolatedProcess

Categories

(GeckoView :: Sandboxing, enhancement, P1)

Unspecified
All
enhancement

Tracking

(firefox89 fixed)

RESOLVED FIXED
89 Branch
Tracking Status
firefox89 --- fixed

People

(Reporter: bugzilla, Assigned: bugzilla)

References

(Blocks 1 open bug)

Details

(Whiteboard: [geckoview:m89][sandboxing])

Attachments

(1 file)

This breaks our MOZ_DEBUG_CHILD_WAIT_FOR_JAVA_DEBUGGER environment variable.

Stack:

    java.lang.SecurityException: Isolated process not allowed to call getRunningAppProcesses
        at android.os.Parcel.readException(Parcel.java:1683)
        at android.os.Parcel.readException(Parcel.java:1636)
        at android.app.ActivityManagerProxy.getRunningAppProcesses(ActivityManagerNative.java:3818)
        at android.app.ActivityManager.getRunningAppProcesses(ActivityManager.java:3053)
        at org.mozilla.gecko.GeckoThread.getProcessName(GeckoThread.java:584)
        at org.mozilla.gecko.GeckoThread.maybeWaitForJavaDebugger(GeckoThread.java:498)
        at org.mozilla.gecko.GeckoThread.run(GeckoThread.java:426)
Assignee: nobody → aklotz
Status: NEW → ASSIGNED
Whiteboard: [geckoview:m89] → [geckoview:m89][sandboxing]
Severity: -- → S3
Priority: -- → P1

We observe that, while we don't have proper access to the ActivityManager,
we can figure out the name of our own process just by using details about the
name of our GeckoServiceChildProcess class, plus the app's package name.

Attachment #9210078 - Attachment description: Bug 1698174: Content services should derive their process name from properties of their own Service; r=#geckoview-reviewers → Bug 1698174: Send a newly created child process's name using IChildProcess.start; r=#geckoview-reviewers
Attachment #9210078 - Attachment description: Bug 1698174: Send a newly created child process's name using IChildProcess.start; r=#geckoview-reviewers → Bug 1698174: Remove support for MOZ_DEBUG_*_WAIT_FOR_JAVA_DEBUGGER environment variables; r=#geckoview-reviewers
Pushed by aklotz@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/16e354847fa2
Remove support for MOZ_DEBUG_*_WAIT_FOR_JAVA_DEBUGGER environment variables; r=geckoview-reviewers,agi
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 89 Branch
See Also: → 1522318

Moving isolated process bugs to the new GeckoView::Sandboxing component.

Component: General → Sandboxing
You need to log in before you can comment on or make changes to this bug.