Re: Re: PHP Crypt functions - security audit

From: Date: Sat, 21 Sep 2013 20:32:17 +0000
Subject: Re: Re: PHP Crypt functions - security audit
References: 1 2 3 4 5 6 7 8 9  Groups: php.internals 
Request: Send a blank email to [email protected] to get a copy of this message
Honestly that didn't even occur to me. I'd *much* rather kill the
additional functions like you suggest. As long as I am not forced to use a
PHP.ini to run I'm happy. ini_set() works fine for this and unless someone
has a compelling reason not to I'll nix the functions.

On Saturday, September 21, 2013, Nikita Popov wrote:

>
> On Sat, Sep 21, 2013 at 10:18 PM, Daniel Lowrey <[email protected]<javascript:_e({},
> 'cvml', '[email protected]');>
> > wrote:
>
>> Hello security-conscious internals people!
>>
>> I've got (what believe to be) a pretty good working solution for the
>> problem of insecure-by-default stream encryption. I need to do some more
>> thorough testing before pushing it upstream to a public fork but here's
>> the
>> quick and dirty:
>>
>
> Sounds really great! One minor nitpick:
>
> - Global CA path defaults may be specified via new "openssl.cafile" and
>> "openssl.capath" php.ini directives. This has the advantage mentioned
>> upthread of allowing distros to customize the .ini file to point to an
>> existing CA file.
>>
>> - Global CA path defaults may be specified at runtime via two new
>> functions:
>>     + bool openssl_set_default_cafile(string $cafile)
>>     + bool openssl_set_default_capath(string $capath)
>>
>
> Why do we need these functions? Can't you just specify it with
> ini_set('openssl.cafile', $file)? I don't immediately see why we need
> additional functions to set those ini options.
>
> Nikita
>
>


Thread (25 messages)

« previous php.internals (#69257) next »