[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #21830 [Applications/Tor Browser]: Copying large text from web console leaks to /tmp
#21830: Copying large text from web console leaks to /tmp
-------------------------------------------------+-------------------------
Reporter: gk | Owner: neillm
Type: defect | Status:
| needs_review
Priority: Medium | Milestone:
Component: Applications/Tor Browser | Version:
Severity: Normal | Resolution:
Keywords: tbb-disk-leak, | Actual Points:
TorBrowserTeam201707R |
Parent ID: | Points:
Reviewer: | Sponsor:
-------------------------------------------------+-------------------------
Comment (by arthuredelstein):
Replying to [comment:8 neillm]:
> This patch has been applied to tor-browser-52.2.0esr-7.0-1-build1 and
tested on Ubuntu 16.04.2 LTS.
Thanks -- I built with the patch and it worked as described. But the
`aContext` argument of `nsTransferable::Init()` appears to have only one
purpose, which is to check for PBM state. So I wonder if you think it
would make sense to change the signature to `nsTransferable::Init(bool
isPrivateBrowsingMode)`? Then perhaps the callers could be modified to
provide PBM state, assuming they have that information:
https://dxr.mozilla.org/mozilla-
central/search?q=%2Bcallers%3A%22nsTransferable%3A%3AInit%28nsILoadContext+%2A%29%22
(I'm not sure if this is a practical idea or not, so feel free to
disagree.)
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/21830#comment:12>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs