Re: Patch: NULL pointer deref in pcf_check_dbms_client() with unreadable map file

2018-05-08 Thread Wietse Venema
Viktor Dukhovni: > - if ((fp = vstream_fopen(cf_file, O_RDONLY, 0)) == 0 > - && errno != EACCES) { > - msg_warn("open \"%s\" configuration \"%s\": %m", > - dp->db_type, cf_file); > + if ((fp = vstream_fopen(cf_file, O_RDONLY, 0)) == 0) { > + if (errn

Patch: NULL pointer deref in pcf_check_dbms_client() with unreadable map file

2018-05-08 Thread Viktor Dukhovni
On Tue, May 08, 2018 at 10:39:56AM -0300, Andreas Hasenack wrote: > Just before there is a check for fp == 0, but looking for errno != EACCES, > and EACCES is exactly what I got: > if ((fp = vstream_fopen(cf_file, O_RDONLY, 0)) == 0 > && errno != EACCES) { > msg_war