Closed Bug 1149094 (CVE-2015-2714) Opened 9 years ago Closed 9 years ago

Mixed content violation log on Fennec leaks sensitive info in URL


(Firefox for Android Graveyard :: General, defect)

Windows 8
Not set


(firefox38 fixed, firefox39 fixed, firefox40 fixed, firefox-esr31 unaffected, firefox-esr38 fixed, b2g-v2.0 unaffected, b2g-v2.0M unaffected, b2g-v2.1 unaffected, b2g-v2.1S unaffected, b2g-v2.2 unaffected, b2g-master unaffected)

Firefox 40
Tracking Status
firefox38 --- fixed
firefox39 --- fixed
firefox40 --- fixed
firefox-esr31 --- unaffected
firefox-esr38 --- fixed
b2g-v2.0 --- unaffected
b2g-v2.0M --- unaffected
b2g-v2.1 --- unaffected
b2g-v2.1S --- unaffected
b2g-v2.2 --- unaffected
b2g-master --- unaffected


(Reporter: sdna.muneaki.nishimura, Assigned: mfinkle)



(Keywords: sec-moderate, Whiteboard: [adv-main38+])


(1 file)

User Agent: Mozilla/5.0 (Windows NT 6.2; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/43.0.2342.2 Safari/537.36

Steps to reproduce:

1. Install Fennec to an Android device which version is 4.0 or less.
2. Launch
3. Ignore SSL error and load the page
4. See logcat (via adb or some way)

Actual results:

Accessed URL contains query string and fragment is leaked to Logcat like below.

W/GeckoConsole( 1364): [JavaScript Warning: "Loading mixed (insecure) display content "" on a secure page" {file: "" line: 0}]

On Android 4.0 or less, any application having android.permission.READ_LOGS permission can retrieve other application's log data. And also, it may be sent to the third party as crash logs. So, sensitive information may be leaked.

Expected results:

Mixed content violation log should remove detailed information of URL,e.g, path, query string and fragment.
Component: Untriaged → General
Flags: needinfo?(snorp)
Product: Firefox → Firefox for Android
Version: 39 Branch → unspecified
Mark, we already had a bug about not dumping console do logcat. Do you remember what happened there? I guess we never ended up turning that on?
Flags: needinfo?(snorp) → needinfo?(mark.finkle)
Flags: sec-bounty?
If it's Android 4.0 or less doesn't that imply the android people realized log access wasn't such a smart idea, making it (at least partially) their problem?

Tanvi: could we simply strip query params from the log output here? It's not really relevant to the mixed content blocking message. then again the secret bit could be anywhere in the path -- the real problem is access to the logs, or dumping console output to the log.

Any web site can put stuff on the console log, that might be perfectly fine for debugging output (visible to the user or developer) and they aren't considering whether it's safe for general log access.
Keywords: sec-moderate
(In reply to Daniel Veditz [:dveditz] from comment #2)
Yeah, in newer Android you can only view your own logs. Not other processes/activites/etc.

A malicious app would have to be installed as part of the system or by yourself. IMHO, you're already owned at that point, so us printing stuff to logcat isn't such a big deal.
There are two urls here... the mixed content url and the url of the place where mixed content was found/injected.  Are we talking about removing querystring/hash for both of them?  Will this impact how we can click on the injection site and see where the mixed content was injected, along with the line number?

(Ex: open the webconsole, enable the security tab, and go to on Firefox desktop).

Could we make the change android only since that is the only place where there is a security risk?

Aren't there other webconsole messages that leak similar data?  In that case, maybe the right solution here is to not dump console to logcat.

According to this, 4.0 or less is about 13% of the android market -  4.1 came out almost 3 years ago.
We can do the following:
1. Ignore this. It's limited to older Android versions and you need to be powned by a malicious app to get into trouble.
2. We stop outputting nsIConsoleSevice output to the Android log for RELEASE_BUILD. So Nightly, Aurora and local developer builds would still have the output.
3. We add some Developer setting to the Settings, allowing developers to see the output.

The best I'm willing to do is #2, unless I get feedback that crash report logs without nsIConsoleSevice output hurts our ability to fix bugs.
Flags: needinfo?(mark.finkle)
This patch drops the GeckoConsole output from RELEASE_BUILD (Beta and Final)
Assignee: nobody → mark.finkle
Attachment #8588594 - Flags: review?(blassey.bugs)
Attachment #8588594 - Flags: review?(blassey.bugs) → review+
Comment on attachment 8588594 [details] [diff] [review]
no-console-in-release v0.1

Approval Request Comment
[Feature/regressing bug #]: Leaking sensitive data to Android log
[User impact if declined]:
[Describe test coverage new/current, TreeHerder]:
[Risks and why]: Low risk. Just stop dumping to Android log. Still shows up in the Browser Console.
[String/UUID change made/needed]: None
Attachment #8588594 - Flags: approval-mozilla-beta?
Attachment #8588594 - Flags: approval-mozilla-aurora?
How far back does this issue go? A long way?
Hi Mark,
Is there a way to do this for just Android 4.0 and less?
Closed: 9 years ago
Resolution: --- → FIXED
Comment on attachment 8588594 [details] [diff] [review]
no-console-in-release v0.1

Should be in 38 beta 4
Attachment #8588594 - Flags: approval-mozilla-beta?
Attachment #8588594 - Flags: approval-mozilla-beta+
Attachment #8588594 - Flags: approval-mozilla-aurora?
Attachment #8588594 - Flags: approval-mozilla-aurora+
(In reply to Tanvi Vyas [:tanvi] from comment #10)
> Hi Mark,
> Is there a way to do this for just Android 4.0 and less?

needinfo'ing Mark for this question.  If there is, maybe we can file another bug to make this change.
Flags: needinfo?(mark.finkle)
Flags: sec-bounty? → sec-bounty+
Whiteboard: [adv-main38+]
Alias: CVE-2015-2714
(In reply to Tanvi Vyas [:tanvi] from comment #15)
> (In reply to Tanvi Vyas [:tanvi] from comment #10)
> > Hi Mark,
> > Is there a way to do this for just Android 4.0 and less?
> needinfo'ing Mark for this question.  If there is, maybe we can file another
> bug to make this change.

Do we want to dump this to the Android Log at all? The output will still show up in the Firefox console and the Remote Debugging tools.

To answer your question, yes, we could probably add a runtime check, but it would be an additional, small performance issue.
Flags: needinfo?(mark.finkle)
Group: core-security → core-security-release
Group: core-security-release
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.