<html>
<head>
<style><!--
.hmmessage P
{
margin:0px;
padding:0px
}
body.hmmessage
{
font-size: 10pt;
font-family:Tahoma
}
--></style></head>
<body class='hmmessage'><div dir='ltr'>
<br><div>hi, new patch :).<br><br>Apparently was required also monitor_config_file_fallback().<br><br>&gt; Hi Ariel,<br>&gt; <br>&gt; actually I reread the patch and I think there is a simpler way to fix<br>&gt; the ticket..directly in the monitor_config_file() function, there is<br>&gt; already a stat() call:<br>&gt; 1823     ret = stat(file, &amp;file_stat);<br>&gt; 1824     if (ret &lt; 0) {<br>&gt; 1825         err = errno;<br>&gt; 1826         DEBUG(0, ("Could not stat file [%s]. Error [%d:%s]\n",<br>&gt; 1827                   file, err, strerror(err)));<br>&gt; 1828         return err;<br>&gt; 1829     }<br>&gt; <br>&gt; <br>&gt; So I think that simply extending the error handler with special-casing<br>&gt; ENOENT as you had in your patch would work fine:<br>&gt; +        if (ret == ENOENT) {<br>&gt; +            DEBUG(SSSDBG_MINOR_FAILURE,<br>&gt; +                 ("file [%s] is missing, but will skip this check.\n",<br>&gt; +                 RESOLV_CONF_PATH));<br>&gt; +        } else {<br>&gt; +            DEBUG(SSSDBG_MINOR_FAILURE,("Could not stat file [%s]. Error [%d:%s]\n",<br>&gt; +                  RESOLV_CONF_PATH, ret, strerror(ret)));<br>&gt; +        }<br>&gt; <br>&gt; _______________________________________________<br>&gt; sssd-devel mailing list<br>&gt; sssd-devel@lists.fedorahosted.org<br>&gt; https://lists.fedorahosted.org/mailman/listinfo/sssd-devel<br></div>                                               </div></body>
</html>