[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [FD] Hilariously Bad SQRL Implementation



Apparently, a barebones Android client exists.

https://twitter.com/SGgrc/status/501201214613123072

Unfortunately, I don't have access to the source code. Otherwise, I might
examine, thoroughly, what it's trying to do and write the spec for him.


On Wed, Aug 20, 2014 at 5:24 PM, Travis Biehn <tbiehn@xxxxxxxxx> wrote:

> An additional note, the SQRL protocol itself is pretty poorly documented
> on the GRC website:
> https://www.grc.com/sqrl/details.htm
>
> Instead of writing technical specifications like online sales copy for
> penis pills perhaps we can get a draft in an ... RFC like format.
>
>
> On Sun, Aug 17, 2014 at 8:22 PM, Scott Arciszewski <kobrasrealm@xxxxxxxxx>
> wrote:
>
>> If any of you are familiar with Stephen Gibson's SQRL protocol for user
>> authentication (really neat idea), you might have come across this PHP
>> implementation before: https://github.com/geir54/php-sqrl
>>
>> Unfortunately, this library is actually pretty terrible. Not only does it
>> pass all of the data off to a Heroku app to perform the signature
>> verification, it is also vulnerable to SQL Injection:
>>
>>
>> https://github.com/geir54/php-sqrl/blob/0fa574520a1843a33a84c3985f934e84af6f2042/sqrl_verify.php#L39-59
>>
>> I thought about submitting a pull request to fix this, but I don't believe
>> there is honestly much here to salvage. So, I'm writing my own
>> implementation here:
>>
>> https://github.com/darkitecht/php-sqrl <- Not ready, at all, for even
>> beta
>> testing.
>>
>> P.S. Also, it uses mt_rand() for challenge generation in a crypto library.
>> Tsk tsk.
>>
>> _______________________________________________
>> Sent through the Full Disclosure mailing list
>> http://nmap.org/mailman/listinfo/fulldisclosure
>> Web Archives & RSS: http://seclists.org/fulldisclosure/
>>
>
>
>
> --
> Twitter <https://twitter.com/tbiehn> | LinkedIn
> <http://www.linkedin.com/in/travisbiehn> | GitHub
> <http://github.com/tbiehn> | TravisBiehn.com <http://www.travisbiehn.com> |
> Google Plus <https://plus.google.com/+TravisBiehn>
>

_______________________________________________
Sent through the Full Disclosure mailing list
http://nmap.org/mailman/listinfo/fulldisclosure
Web Archives & RSS: http://seclists.org/fulldisclosure/