Re: Login forms autofill

WhiteWinterWolf <[email protected]> Tue, 14 Nov 2017 12:02:40 +0100
Newsgroups gmane.comp.mozilla.security
Message-ID <[email protected]>
Hello Tanvi,

I already knew this setting, it highlights the desire from Mozilla to 
keep the autofill feature active as much as possible, even if this means 
security trade-offs:

- Initial issue: autofill fills credential even on forms submitting to 
unusual third-party websites, some people expressed some concerns that 
the autofill feature should be disabled but instead Firefox was modified 
to more closely scrutinize the authentication form submission URL before 
automatically filling it.

- Then it appeared that an attacker in control of the DNS answers can 
keep the original URL while still redirecting automatic the 
authentication forms submission to its own servers, so the setting you 
mention has been added to limit autofilling to HTTPS forms.

- Then it appeared that JavaScript can be used to change the submission 
URL once the authentication form has been automatically filled. I've 
just checked with Firefox 52.4.0 on Linux Debian, Firefox is still 
vulnerable to this attack: an attacker just has to change the submission 
URL after Firefox automatically filled users credentials to send them to 
any arbitrary HTTPS URL (frankly, I thought this was solved a long time 
ago and reinforces me even more in my opinion).

- Would even Firefox be modified to handle automatically filled 
authentication forms as a special case and turn some properties 
read-only from JavaScript, potentially breaking some websites by the 
way, this wouldn't prevent attacks relying on malicious or leaked CA 
certificates for instance, + any additional techniques than the ones 
mentioned above which most likely *will* be discovered in the future, as 
long as browsers provide such functionality.


In my opinion, as a rule a web page should *never* be able to 
automatically extract sensitive information from the browser's database 
without user's consent.


- You can try to filter the form and submission URLs, the protocols, the 
JavaScript methods, server certificates and I don't know what else and 
still leave gaping holes in the process.

- Or you can simply require the user to click on an authentication form 
to fill it. Problem solved.

In my opinion, the second option is both more secure, easier to 
implement and more user friendly. But that's just my opinion.

Regards,
Simon.


Le 13/11/2017 à 22:27, Tanvi Vyas a écrit :
> Hello Simon,
> 
> Note that we also have an additional preference - signon.autofillForms.http
> - that is set to false by default.  This preference turns of autofilling
> passwords on HTTP pages without user interaction.  The user has to select
> the username in order for the password to be filled.  The reason this
> preference was added was to prevent attacks like the one you reference.
> 
> Thanks!
> 
> ~Tanvi
> 
> On Thu, Nov 2, 2017 at 10:18 AM, WhiteWinterWolf <
> [email protected]> wrote:
> 
>> Le 02/11/2017 à 17:56, Gervase Markham a écrit :
>>
>>> On 02/11/17 11:23, WhiteWinterWolf wrote:
>>>
>>>> In particular, I wonder why there is an official recommendation to
>>>> change `signon.autofillForms` default value while, at the same time,
>>>> keeping the setting out of reach of casual users.
>>>>
>>>
>>> The site you reference is not official, and so what is says is not an
>>> official recommendation. It seems the Firefox developers have decided
>>> that "true" is the correct value for this preference, and have further
>>> decided it doesn't need UI.
>>>
>>
>> Thank you for your answer. This is sad (from my personal point-of-view)
>> but this makes things logical now,
>>
>> Regards,
>> Simon.
>>
>> --
>> WhiteWinterWolf
>> https://www.whitewinterwolf.com
>> _______________________________________________
>> dev-security mailing list
>> [email protected]
>> https://lists.mozilla.org/listinfo/dev-security
>>
> _______________________________________________
> dev-security mailing list
> [email protected]
> https://lists.mozilla.org/listinfo/dev-security
> 

-- 
WhiteWinterWolf
https://www.whitewinterwolf.com
_______________________________________________
dev-security mailing list
[email protected]
https://lists.mozilla.org/listinfo/dev-security